Can't set owner to $USER in a query when restrict_owner is on

[Kamil Kisiel <kamil@…>]

Using the $USER variable normally allows you to specify the currently logged in user in a query, but there doesn't appear to be any way to achieve this when the restrict_owner option is set.

[osimons]

The issue is correctly stated.

Seeing 'reporter' with flexible options like 'is', 'is not', 'contains' and so on makes me wish for the same for 'owner' even with restrict_owner enabled. For reporting purposes that makes a lot of sense.

And, there is also an e-mail obfuscation issue involved: anyone with just TICKET_VIEW permission will get it obfuscated when viewing a specific ticket (properties are not available with just view), but in Custom Query it shows up as a filter unmodified.

+1 on making 'owner' like 'reporter' regardless of restrict_owner setting.

[osimons]

When fixing this, also remember to test the issue of reopened comment:ticket:1467:14.

[osimons]

Here is a patch to fix make the field 'owner' field always behave like a text field regardless of setting:

• more flexible querying using also contains, startswith, endswith and so on.
• supports $USER.
• e-mail obfuscation is no longer an issue if username is an email address and user only has TICKET_VIEW permissions.

trac/ticket/query.py

@@ -593 +593 @@
         for field in self.fields:
             if field['type'] == 'textarea':
                 continue
+            if field['name'] == 'owner' and field['type'] == 'select':
+                # restrict_owner = true => make it a regular text field instead
+                field['type'] = 'text'
+                del field['options']
             field_data = {}
             field_data.update(field)
             del field_data['name']

Does other devs agree with this change?

[osimons]

Remember seeing some other tickets and discussions that want similar treatment for versions, milestones, components and all similar select fields ('all versions starting with 0.10...'), and perhaps the right way to go about this is an option in the trac.ini [query] section that turns a switch to convert all select fields to regular text fields?