Query page javascript errors

[Bryan Kitts]

DynamicFieldsPlugin seems to throw an error on Trac query page for my configuration (Trac 0.11.5, tracd, multiple Trac databases on same server, Firefox 3.6 and IE7).

The only obvious effect of this is that the Filters and Columns sections of the Trac Query page are no longer collapsible. Not sure if it causes any other problems.

The cause seems to be in dynfields.html. When I navigate to the Query page http://dblonws29921.uk.db.com/MVProjects/query window.location.pathname is actually set to '/MVProjects/query' so the code chooses the wrong branch.

Adding the substr() below fixes it for me. Not sure if there's a neater solution.

jQuery(document).ready(function($){
    var triggers = get_triggers(); // trigger fields
    
   if (window.location.pathname.substr(window.location.pathname.length-6,6) == '/query'){

[Hope that makes sense --Bryan]

[Jan Beilicke]

I had a similar issue with Trac 0.12 and plugin branch 0.11. I fixed it with:

$ svn diff ./dynfields/templates/dynfields.html 
Index: dynfields/templates/dynfields.html
===================================================================
--- dynfields/templates/dynfields.html  (revision 9938)
+++ dynfields/templates/dynfields.html  (working copy)
@@ -70,7 +70,7 @@
 jQuery(document).ready(function($){
     var triggers = get_triggers(); // trigger fields
     
-       if (window.location.pathname == '/query'){
+       if (window.location.pathname.indexOf('/query')){
         // hide all "hide_always" fields
         jQuery.each(triggers, function(trigger,specs){
                        jQuery.each(specs, function(i,spec){

[Rob Guttman]

(In [9939]) refs #8553: fixed two JS errors.

[Rob Guttman]

Thanks for reporting this and the suggested fixes. I just fixed this and another JS error in the plugin on the 0.11 branch only. Please let me know if it now works for you or not. Thanks.

[Jan Beilicke]

Replying to robguttman:

Thanks for reporting this and the suggested fixes. I just fixed this and another JS error in the plugin on the 0.11 branch only. Please let me know if it now works for you or not. Thanks.

Works fine (for me), thanks for the quick patch!

BTW: The other fixed error regarding the id was causing my problem. It worked with the patch I mentioned above, so I didn't get into detail regarding the underlying issue when adding my comment. Sorry for that.

[Rob Guttman]

Great to hear! Closing.