This is a backport. See #2762953: Select elements should use strict comparison, the problem is exactly the same.

CommentFileSizeAuthor
#5 select_elements_should-2770641-5.patch1.91 KBcilefen
#2 select_elements_should-2770641-2.patch1.8 KBcilefen
#2 select_elements_should-2770641-2-TEST.patch1.19 KBcilefen

Comments

cilefen created an issue. See original summary.

cilefen’s picture

cilefen commented
Status: Active » Needs review
Issue tags: +govcon2016
StatusFileSize
new select_elements_should-2770641-2-TEST.patch1.19 KB
new select_elements_should-2770641-2.patch1.8 KB

The last submitted patch, 2: select_elements_should-2770641-2-TEST.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 2: select_elements_should-2770641-2.patch, failed testing.

cilefen’s picture

cilefen commented
Status: Needs work » Needs review
StatusFileSize
new select_elements_should-2770641-5.patch1.91 KB
cilefen’s picture

cilefen commented
Title: Select elements should use strict comparison » Backport from D8: Select elements should use strict comparison
stefan.r’s picture

stefan.r commented 
+++ b/includes/form.inc
@@ -2845,7 +2845,10 @@ function form_select_options($element, $choices = NULL) {
+      if ($value_is_array) {
+        $element['#value'] = array_map('strval', $element['#value']);
+      }

I'd like to see this go into D8 first, but other than that, RTBC, and thanks for the backport!

quietone’s picture

quietone commented
Status: Needs review » Closed (outdated)
Issue tags: +Bug Smash Initiative

I tested this on a fresh install of the standard profile on Drupal 7.90-dev. I followed the steps in #4 and was not able to reproduce the problem. I don't know when it was fixed, just closing as outdated.

If that is wrong, reopen and explain what needs to be done.

Thanks