Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changing tab name has led to data loss of the selection within the media picker #160

Closed
Jeavon opened this issue Nov 16, 2017 · 8 comments
Closed
Assignees
Labels

Comments

@Jeavon
Copy link

Jeavon commented Nov 16, 2017

Whilst adding a load of new properties a existing tab was renamed, this causes the property to be deleted and re-created which of course leads to loss of the existing data.

In the attached screen shot the tab named "Info" was renamed "BM View" the logo property didn't change.

image

@KevinJump
Copy link
Owner

:(
In theory it shouldn't actually delete the property as they are matched by an internal key value (but i will check that!) so the tab doesn't matter.

The change might well say delete because it is a comparison of XML files it doesn't match the internal process completely :(

@Jeavon
Copy link
Author

Jeavon commented Nov 16, 2017

The result was slightly odd because the umbraco.config and IPublishedContent contained the values but the Content Service and therefore the Umbraco UI didn't have any values so for some reason it does seem that internally the property was dropped and recreated...

@KevinJump
Copy link
Owner

yeah i need to do the tests - it shouldn't happen - but that doesn't mean it didn't ;)

are you all latest umbraco / usync ? (just for me to recreate it)

@Jeavon
Copy link
Author

Jeavon commented Nov 16, 2017

Yup we are

  <package id="uSync" version="4.0.1" targetFramework="net462" />
  <package id="uSync.Core" version="6.0.1" targetFramework="net462" />
  <package id="UmbracoCms" version="7.7.5" targetFramework="net462" />

Thanks Kevin!

@KevinJump KevinJump self-assigned this Nov 16, 2017
@KevinJump
Copy link
Owner

Right i am close :)

I think there has been a change in behaviour in the core - but it's my bad for relying on it

the lookup in theory is fine, but before we get there we create a new tab and delete the old one - I am failry sure that in older umbraco's this just put all of the values in the tab into 'generic' and then the next bit of the process which updates the properties would put them back in.

looks like deleting a tab now deletes the properties in the tab :(

bit of a logic re-write coming up (not massive though).

@Jeavon
Copy link
Author

Jeavon commented Nov 16, 2017

Phew, very glad you've tracked it down!

KevinJump added a commit that referenced this issue Nov 16, 2017
Some change means the remving the tab also removes the properties - so
no we don't remove the tabs until we have gotten to the end of the
process.
@KevinJump KevinJump added this to the v4.0.1 Release (Tab Rename Fix) milestone Nov 16, 2017
@KevinJump KevinJump added the bug label Nov 16, 2017
@KevinJump
Copy link
Owner

Hi
fixed - and released uSync 4.0.2 (Core 6.0.2) - just waiting for nuget to index it

@Jeavon
Copy link
Author

Jeavon commented Nov 16, 2017

Completely awesome Kevin!

Many thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants