Lightroom SDK 5: LrCatalog:with...Do methods - add "error" return status.

  • 2
  • Idea
  • Updated 6 years ago
Fixing this bug should be top priority for the LrCatalog:with...Do methods:

http://feedback.photoshop.com/photosh...

Assuming this bug is fixed, I'd like to propose an Idea:

1. Make timeout param table mandatory in Lr5 environment, and throw deprecation error (as governed by config.lua) if timeout param table is not passed in Lr4 environment.
2. If function error, return "error" status.

So instead of (in the documentation):

Return value
(string) If 'func' does not throw an error, the returned string will be either "executed", "queued", or "aborted".

it would be:

Return value
(string) If 'func' does not throw an error, the returned string will be either "executed", "queued", or "aborted", and if 'func' *does* throw an error, the returned string will be "error", and the other return value will be an appropriate error message.


Reminder to all plugin authors: without catalog contention logic in your catalog access code, your (catalog updating) plugins will seem flaky (have frequent or occasional errors) to all users who've got other plugins updating the catalog in the background.
 

Photo of Rob Cole

Rob Cole

  • 4831 Posts
  • 382 Reply Likes

Posted 6 years ago

  • 2

Be the first to post a reply!