Issue #2489912 by mikeburrelljr, opdavies, larowlan: Update Drupal.org URL in core modules

core/modules/aggregator/src/Tests/FeedValidationTest.php

@@ -39,7 +39,7 @@ public function testValidation() {
     // Add feed.
     $feed = Feed::create([
       'title' => 'Feed 1',
-      'url' => 'http://drupal.org/planet/rss',
+      'url' => 'https://www.drupal.org/planet/rss.xml',
       'refresh' => 900,
     ]);
 
@@ -52,7 +52,7 @@ public function testValidation() {
     /* @var \Drupal\aggregator\FeedInterface $feed */
     $feed = Feed::create([
       'title' => 'Feed 1',
-      'url' => 'http://drupal.org/planet/rss',
+      'url' => 'https://www.drupal.org/planet/rss.xml',
       'refresh' => 900,
     ]);
 

core/modules/aggregator/src/Tests/Views/IntegrationTest.php

@@ -68,7 +68,7 @@ protected function setUp() {
   public function testAggregatorItemView() {
     $feed = $this->feedStorage->create(array(
       'title' => $this->randomMachineName(),
-      'url' => 'http://drupal.org/',
+      'url' => 'https://www.drupal.org/',
       'refresh' => 900,
       'checked' => 123543535,
       'description' => $this->randomMachineName(),
@@ -85,7 +85,7 @@ public function testAggregatorItemView() {
       $values['description'] = $this->randomMachineName();
       // Add a image to ensure that the sanitizing can be tested below.
       $values['author'] = $this->randomMachineName() . '<img src="http://example.com/example.png" \>"';
-      $values['link'] = 'http://drupal.org/node/' . mt_rand(1000, 10000);
+      $values['link'] = 'https://www.drupal.org/node/' . mt_rand(1000, 10000);
       $values['guid'] = $this->randomString();
 
       $aggregator_item = $this->itemStorage->create($values);

core/modules/comment/comment.module

@@ -55,7 +55,7 @@
  * Comments changed after this time may be marked new, updated, or read,
  * depending on their state for the current user. Defaults to 30 days ago.
  *
- * @todo Remove when http://drupal.org/node/1029708 lands.
+ * @todo Remove when https://www.drupal.org/node/1029708 lands.
  */
 define('COMMENT_NEW_LIMIT', REQUEST_TIME - 30 * 24 * 60 * 60);
 
@@ -203,8 +203,8 @@ function comment_node_links_alter(array &$node_links, NodeInterface $node, array
   // Comment links are only added to node entity type for backwards
   // compatibility. Should you require comment links for other entity types you
   // can do so by implementing a new field formatter.
-  // @todo Make this configurable from the formatter see
-  //   http://drupal.org/node/1901110
+  // @todo Make this configurable from the formatter. See
+  //   https://www.drupal.org/node/1901110.
 
   $links = \Drupal::service('comment.link_builder')->buildCommentedEntityLinks($node, $context);
   $node_links += $links;

core/modules/comment/comment.tokens.inc

@@ -18,7 +18,7 @@ function comment_token_info() {
     'needs-data' => 'comment',
   );
 
-  // @todo Make this work per field. See http://drupal.org/node/2031903
+  // @todo Make this work per field. See https://www.drupal.org/node/2031903.
   $entity['comment-count'] = array(
     'name' => t("Comment count"),
     'description' => t("The number of comments posted on an entity."),

core/modules/comment/src/CommentManager.php

@@ -195,7 +195,7 @@ public function forbiddenMessage(EntityInterface $entity, $field_name) {
    */
   public function getCountNewComments(EntityInterface $entity, $field_name = NULL, $timestamp = 0) {
     // @todo Replace module handler with optional history service injection
-    //   after http://drupal.org/node/2081585
+    //   after https://www.drupal.org/node/2081585.
     if ($this->currentUser->isAuthenticated() && $this->moduleHandler->moduleExists('history')) {
       // Retrieve the timestamp at which the current user last viewed this entity.
       if (!$timestamp) {
@@ -209,7 +209,7 @@ public function getCountNewComments(EntityInterface $entity, $field_name = NULL,
           }
           else {
             // Default to 30 days ago.
-            // @todo Remove once http://drupal.org/node/1029708 lands.
+            // @todo Remove once https://www.drupal.org/node/1029708 lands.
             $timestamp = COMMENT_NEW_LIMIT;
           }
         }

core/modules/config/config.module

@@ -16,7 +16,7 @@ function config_help($route_name, RouteMatchInterface $route_match) {
       $output = '';
       $output .= '<h3>' . t('About') . '</h3>';
       $output .= '<p>' . t('The Configuration manager module provides a user interface for importing and exporting configuration changes; i.e., for staging configuration data between multiple instances of this web site. For more information, see the online handbook entry for <a href="!url">Configuration manager module</a>', array(
-        '!url' => 'http://drupal.org/documentation/administer/config',
+        '!url' => 'https://www.drupal.org/documentation/administer/config',
       )) . '</p>';
       return $output;
 

core/modules/dblog/dblog.module

@@ -20,7 +20,7 @@ function dblog_help($route_name, RouteMatchInterface $route_match) {
     case 'help.page.dblog':
       $output = '';
       $output .= '<h3>' . t('About') . '</h3>';
-      $output .= '<p>' . t('The Database Logging module logs system events in the Drupal database. For more information, see the online handbook entry for the <a href="!dblog">Database Logging module</a>.', array('!dblog' => 'http://drupal.org/documentation/modules/dblog')) . '</p>';
+      $output .= '<p>' . t('The Database Logging module logs system events in the Drupal database. For more information, see the online handbook entry for the <a href="!dblog">Database Logging module</a>.', array('!dblog' => 'https://www.drupal.org/documentation/modules/dblog')) . '</p>';
       $output .= '<h3>' . t('Uses') . '</h3>';
       $output .= '<dl>';
       $output .= '<dt>' . t('Monitoring your site') . '</dt>';

core/modules/editor/editor.module

@@ -68,7 +68,7 @@ function editor_element_info_alter(&$types) {
  * Implements hook_form_FORM_ID_alter().
  */
 function editor_form_filter_admin_overview_alter(&$form, FormStateInterface $form_state) {
-  // @todo Cleanup column injection: http://drupal.org/node/1876718
+  // @todo Cleanup column injection: https://www.drupal.org/node/1876718.
   // Splice in the column for "Text editor" into the header.
   $position = array_search('name', $form['formats']['#header']) + 1;
   $start = array_splice($form['formats']['#header'], 0, $position, array('editor' => t('Text editor')));

core/modules/filter/src/FilterFormatFormBase.php

@@ -111,7 +111,7 @@ public function form(array $form, FormStateInterface $form_state) {
       '#suffix' => '</div>',
       // This item is used as a pure wrapping container with heading. Ignore its
       // value, since 'filters' should only contain filter definitions.
-      // @see http://drupal.org/node/1829202
+      // See https://www.drupal.org/node/1829202.
       '#input' => FALSE,
     );
     // Filter order (tabledrag).

core/modules/filter/src/Tests/FilterAPITest.php

@@ -67,9 +67,9 @@ function testCheckMarkupFilterOrder() {
    * Tests the ability to apply only a subset of filters.
    */
   function testCheckMarkupFilterSubset() {
-    $text = "Text with <marquee>evil content and</marquee> a URL: http://drupal.org!";
-    $expected_filtered_text = "Text with evil content and a URL: <a href=\"http://drupal.org\">http://drupal.org</a>!";
-    $expected_filter_text_without_html_generators = "Text with evil content and a URL: http://drupal.org!";
+    $text = "Text with <marquee>evil content and</marquee> a URL: https://www.drupal.org!";
+    $expected_filtered_text = "Text with evil content and a URL: <a href=\"https://www.drupal.org\">https://www.drupal.org</a>!";
+    $expected_filter_text_without_html_generators = "Text with evil content and a URL: https://www.drupal.org!";
 
     $actual_filtered_text = check_markup($text, 'filtered_html', '', array());
     $this->verbose("Actual:<pre>$actual_filtered_text</pre>Expected:<pre>$expected_filtered_text</pre>");

core/modules/filter/src/Tests/FilterUnitTest.php

@@ -205,20 +205,20 @@ function testCaptionFilter() {
 
     // All the tricky cases encountered at https://drupal.org/node/2105841.
     // A plain URL preceded by text.
-    $input = '<img data-caption="See http://drupal.org" src="llama.jpg" />';
-    $expected = '<figure><img src="llama.jpg" /><figcaption>See http://drupal.org</figcaption></figure>';
+    $input = '<img data-caption="See https://www.drupal.org" src="llama.jpg" />';
+    $expected = '<figure><img src="llama.jpg" /><figcaption>See https://www.drupal.org</figcaption></figure>';
     $this->assertIdentical($expected, $test_with_html_filter($input));
     $this->assertIdentical($input, $test_editor_xss_filter($input));
 
     // An anchor.
-    $input = '<img data-caption="This is a &lt;a href=&quot;http://drupal.org&quot;&gt;quick&lt;/a&gt; test…" src="llama.jpg" />';
-    $expected = '<figure><img src="llama.jpg" /><figcaption>This is a <a href="http://drupal.org">quick</a> test…</figcaption></figure>';
+    $input = '<img data-caption="This is a &lt;a href=&quot;https://www.drupal.org&quot;&gt;quick&lt;/a&gt; test…" src="llama.jpg" />';
+    $expected = '<figure><img src="llama.jpg" /><figcaption>This is a <a href="https://www.drupal.org">quick</a> test…</figcaption></figure>';
     $this->assertIdentical($expected, $test_with_html_filter($input));
     $this->assertIdentical($input, $test_editor_xss_filter($input));
 
     // A plain URL surrounded by parentheses.
-    $input = '<img data-caption="(http://drupal.org)" src="llama.jpg" />';
-    $expected = '<figure><img src="llama.jpg" /><figcaption>(http://drupal.org)</figcaption></figure>';
+    $input = '<img data-caption="(https://www.drupal.org)" src="llama.jpg" />';
+    $expected = '<figure><img src="llama.jpg" /><figcaption>(https://www.drupal.org)</figcaption></figure>';
     $this->assertIdentical($expected, $test_with_html_filter($input));
     $this->assertIdentical($input, $test_editor_xss_filter($input));
 

core/modules/language/src/LanguageNegotiatorInterface.php

@@ -100,7 +100,7 @@
  * @endcode
  *
  * For more information, see
- * @link http://drupal.org/node/1497272 Language Negotiation API @endlink
+ * @link https://www.drupal.org/node/1497272 Language Negotiation API @endlink
  */
 interface LanguageNegotiatorInterface {
 

core/modules/link/src/Plugin/Field/FieldFormatter/LinkSeparateFormatter.php

@@ -7,7 +7,7 @@
  * @todo
  * Merge into 'link' formatter once there is a #type like 'item' that
  * can render a compound label and content outside of a form context.
- * http://drupal.org/node/1829202
+ * @see https://www.drupal.org/node/1829202
  */
 
 namespace Drupal\link\Plugin\Field\FieldFormatter;

core/modules/link/src/Tests/LinkItemTest.php

@@ -82,7 +82,7 @@ public function testLinkItem() {
     $this->assertEqual($entity->field_test->options['query'], $parsed_url['query']);
 
     // Verify changing the field value.
-    $new_url = 'http://drupal.org';
+    $new_url = 'https://www.drupal.org';
     $new_title = $this->randomMachineName();
     $new_class = $this->randomMachineName();
     $entity->field_test->uri = $new_url;

core/modules/locale/locale.batch.inc

@@ -11,7 +11,7 @@
  * Load the common translation API.
  */
 // @todo Combine functions differently in files to avoid unnecessary includes.
-// Follow-up issue http://drupal.org/node/1834298
+// Follow-up issue: https://www.drupal.org/node/1834298.
 require_once __DIR__ . '/locale.translation.inc';
 
 /**

core/modules/locale/locale.bulk.inc

@@ -34,7 +34,7 @@
  * @todo
  *   Integrate with update status to identify projects needed and integrate
  *   l10n_update functionality to feed in translation files alike.
- *   See http://drupal.org/node/1191488.
+ *   See https://www.drupal.org/node/1191488.
  */
 function locale_translate_batch_import_files(array $options, $force = FALSE) {
   $options += array(
@@ -217,7 +217,7 @@ function locale_translate_batch_import($file, array $options, array &$context) {
         // Maximize the progress bar at 95% before completion, the batch API
         // could trigger the end of the operation before file reading is done,
         // because of floating point inaccuracies. See
-        // http://drupal.org/node/1089472
+        // https://www.drupal.org/node/1089472.
         $context['finished'] = min(0.95, $report['seek'] / filesize($file->uri));
         if (isset($options['message'])) {
           $context['message'] = t('!message (@percent%).', array('!message' => $options['message'], '@percent' => (int) ($context['finished'] * 100)));

core/modules/locale/locale.compare.inc

@@ -12,7 +12,7 @@
  * Load common APIs.
  */
 // @todo Combine functions differently in files to avoid unnecessary includes.
-// Follow-up issue http://drupal.org/node/1834298
+// Follow-up issue: https://www.drupal.org/node/1834298.
 require_once __DIR__ . '/locale.translation.inc';
 
 /**
@@ -70,7 +70,7 @@ function locale_translation_build_projects() {
           // dev release is the one. Releases are sorted the most recent first.
           // For example the major release number for a contrib module
           // 8.x-2.x-dev is "2", for core 8.1.0-dev is "8".
-          // @todo http://drupal.org/node/1774024 Make a helper function.
+          // @todo https://www.drupal.org/node/1774024 Make a helper function.
           if ($project_release['version_major'] == $matches[1] &&
               (!isset($project_release['version_extra']) || $project_release['version_extra'] != 'dev')) {
             $release = $project_release;

core/modules/locale/locale.fetch.inc

@@ -9,7 +9,7 @@
  * Load the common translation API.
  */
 // @todo Combine functions differently in files to avoid unnecessary includes.
-// Follow-up issue http://drupal.org/node/1834298
+// Follow-up issue: https://www.drupal.org/node/1834298.
 require_once __DIR__ . '/locale.translation.inc';
 
 /**

core/modules/locale/locale.module

@@ -199,7 +199,8 @@ function locale_theme() {
  * Implements hook_ENTITY_TYPE_insert() for 'configurable_language'.
  */
 function locale_configurable_language_insert(ConfigurableLanguageInterface $language) {
-  // @todo move these two cache clears out. See http://drupal.org/node/1293252
+  // @todo move these two cache clears out. See
+  //   https://www.drupal.org/node/1293252.
   // Changing the language settings impacts the interface: clear render cache.
   \Drupal::cache('render')->deleteAll();
   // Force JavaScript translation file re-creation for the new language.
@@ -210,7 +211,8 @@ function locale_configurable_language_insert(ConfigurableLanguageInterface $lang
  * Implements hook_ENTITY_TYPE_update() for 'configurable_language'.
  */
 function locale_configurable_language_update(ConfigurableLanguageInterface $language) {
-  // @todo move these two cache clears out. See http://drupal.org/node/1293252
+  // @todo move these two cache clears out. See
+  //   https://www.drupal.org/node/1293252.
   // Changing the language settings impacts the interface: clear render cache.
   \Drupal::cache('render')->deleteAll();
   // Force JavaScript translation file re-creation for the modified language.
@@ -465,7 +467,7 @@ function locale_system_remove($components) {
       locale_translate_delete_translation_files($list, array());
 
       // Remove translatable projects.
-      // Followup issue http://drupal.org/node/1842362 to replace the
+      // Follow-up issue https://www.drupal.org/node/1842362 to replace the
       // {locale_project} table. Then change this to a function call.
       \Drupal::service('locale.project')->deleteMultiple($list);
 
@@ -487,7 +489,7 @@ function locale_cache_flush() {
  * Implements hook_js_alter().
  */
 function locale_js_alter(&$javascript, AttachedAssetsInterface $assets) {
-  // @todo remove this in https://www.drupal.org/node/2421323
+  // @todo Remove this in https://www.drupal.org/node/2421323.
   $files = array();
   foreach ($javascript as $item) {
     if (isset($item['type']) && $item['type'] == 'file') {
@@ -919,7 +921,7 @@ function locale_translation_get_status($projects = NULL, $langcodes = NULL) {
  *   File object also containing timestamp when the translation is last updated.
  */
 function locale_translation_status_save($project, $langcode, $type, $data) {
-  // Followup issue: http://drupal.org/node/1842362
+  // Follow-up issue: https://www.drupal.org/node/1842362.
   // Split status storage per module/language and expire individually. This will
   // improve performance for large sites.
 

core/modules/locale/locale.translation.inc

@@ -58,8 +58,8 @@ function locale_translation_get_projects(array $project_names = array()) {
   if (empty($projects)) {
     // Get project data from the database.
     $row_count = \Drupal::service('locale.project')->countProjects();
-    // http://drupal.org/node/1777106 is a follow-up issue to make the check for
-    // possible out-of-date project information more robust.
+    // https://www.drupal.org/node/1777106 is a follow-up issue to make the
+    // check for possible out-of-date project information more robust.
     if ($row_count == 0 && \Drupal::moduleHandler()->moduleExists('update')) {
       module_load_include('compare.inc', 'locale');
       // At least the core project should be in the database, so we build the
@@ -226,7 +226,7 @@ function locale_translation_source_check_file($source) {
  *   - "keep": TRUE to keep the downloaded file.
  */
 function locale_translation_source_build($project, $langcode, $filename = NULL) {
-  // Followup issue: http://drupal.org/node/1842380
+  // Follow-up issue: https://www.drupal.org/node/1842380.
   // Convert $source object to a TranslatableProject class and use a typed class
   // for $source-file.