When an invoice is automatically created from a timetracking, the request date and due date are both 01-Jan-70.
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | storm-1.33-patched.tar_.gz | 218.61 KB | francewhoa |
| #8 | 8-edit-inv-1d.png | 27.83 KB | francewhoa |
| #4 | remove_787822.patch | 3.15 KB | tchurch |
Comments
Comment #1
tchurch commentedI think this was caused because of #787822: Implement support for Date popup module on Ticket and Invoice Date Fields being in version 1.33.
This should automatically be fixed when the above issue is fixed.
If #794638: Replace 0 with NULL for all date related fields. is also committed (once I've done it) it should clear up the issues with the dates.
I've manually patched my clients website to fix this on 1.33.
I can post the patch here too if anyone else wants to use it. Note - the patch is only for 1.33 as it looks like the -dev version doesn't contain this error.
Comment #2
juliangb commentedI tested this by invoicing just a timetracking. Surely this has nothing to do with tickets?
Does your patch simply remove the ticket date code as per -dev?
Comment #3
tchurch commentedThe code for #787822: Implement support for Date popup module on Ticket and Invoice Date Fields was added for processing of ticket AND invoice dates to allow using date_popup.
The temporary patch I have (for 1.33) would be for invoice dates (as these could be blank).
My client doesn't use tickets and found the problem in invoicing timetracking records.
Comment #4
tchurch commentedI've attached a patch to remove the invoice code for 1.33 if anyone wants to use it.
Comment #5
tchurch commented( I expect this patch to fail on -dev because of #684016: Fix E_NOTICE notifications but it does work.)
Comment #6
tchurch commentedComment #8
francewhoaI can confirm this issue with Storm 1.33. Mine are dated
1969-12-31I would be happy to contribute patch testing.
Detail steps to reproduce this issue. Scroll down the handbook under
Step 6/6 Add one Invoicesection. Hope this helps: http://drupal.org/node/854056Note to self: Find attached screenshot to clarify issue location.
Comment #9
tchurch commentedWell, the patch failed because the problem does NOT exist on the dev version. It only exists on 1.33.
I suggest anyone having the problem should manually apply the patch or upgrade to the -dev version.
Comment #10
francewhoaThanks tchurch. Are you referring to the patch in comment #4? If not could you post the patch here too? I would like to use it for 1.33.
Comment #11
francewhoaConfirming patch in #4 works with Storm 1.33. Thanks tchurch :)
@all: If you're not able to apply patch find attached the already patched Storm 1.33 module. Not for production/live site.
Comment #12
francewhoaPatch in #4 fix issue with dates. But creates an issue with Invoice View. Multiple copies of same invoice are display in Invoice View. Opening a new issue #857154: Multiple copies of same invoice are display in Invoice View