search

ngoctuan / my-films

Automatically exported from code.google.com/p/my-films
0 stars 0 forks source link

Dates not recognized by AMC3 or 4 #156

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Import films (AMCU)
2. Date format is now stored as Date="2012/08/10"

What is the expected output? What do you see instead?
This date format is not recognized by AMC4 - when you open your AMC4 catalog in 
AMC, the Date Added field is not checked. If you save your catalog in AMC4 
dates are lost.

Please use labels and text to provide additional information.
This was working correctly previously, not sure which build changed it - but in 
old backup files AMCU used a different data format: Date="8/10/2012"

Original issue reported on code.google.com by Dade...@gmail.com on 10 Aug 2012 at 9:18

GoogleCodeExporter commented 9 years ago 
There was a change done in rev #2216 on June 11th.
Afaik, we discussed that before with the result it was a problem in AMC - and 
you told me, that soulsnake was working on it!?
If that is not fixable in AMC, we have another problem - as this was done to 
fix the recently added view and sort by date. As we do that now with string 
comparer, we need such a format to get proper sorting. We removed the 
"dateadded" calculated field long ago. Readding this will be a lot of work - as 
existing date field handling has to be changed in many places and methods - and 
new method has to be integrated.
So you want me to revert that change and just see, what happens? iiec, there 
were problems in several date related views/areas, but I can't remember exactly 
...
pladv

Original comment by Guzz...@googlemail.com on 10 Aug 2012 at 11:21

GoogleCodeExporter commented 9 years ago 
Different issue - IIRC the fix in 2216 was related to the issue I reported 
where watched status was set for incorrect movie when several movies were 
Imported on the same day (same date added) 

Soulsnake did fix the previous issues with date formats - AMC4 now correct 
supports either your system date format or the international standard 
yyyy-mm-dd.  However "2012/08/10" is not supported in either AMC3 or AMC4.  Can 
you just change the slash '/' to a dash '-' ? e.g. 2012-08-10

Original comment by Dade...@gmail.com on 12 Aug 2012 at 3:49

GoogleCodeExporter commented 9 years ago 
Yep, will do so we can test - if that works, it would hopefully be fine for all 
requirements.

Original comment by Guzz...@googlemail.com on 12 Aug 2012 at 11:54

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r2373.

Original comment by Mikeplan...@web.de on 12 Aug 2012 at 12:10

GoogleCodeExporter commented 9 years ago 
Date format works correctly for  AMC4 but unfortunately yyyy-mm-dd format is 
NOT supported for AMC3 :(

As far as I can see, all works correctly in My Films for AMC3 catalog types - 
you can sort by date correctly, dates display correctly, view by recently 
added, age added or any date field works correctly. The ONLY issue I can see is 
IF you save your catalog in AMC3 then all dates are lost and none of the above 
work correctly any longer.

Original comment by Dade...@gmail.com on 17 Aug 2012 at 5:05

GoogleCodeExporter commented 9 years ago 
I should have mentioned the BIG issue is that existing AMC3 catalogs will not 
work correctly in MF 6 with the new date format because they still display 
dates in local date setting format, e.g. "8/17/2012" in my system with US 
language settings.

Original comment by Dade...@gmail.com on 17 Aug 2012 at 5:21

GoogleCodeExporter commented 9 years ago 
Ok, I have now applied some changes:
Revert AMCU date handling to use local culture settings (that should match 
AMC3/4 handling, if I understand you correctly.
Replace film list sorting to use "DateAdded" instead of "Date" - also speeds up 
film lists sorted by date significantly.
Rewrote the handling of DateAdded to set Value also for invalid or missing 
"Date" field content (so sorting works properly).

Original comment by Guzz...@googlemail.com on 18 Aug 2012 at 3:34

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r2396.

Original comment by Mikeplan...@web.de on 18 Aug 2012 at 3:45

GoogleCodeExporter commented 9 years ago

Original comment by Dade...@gmail.com on 16 Sep 2012 at 6:27