- Making errors more descriptive
- When we cannot not get userID we should delete database
Details
Details
- Throw errors from setting/getting userID and check if errors are printed.
- Throw error from getting and make sure database is deleted
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| web/database/sqlite-data-handler.js | ||
|---|---|---|
| 43–59 ↗ | (On Diff #34514) | It is possible that await databaseModule.init({ clearDatabase: true }); will throw and we catch it, and then await databaseModule.schedule({
type: workerRequestMessageTypes.SET_CURRENT_USER_ID,
userID: currentLoggedInUserID,
});will succeed... won't that result in assigning a new user to the old data? |
| web/database/sqlite-data-handler.js | ||
|---|---|---|
| 43–59 ↗ | (On Diff #34514) | yes, good catch, fixed |