When checking if a tag is present, check on the split array of tags, …

…not the raw string.
performant-software · Sep 23, 2014 · 5a5de87 · 5a5de87
1 parent b2bf185
commit 5a5de87
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 1 deletion.
diff --git a/tests/jasmine/tests/neatline/record/HasTag.spec.js b/tests/jasmine/tests/neatline/record/HasTag.spec.js
@@ -53,4 +53,17 @@ describe('Record | Has Tag', function() {
   });
 
 
+  it('substring tags', function() {
+
+    var record = new Neatline.Shared.Record.Model({
+      tags: 'tag1, tag2'
+    });
+
+    expect(record.hasTag('tag1')).toBeTruthy();
+    expect(record.hasTag('tag2')).toBeTruthy();
+    expect(record.hasTag('tag')).toBeFalsy();
+
+  });
+
+
 });
diff --git a/...avascripts/dist/production/ckeditor/plugins/magicline/images/hidpi/icon-rtl.png b/...avascripts/dist/production/ckeditor/plugins/magicline/images/hidpi/icon-rtl.png
diff --git a/...ared/javascripts/dist/production/ckeditor/plugins/magicline/images/icon-rtl.png b/...ared/javascripts/dist/production/ckeditor/plugins/magicline/images/icon-rtl.png
diff --git a/views/shared/javascripts/src/shared/record/record.model.js b/views/shared/javascripts/src/shared/record/record.model.js
@@ -122,7 +122,7 @@ Neatline.module('Shared.Record', function(Record) {
      * @return {Boolean}
      */
     hasTag: function(tag) {
-      return _.contains(this.get('tags'), tag);
+      return _.contains(this.splitTags(), tag);
     },