Custom query - radio button shows wrong selection even though results are correct

[sid@…]

I have a custom ticket field called "crb_approved", and I created this custom query with a link from another page:

[query:status=closed&crb_approved=0&order=id Unapproved closed tickets]

When I view the report, the results are correct, but the crb_approved radio button displayed on the Custom Query page shows up with "True" (should be "False") for the crb_approved field based on the query.

[sid]

The reason I noticed this is when I close some of those tickets in the custom query, then hit update, the incorrect value for the checkbox is used, and the wrong results come back. Quite confusing.

[sid]

The workaround is to use this query instead:

[query:status=closed&crb_approved=!1&order=id Unapproved closed tickets]

It chooses the correct radio button, and pressing Update on the query afterwards displays the correct results.

[sid]

I can duplicate this easily on 0.11.

1. Add a checkbox custom field to trac.ini.

   approved = checkbox
   approved.label = Approved by me
   approved.value = 0
   

2. Create a few sample tickets, checking the checkbox in some, and not in others
3. Run a custom query using that custom field as a filter
4. Try to choose "No" for that custom field, then press "Update".

The "Yes" radio button will show every time. The results are correct though, but if you hit update again without paying attention, the results are dramatically different.

[sid]

Turns out to be a pretty easy fix:

templates/query.html

@@ -85 +85 @@
 
                         <py:when test="'checkbox'">
                           <input type="radio" id="${field_name}_on" name="$field_name" value="1"
-                                 checked="${constraint.mode != '!' or None}" />
+                                 checked="${constraint.mode != '!' or constraint_value == '1' or None}" />
                           <label for="${field_name}_on">yes</label>
                           <input type="radio" id="${field_name}_off" name="$field_name" value="0"
-                                 checked="${constraint.mode == '!' or None}" />
+                                 checked="${constraint.mode == '!' or constraint_value == '0' or None}" />
                           <label for="${field_name}_off">no</label>
                         </py:when>

I tested this and it fixes the example in comment:3, and also my originally reported issue as well.

[Christian Boos]

OK, I'll try to commit that this evening.

[Christian Boos]

Applied in r4462. Also fixed the initial value for the _off radio button when added by Javascript by using "0" instead of "!1". Besides being clearer, this is also not the same: "!1" also matched the tickets for which the field was unset.

There's still a very minor issue when the field is not added by Javascript. As this is very minor and will probably be fixed when the fields are reworked (WorkFlow branch), I'm closing this as fixed.

Thanks for the patch, sid!