hipsterjazzbo / LaraParse

LaraParse provides a nice integration for using Parse (parse.com) with Laravel 5+
MIT License
27 stars 19 forks source link

The app will crash due to an uncaught ObjectNotFound exception in `ParseUserProvider->retrieveById()` #34

Closed triztian closed 9 years ago

triztian commented 9 years ago

Returns null when the retrieveById() function searches for an user that does not exist and the session has not been closed. It only handles the Object not found exception.

Refer to this code segment on the parse sdk to see where the exception is thrown:

nicklee1990 commented 9 years ago

what are your through RE my comment regarding referential integrity (housekeeping) and allowing users of the repo to catch this exception themselves? Anyone using Parse should have something in their handler file for Parse Exceptions, as this aligns with Parse's documentation on Exceptions. I believe your code belongs there in your Laravel Handler.php class.

My concern is that this is a valid exception, and should be handled by app specific logic. By implementing this pull request, we're taking away the ability for users to 'cleanup' their data. Nosql DBs like Mongo (and therefore Parse) rely on applications to manage data integrity including referential integrity