Can't change select dropdown value

[Chris Heller]

I'm not sure if this is just something with particular environment configuration, but I was repeatedly getting stuck on not being able to change the value for a dynamic variable. The dropdown was populated properly, but the value would not stick. The form was properly submitted to the server though.

it turns out the problem is in this line

input.val(select.val()); // set hidden text input

The select.val() portion was always returning undefined for some reason, even when an option was selected. I tested this on a few different browsers and had the same behavior.

[Chris Heller]

Here's the diff for what ended up fixing it.

dynvars/templates/dynvars.html

@@ -25 +25 @@
         var select = jQuery(this);
         var name = select.attr('id');
         var input = jQuery('#prefs input[name='+name+']');
-              input.val(select.val()); // set hidden text input
+              var optionSelected = select.find("option:selected");
+              var valueSelected  = optionSelected.attr("value");
+              input.val(valueSelected)  // set hidden text input
               input.parents('form:first').submit();
           });
       }

By grabbing the selected option and getting its value attribute (.val() doesn't give the correct answer here either) and using that to populate the hidden input field, then things worked.

I didn't do any further troubleshooting to determine what was causing the original problem though.

[Ryan J Ollos]

I can't reproduce and the ​code appears correct. Could you explain a bit more what you mean by the value would not stick? Do you see any errors in the JavaScript console? Do you have any other plugins installed? Which version of Trac are you running?

[Ryan J Ollos]

In 14917:

0.2.0dev: Replace virtual script with a script.

Use add_script_data to pass data to the script.

Refs #12538.

[Ryan J Ollos]

Could you please try the new version of the plugin and report back? The code is simpler now, the performance should be better and it should be easier to debug.

[Ryan J Ollos]

In 14918:

0.2.0dev: Remove debug alert dialog.

Refs #12538.

[Ryan J Ollos]

Feel free to follow-up if you continue to have issues.