diff --git a/core/includes/form.inc b/core/includes/form.inc index c33ff6f..528ff71 100644 --- a/core/includes/form.inc +++ b/core/includes/form.inc @@ -858,8 +858,6 @@ function form_select_options($element, $choices = NULL) { // isset() fails in this situation. $value_valid = isset($element['#value']) || array_key_exists('#value', $element); $value_is_array = $value_valid && is_array($element['#value']); - // Check if the element is multiple select and no value has been selected. - $empty_value = (empty($element['#value']) && !empty($element['#multiple'])); $options = ''; foreach ($choices as $key => $choice) { if (is_array($choice)) { @@ -872,8 +870,7 @@ function form_select_options($element, $choices = NULL) { } else { $key = (string) $key; - $empty_choice = $empty_value && $key == '_none'; - if ($value_valid && ((!$value_is_array && (string) $element['#value'] === $key || ($value_is_array && in_array($key, $element['#value']))) || $empty_choice)) { + if ($value_valid && (!$value_is_array && (string) $element['#value'] === $key || ($value_is_array && in_array($key, $element['#value'])))) { $selected = ' selected="selected"'; } else { diff --git a/core/modules/comment/comment.module b/core/modules/comment/comment.module index d0ea461..dbf72b1 100644 --- a/core/modules/comment/comment.module +++ b/core/modules/comment/comment.module @@ -436,13 +436,6 @@ function comment_entity_view(EntityInterface $entity, EntityViewDisplayInterface // http://drupal.org/node/1901110 return; } - if ($view_mode == 'search_index' || $view_mode == 'search_result' || $view_mode == 'print') { - // Do not add any links if the entity displayed for: - // - search indexing. - // - constructing a search result excerpt. - // - print. - return; - } $fields = \Drupal::service('comment.manager')->getFields('node'); foreach ($fields as $field_name => $detail) { // Skip fields that entity does not have. @@ -521,9 +514,11 @@ function comment_entity_view(EntityInterface $entity, EntityViewDisplayInterface } } } - else { + elseif ($view_mode != 'search_index' && $view_mode != 'search_result') { // Entity in other view modes: add a "post comment" link if the user is // allowed to post comments and if this entity is allowing new comments. + // But we don't want this link if we're building the entity for search + // indexing or constructing a search result excerpt. if ($commenting_status == COMMENT_OPEN) { $comment_form_location = $instance->getSetting('form_location'); if (user_access('post comments')) { diff --git a/core/modules/comment/lib/Drupal/comment/Plugin/Field/FieldFormatter/CommentDefaultFormatter.php b/core/modules/comment/lib/Drupal/comment/Plugin/Field/FieldFormatter/CommentDefaultFormatter.php index 3250990..bcdf86f 100644 --- a/core/modules/comment/lib/Drupal/comment/Plugin/Field/FieldFormatter/CommentDefaultFormatter.php +++ b/core/modules/comment/lib/Drupal/comment/Plugin/Field/FieldFormatter/CommentDefaultFormatter.php @@ -143,8 +143,8 @@ public function viewElements(FieldItemListInterface $items) { } // Append comment form if the comments are open and the form is set to - // display below the entity. Do not show the form for the print view mode. - if ($status == COMMENT_OPEN && $comment_settings['form_location'] == COMMENT_FORM_BELOW && $this->viewMode != 'print') { + // display below the entity. + if ($status == COMMENT_OPEN && $comment_settings['form_location'] == COMMENT_FORM_BELOW) { // Only show the add comment form if the user has permission. if ($this->currentUser->hasPermission('post comments')) { // All users in the "anonymous" role can use the same form: it is fine diff --git a/core/modules/comment/lib/Drupal/comment/Tests/CommentBookTest.php b/core/modules/comment/lib/Drupal/comment/Tests/CommentBookTest.php deleted file mode 100644 index 4d92e9d..0000000 --- a/core/modules/comment/lib/Drupal/comment/Tests/CommentBookTest.php +++ /dev/null @@ -1,84 +0,0 @@ - 'Book commenting', - 'description' => 'Test visibility of comments on book pages.', - 'group' => 'Book', - ); - } - - function setUp() { - parent::setUp(); - - // Create comment field on book. - \Drupal::service('comment.manager')->addDefaultField('node', 'book'); - } - - /** - * Tests comments in book export. - */ - public function testBookCommentPrint() { - $book_node = entity_create('node', array( - 'type' => 'book', - 'title' => 'Book title', - 'body' => 'Book body', - )); - $book_node->book['bid'] = 'new'; - $book_node->save(); - - $comment_subject = $this->randomName(8); - $comment_body = $this->randomName(8); - $comment = entity_create('comment', array( - 'subject' => $comment_subject, - 'comment_body' => $comment_body, - 'entity_id' => $book_node->id(), - 'entity_type' => 'node', - 'field_name' => 'comment', - 'status' => CommentInterface::PUBLISHED, - )); - $comment->save(); - - $commenting_user = $this->drupalCreateUser(array('access printer-friendly version', 'access comments', 'post comments')); - $this->drupalLogin($commenting_user); - - $this->drupalGet('node/' . $book_node->id()); - - $this->assertText($comment_subject, 'Comment subject found'); - $this->assertText($comment_body, 'Comment body found'); - $this->assertText(t('Add new comment'), 'Comment form found'); - $this->assertField('subject', 'Comment form subject found'); - - $this->drupalGet('book/export/html/' . $book_node->id()); - - $this->assertText(t('Comments'), 'Comment thread found'); - $this->assertText($comment_subject, 'Comment subject found'); - $this->assertText($comment_body, 'Comment body found'); - - $this->assertNoText(t('Add new comment'), 'Comment form not found'); - $this->assertNoField('subject', 'Comment form subject not found'); - } - -} diff --git a/core/modules/node/lib/Drupal/node/Tests/Views/FrontPageTest.php b/core/modules/node/lib/Drupal/node/Tests/Views/FrontpageTest.php similarity index 100% rename from core/modules/node/lib/Drupal/node/Tests/Views/FrontPageTest.php rename to core/modules/node/lib/Drupal/node/Tests/Views/FrontpageTest.php diff --git a/core/modules/options/lib/Drupal/options/Tests/OptionsWidgetsTest.php b/core/modules/options/lib/Drupal/options/Tests/OptionsWidgetsTest.php index 41ba632..57d3e14 100644 --- a/core/modules/options/lib/Drupal/options/Tests/OptionsWidgetsTest.php +++ b/core/modules/options/lib/Drupal/options/Tests/OptionsWidgetsTest.php @@ -378,7 +378,6 @@ function testSelectListMultiple() { // Display form: with no field data, nothing is selected. $this->drupalGet('entity_test/manage/' . $entity->id()); - $this->assertOptionSelected("edit-card-2", '_none'); $this->assertNoOptionSelected('edit-card-2', 0); $this->assertNoOptionSelected('edit-card-2', 1); $this->assertNoOptionSelected('edit-card-2', 2); diff --git a/core/modules/simpletest/lib/Drupal/simpletest/TestBase.php b/core/modules/simpletest/lib/Drupal/simpletest/TestBase.php index cf162fd..608318e 100644 --- a/core/modules/simpletest/lib/Drupal/simpletest/TestBase.php +++ b/core/modules/simpletest/lib/Drupal/simpletest/TestBase.php @@ -1053,15 +1053,6 @@ private function prepareEnvironment() { // Reset and create a new service container. $this->container = new ContainerBuilder(); - // @todo Remove this once this class has no calls to t() and format_plural() - $this->container->setParameter('language.default_values', Language::$defaultValues); - $this->container->register('language.default', 'Drupal\Core\Language\LanguageDefault') - ->addArgument('%language.default_values%'); - $this->container->register('language_manager', 'Drupal\Core\Language\LanguageManager') - ->addArgument(new Reference('language.default')); - $this->container->register('string_translation', 'Drupal\Core\StringTranslation\TranslationManager') - ->addArgument(new Reference('language_manager')); - // Register info parser. $this->container->register('info_parser', 'Drupal\Core\Extension\InfoParser'); diff --git a/core/modules/taxonomy/lib/Drupal/taxonomy/Tests/TermFieldMultipleVocabularyTest.php b/core/modules/taxonomy/lib/Drupal/taxonomy/Tests/TermFieldMultipleVocabularyTest.php index 81e82a9..0982864 100644 --- a/core/modules/taxonomy/lib/Drupal/taxonomy/Tests/TermFieldMultipleVocabularyTest.php +++ b/core/modules/taxonomy/lib/Drupal/taxonomy/Tests/TermFieldMultipleVocabularyTest.php @@ -87,9 +87,7 @@ function testTaxonomyTermFieldMultipleVocabularies() { // Submit an entity with both terms. $this->drupalGet('entity_test/add'); - // Just check if the widget for the select is displayed, the NULL value is - // used to ignore the value check. - $this->assertFieldByName("{$this->field_name}[]", NULL, 'Widget is displayed.'); + $this->assertFieldByName("{$this->field_name}[]", '', 'Widget is displayed.'); $edit = array( 'user_id' => mt_rand(0, 10), 'name' => $this->randomName(), @@ -127,9 +125,7 @@ function testTaxonomyTermFieldMultipleVocabularies() { // The widget should still be displayed. $this->drupalGet('entity_test/add'); - // Just check if the widget for the select is displayed, the NULL value is - // used to ignore the value check. - $this->assertFieldByName("{$this->field_name}[]", NULL, 'Widget is still displayed.'); + $this->assertFieldByName("{$this->field_name}[]", '', 'Widget is still displayed.'); // Term 1 should still pass validation. $edit = array(