There was a problem with the addition of the year. Now in 2013 and the selection of only up to 2012. More information on the picture.
foto
How to add 2013.
Such a problem in Projects, Tasks, Tickets, Timetrackings, Invoices, Expenses.

CommentFileSizeAuthor
#6 storm-yearend-d7.patch1.2 KBjuliangb
#1 Storm-Years-Capture.PNG24.38 KBjuliangb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Version: 6.x-2.1 » 7.x-1.x-dev
Assigned: BrdNLSrg » Unassigned
FileSize
Storm-Years-Capture.PNG24.38 KB

You can fix this in the admin settings - see the attached screenshot.

However, for new sites, I agree, we should change the default.

Setting this to be changed in D7 version first, but we should also change this in D6.

BrdNLSrg’s picture

BrdNLSrg CreditAttribution: BrdNLSrg commented

Thanks. Sorry I stupid

davad’s picture

davad CreditAttribution: davad commented
Title: bug with a year » Change default year to 2013

Changing the title to be more descriptive.

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Title: Change default year to 2013 » Change default end year to 2013

Ultra-descriptive!

davad’s picture

davad CreditAttribution: davad commented

Yup, Julian that's ultra-descriptive alright :)

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Status: Active » Needs review
FileSize
storm-yearend-d7.patch1.2 KB

Here's a patch for D7.

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Version: 7.x-1.x-dev » 6.x-2.x-dev

Committed to D7.

Let's try testing the same patch on D6. I think the patch will work, but the tests may be failing at the moment.

juliangb’s picture

juliangb CreditAttribution: juliangb commented

#6: storm-yearend-d7.patch queued for re-testing.

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented
Status: Needs work » Needs review

What about calculating and using the current year, so this doesn't have to be fixed every single year?

Status: Needs review » Needs work

The last submitted patch, storm-yearend-d7.patch, failed testing.

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Status: Needs review » Needs work

@Xano, yes, that'd be a much better method.

I wanted to fix this quickly so that we could move back to getting the D7 port done.

Happy to commit a patch to do this in a more future-friendly manner!

juliangb’s picture

juliangb CreditAttribution: juliangb commented
Status: Needs work » Fixed

Committed to 2.x-2.x

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Add a new year