Patch: DueIn is not set in html/Ticket/Create.html

Hi,

here’s a patch, quite important for those using Priorities.
html/Ticket/Create.html did not set the ticket’s Due date
to the queue default.

Also “min” is added to Time Worked and Time Left fields,
and “days” to Due field.

Stan

rt3.0.4.ssinyagin.20030711-1.TicketCreateDue.diff (2.36 KB)

Sorry, the patch is wrong: it sets the number of days, whereas it must
set the date in ISO format.
Will work on that next week.
Stan

See the correct patch, attached.
It sets the Due field to “now + N days”, and it’s
understood correctly by Time::ParseDate.

Regards,
Stan

rt3.0.4.ssinyagin.20030714-1.TicketCreateDue.diff (2.38 KB)

Probably, “Starts” should also be set to “now”.— Stanislav Sinyagin ssinyagin@yahoo.com wrote:

See the correct patch, attached.
It sets the Due field to “now + N days”, and it’s
understood correctly by Time::ParseDate.

See the final patch attached.— Stanislav Sinyagin ssinyagin@yahoo.com wrote:

Probably, “Starts” should also be set to “now”.

rt3.0.4.ssinyagin.20030714-2.TicketCreateDue.diff (2.43 KB)

Stanislav,
thanks for the patch! Could You provide it in unified diff format? It
is so much easier to read…

Regards,
Harald

As requested, here’s the unified diff.— Harald Wagener hwagener@hamburg.fcb.com wrote:

thanks for the patch! Could You provide it in unified diff format? It
is so much easier to read…

rt3.0.4.ssinyagin.20030714-3.TicketCreateDue.diff (1.82 KB)

Thanks for the patch.

We took a variant of the Due setting fix (which uses simple ISO dates) as change #43 ,
but not the Starts fix, as by default, I don’t think that field should be filled in with the
current time.

Could you explain what’s the purpose of Starts field, and what should be the default value?

We didn’t take the patch for “min” as it’s not properly localized. see the RT dev style guide
for how to localize in RT 3.0.5pre2

OK, I’ll provide the localized version soon.

Regards,
Stan

Thanks for the patch.

We took a variant of the Due setting fix (which uses simple ISO dates) as change #43 ,
but not the Starts fix, as by default, I don’t think that field should be filled in with the
current time.

Could you explain what’s the purpose of Starts field, and what should be the default value?

Roughly, project management. Most sites will likely never use it and
setting it to a date before a task starts (rather than leaving it empty) doesn’t really
do anyone any justice.

http://www.bestpractical.com/rt – Trouble Ticketing. Free.