search

gminaker / databases

0 stars 0 forks source link

Process Delivery Testing #5

Closed gminaker closed 9 years ago

gminaker commented 9 years ago

Test and debug functionality.

@thpierce can you be more specific about the errors you were getting? It's working OK on my end..

thpierce commented 9 years ago

Specific error:

Notice: Undefined index: date in C:\Users\Thomas\Documents\School\Year5-_Bio_Major_CS_Minor\T1\CPSC_304\Phase_3\views\manager\process_delivery.php on line 29

One for each element in list.

Will work on it!

gminaker commented 9 years ago

Issue resolved. fix is in master.

thpierce commented 9 years ago

Can you double check that fix/push. I'm not getting fix. I don't see update in master.

thpierce commented 9 years ago

Value checking

Also, should check that adding purchases updates this

anniechen92 commented 9 years ago

Input: 2014/07/12 for 3814 2014-11-21 powerpuffgirls 2014-11-25 Expected: Something about not being able to process (as delivered date is before purchase date) Returned: Notice: Delivery dates were submitted successfully!

mikegfink commented 9 years ago

Why would there be an error in adding a delivery date if we messed up setting the expected date?

The expected date should never be set to earlier than purchase date when it’s being set (if it is then that’s a bug in the expected date code). Note that we can manually add bad entries through sql statements.

But either way we should be able to set the delivery date to any date today or earlier (presuming a real life situation). Or just today if that’s what we want to implement.

gminaker commented 9 years ago

No further action is being taken on this issue.