[patch] Milestones don't show up in calendar if their due time is not 00:00:00 (h:m:s)

[delsauce@…]

Not sure why the due time of milestones in our install is not 00:00:00, but they are. When this is the case, no milestones will show up in the calendar because the SELECT statement in the macro that searches for the milestones is looking for an exact time match.

I modified the query so that the milestone time isn't taken into consideration (the entire day will match). Here is the line of code that was changed:

 cursor.execute("""
                        SELECT name
                          FROM milestone
                         WHERE due >= %s and due < %s
                    """, (duedatestamp,duedatestamp_eod, ))

[Ryan J Ollos]

Based on the number of tickets opened, this certainly seems to be a popular macro!

[Steffen Hoffmann]

Glad you found it. I was already suspecting a bug in the milestone part, since my test milestones did never show up as well.

I confirm that this flaw exist not only in the current version from 0.11 branch but up from the first publicly released version up to the latest development version in trunk. In short: this never work, despite all data necessary (duedatestamp, duedatestamp_eod) was prepared and in place.

I've build a fixed version of trunk with the suggested code improvement and it's working well. I'll merge the change to all branches now. Expect a new maintenance release tonight.

[Steffen Hoffmann]

Let's aim at fixing this within one hour after ticket creation to account for the high popularity. :-)

[Steffen Hoffmann]

(In [8108]) WikiTicketCalendarMacro: Improve milestone query to actually make it work right, closes #7225.

This is a long standing flaw persisting from the earliest version up to current trunk. Thanks to delsauce@… for report and suggested code change.

[delsauce@…]

No worries-- glad to help. Thanks for resolving so quickly.