Provide manual addition of further tickets

[Marcel Gestewitz]

Since the PageTickets macro is related to the TicketQuery it would be a nice feature if the macro allows the manual entry of further ticket number as argument.

This is of interest for the case that someone wants to refer to further tickets which are not listed somewhere else in the text.

I managed this by append passed ticket to those which were found with the regular expression:

...
args, kw = parse_args(content)
if kw.has_key('id'):
        tickets.append(kw['id'])
if not tickets:
        return 'No tickets found'
...

Now it is possible to pass ticket number as id, similar to the TicketQuery. Those are listed as union with the tickets referenced somewhere else in text.

[clemens]

As I understand the proposal would allow to optionally pass one or more ticket number(s) to the macro e.g. [[PageTickets(id=12|34|56)]]. And since the PageTicketsMacro is based on the powerful TicketQuery macro the result would be a union of the manually listed tickets plus those from the PageTickets search result.

The feature can be useful in those cases where the PageTicketsMacro is unable to automatically find the ticket number(s).

[lucid]

Thanks for the suggestion!

For further suggestions, it would be helpful to get suggested code changes in standard patch format. Please refer to ​t:TracDev/SubmittingPatches#Makethepatch for information how to do this. It helps to avoid confusion and unambiguously make clear your suggestion. I'm not 100% sure how exactly you intended to change the code, and had to postpone applying the changes until I had more time to fully analyze, reverse engineer and test everything. So investing a bit of time here will help a quicker turnaround for your suggestions.

[lucid]

In 17330:

PageTicketsMacro: Don't ignore explicit additional id=... arguments (fix #13525)