braveulysses / yelp-without-pity Goto Github PK

Currently, YWP only works if the user is logged in to Yelp. Non-logged in user should be supported.

Depending on how this is implemented, it might be desirable to track settings on a per-login account basis, with non-logged in users counting as a single account. One way to do this would be to add a login column to the settings table, checking if the user is logged in when the script starts, then saving the account name (or null) every time a row is added.

The Safari Web Inspector lists two identical settings databases for YWP. Is this a Safari bug or a problem with the extension?

Expected:
Only one copy of yelpWithoutPityDB should be shown by Web Inspector.

Actual:
Two are listed.

Notes:
Preferences -> Security -> Show Databases only lists one db, making it likely that this is a Web Inspector bug. If so, a bug needs to be filed at webkit.org.

Steps:

1. Go to yelp.com.
2. Open Web Inspector.
3. Look in Storage -> Databases. Two copies of yelpWithoutPityDB will be present.