drtak34 / my-films

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

Switching users does not sync MUS data to AMC watched field #228

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
Take your AMC4 config - and just create a new user.
Expected: All Films unwatched.
Instead: All films watched (in facade - the real MUS status is correctly 
unwatched)

When you switch back to the previous user it does NOT update the field selected 
for watched status (e.g. Checked field) based on the MUS data for that user. 
Ofc MF displays watched status/info correctly in GUI since it is based on MUS 
data, but it does NOT display checked field correctly in AMC!

What is the expected output? What do you see instead?
Facade watched status should be displayed correctly and properly synced to AMC 
watched field.

Please use labels and text to provide additional information.
MF6.0.0beta

Original issue reported on code.google.com by Guzz...@googlemail.com on 24 Oct 2012 at 9:27

GoogleCodeExporter commented 9 years ago
When crearting a new user, a backup of the DB should be created too.

Original comment by Guzz...@googlemail.com on 24 Oct 2012 at 10:00

GoogleCodeExporter commented 9 years ago
Actually, it DOES sync the data to the watched field - but we didn't do a "save 
to disk" - so unless there was any other operation, that trioggered a save2disk 
operation, the updated data did not reflect in the DB file.

Original comment by Guzz...@googlemail.com on 27 Oct 2012 at 11:32

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

Original comment by Mikeplan...@web.de on 27 Oct 2012 at 11:33

GoogleCodeExporter commented 9 years ago
I can verfiy in build 2663 that adding a new user correctly sets all films to 
unwatched, and displays them correctly in AMC (unchecked) if checked field is 
selected as the watch field.

Original comment by Dade...@gmail.com on 4 Nov 2012 at 10:51

GoogleCodeExporter commented 9 years ago
Oops - new user is NOT saved until an update operation is performed (such as 
watch a movie or set it watched). No MUS data is entered for any movie until an 
MUS field is updated. Is that intended?

Original comment by Dade...@gmail.com on 4 Nov 2012 at 11:04

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
The new username should be saved in config.
It does not reflect in MUS field until there is any change applied for that 
user, that is correct.
Does it cause problems other than the user not being present in MUS field?

Original comment by Guzz...@googlemail.com on 5 Nov 2012 at 8:53

GoogleCodeExporter commented 9 years ago
Where in MyFIlms.xml? AFAIK it only stores the default user?

Yes it causes other issues - if you do not update anything for the new user, 
then switch to another user, the new user no longer displays.

Even when you exit/reenter MF or MP you can no longer select the new user.

Once you have updated any MUS data for that user, then you can switch to 
another user and back.

There may be some relationship to Issue #190: dd option to delete username and 
all it's data (MultiUserStates) as well because you cannot delete a new user 
either.

Original comment by Dade...@gmail.com on 5 Nov 2012 at 11:16

GoogleCodeExporter commented 9 years ago
confirmed - there was a bug in the method creating new MUS data.

Original comment by Guzz...@googlemail.com on 7 Nov 2012 at 2:41

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

Original comment by Mikeplan...@web.de on 7 Nov 2012 at 2:44

GoogleCodeExporter commented 9 years ago
Works like a charm in build 2673 ;)
Only the default user is saved to config - UserProfileName but all works as 
expected and MUS data is generated for all films now for new user and correctly 
set to unwatched.

Original comment by Dade...@gmail.com on 8 Nov 2012 at 7:56