Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix remove unique strings has capital letters. #69

Closed
wants to merge 1 commit into from

2 participants

@midworld

There is one more point that delete unique property.

@maritz maritz referenced this pull request from a commit
@maritz Add failing test for #69 d67e923
@maritz
Owner

Thanks! Applied, pushed and published.

@maritz maritz closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Aug 21, 2012
  1. @midworld
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 1 deletion.
  1. +4 −1 lib/store.js
View
5 lib/store.js
@@ -305,8 +305,11 @@ var __realDelete = function __realDelete(silent, callback) {
for (p in this.properties) {
if (this.properties.hasOwnProperty(p)) {
if (this.properties[p].unique) {
+ var propLower = this.properties[p].type === 'string' ?
+ this.properties[p].__oldValue.toLowerCase() :
+ this.properties[p].__oldValue;
multi.del(Nohm.prefix.unique + this.modelName + ':' + p + ':' +
- this.properties[p].__oldValue);
+ propLower);
}
if (self.properties[p].index) {
multi.srem(Nohm.prefix.index + self.modelName + ':' + p + ':' +
Something went wrong with that request. Please try again.