Skip to content

Commit

Permalink
Merge pull request #207 from ottemo/awk_hotfix_order-id-filtering
Browse files Browse the repository at this point in the history
order id filtering bug
  • Loading branch information
James Vastbinder committed Nov 18, 2015
2 parents 19e9a0e + 6dca190 commit fc64554
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
*.sublime-workspace

app/bower_components
app/scripts/main.js
app/bundle.js
app/media
dist
Expand Down
4 changes: 3 additions & 1 deletion app/scripts/design/directives/guiTableManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,9 @@ angular.module("designModule")
*/
saveCurrentActiveFilters = function (filterDetails) {
if (filterDetails.filterValue) {
if (-1 !== ['text', 'string', 'varchar'].indexOf(filterDetails.dataType)) {
//REFACTOR: dataType is actually assigned by a request for attribute information from the server,
// and apparently id is it's own type
if (-1 !== ['text', 'string', 'varchar', 'id'].indexOf(filterDetails.dataType)) {
activeFilters[filterDetails.attribute.toLowerCase()] = filterDetails.filterValue.replace(/~/g, "").split(",");
} else {
activeFilters[filterDetails.attribute.toLowerCase()] = filterDetails.filterValue.replace(/~/g, "");
Expand Down
2 changes: 1 addition & 1 deletion gulpfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ gulp.task('replace', function () {

// Empties folders to start fresh
gulp.task('clean', function () {
return del(['dist']);
return del(['dist','app/scripts/config.js']);
});

gulp.task('jshint', function () {
Expand Down

0 comments on commit fc64554

Please sign in to comment.