Normalize default value of checkbox custom field

[Jun Omae]

Creating new ticket with checkbox field which has 42 for default value via web ui, the field would have 1 or 0.

[ticket-custom]
check = checkbox
check.value = 42

sqlite> SELECT t.id, t.summary, c.value FROM ticket AS t LEFT JOIN ticket_custom AS c ON c.ticket=t.id AND c.name='check';
1|Checked|1
2|Unchecked|0

However, the default value would be stored for the checkbox field via application code instead of 1|0.

$ ~/venv/trac/1.0.15/bin/python
Python 2.5.6 (r256:88840, Oct 21 2014, 22:49:55)
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from trac.env import Environment
>>> env = Environment('/dev/shm/tracenv')
>>> from trac.ticket.model import Ticket
>>> ticket = Ticket(env)
>>> ticket['summary'] = 'via plugin'
>>> ticket.insert()
3
>>> for name in sorted(ticket.values): name, ticket[name]
...
('changetime', datetime.datetime(2017, 7, 6, 12, 42, 59, 479729, tzinfo=<FixedOffset "UTC" 0:00:00>))
(u'check', u'42')   # <== neither 1 nor 0
('owner', '')
('priority', u'major')
('summary', 'via plugin')
('time', datetime.datetime(2017, 7, 6, 12, 42, 59, 479729, tzinfo=<FixedOffset "UTC" 0:00:00>))
('type', u'defect')

I consider we should normalize default value for checkbox custom fields to have 1|0.

[Ryan J Ollos]

Any value other than 1 results in the checkbox defaulting to 0: tags/trac-1.2.2/trac/ticket/templates/ticket.html@:267#L244​.

I considered using as_int, but maybe better to use as_bool:

trac/ticket/api.py

@@ -27 +27 @@
 from trac.core import *
 from trac.perm import IPermissionRequestor, PermissionCache, PermissionSystem
 from trac.resource import IResourceManager
-from trac.util import Ranges, as_int
+from trac.util import Ranges, as_bool, as_int
 from trac.util.text import shorten_line
 from trac.util.translation import _, N_, deactivate, gettext, reactivate
 from trac.wiki import IWikiSyntaxProvider, WikiParser
@@ -422 +422 @@
                     field['optional'] = True
                     if '' in field['options']:
                         field['options'].remove('')
+            elif field['type'] == 'checkbox':
+                field['value'] = '1' if as_bool(field['value']) else '0'
             elif field['type'] == 'text':
                 field['format'] = config.get(name + '.format', 'plain')
             elif field['type'] == 'textarea':

That would allow checkbox.value = true to be accepted. We don't need to document that as allowed, but it might be better fallback behavior.

>>> from trac.env import Environment
>>> from trac.ticket.model import Ticket
>>> env = Environment('/Users/rjollos/Documents/Workspace/trac-dev/tracenvs/proj-1
.2')
>>> env.config.get('ticket-custom', 'checkbox1.value')
u'42'
>>> ticket = Ticket(env)
>>> ticket['summary'] = 'via plugin'
>>> ticket.insert()
21
>>> for name in sorted(ticket.values): name, ticket[name]
...
('changetime', datetime.datetime(2017, 10, 2, 23, 28, 12, 648549, tzinfo=<FixedOff
set "UTC" 0:00:00>))
(u'checkbox1', 1)
('owner', u'< default >')
('priority', u'blocker')
('summary', 'via plugin')
('time', datetime.datetime(2017, 10, 2, 23, 28, 12, 648549, tzinfo=<FixedOffset "U
TC" 0:00:00>))

[Ryan J Ollos]

Fixed in r16348, r16349.