search

iMediaSandboxing / iMedia

The Karelia iMedia Browser, framework and application for browsing media on Mac OS X
http://www.karelia.com/imedia/
Other
18 stars 12 forks source link

Fix top level node loading #71

Closed danielpunkass closed 8 years ago

danielpunkass commented 9 years ago

Ensure that a top level node that happens to also not have a parent will still be reloaded. This seems contrived but I run into this in MarsEdit, where I use a customized IMBLIbraryController that possesses only one node with images, for some specialized media views.

jjac commented 9 years ago

@danielpunkass It looks like your patch would take away our guard against race conditions for top-level nodes. On the other hand you stated a valid use case where nodes without a parent node want to be populated / get reloaded.

After discussing the issue with @peterb180369 we came to the conclusion that our way of identifying an "outdated" node is currently not to the point.

@peterb180369 is now investigating a solution with "versioned" nodes that would also handle your use case gracefully.

danielpunkass commented 9 years ago

Thanks, @jjac and @peterb180369 for considering a change to accommodate my unusual use!

jjac commented 8 years ago

This issue is fixed with commits 264b676d and ab52d58f (as suggested in my comment on July 6th)