Skip to content

Commit

Permalink
fix(review): implement review corrections
Browse files Browse the repository at this point in the history
This commit implements feedback from @sfount on the PR.  The server was
also scanned for errant `uuid()` calls instead of `uuid.v4()`.
  • Loading branch information
jniles committed Jun 13, 2016
1 parent 436fe44 commit c98b7e7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion client/src/js/services/system.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ function SystemService($http, util) {
}

function handleErrorEvent(event) {
if (event.readyState == EventSource.CLOSED) {
if (event.readyState === EventSource.CLOSED) {
console.log('Connection was closed.');
}
}
Expand Down
2 changes: 1 addition & 1 deletion server/controllers/admin/suppliers.js
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ function create(req, res, next) {
const data = db.convert(req.body, ['creditor_uuid']);

// provide uuid if the client has not specified
const uid = data.uuid || uuid();
const uid = data.uuid || uuid.v4();
data.uuid = db.bid(uid);

let sql =
Expand Down

0 comments on commit c98b7e7

Please sign in to comment.