Skip to content

Conversation

@sanex3339
Copy link
Contributor

Added ability to remove array value.

Example:
url > ?test[]=test1&test[]=test2&test[]=test3
$.query.REMOVE('test, 'test2');

Result:
url > ?test[]=test1&test[]=test3

Added ability to remove array value.

Example:
url > ?test[]=test1&test[]=test2&test[]=test3
 $.query.REMOVE('test, 'test2');

Result:
url > ?test[]=test1&test[]=test3
@alrusdi
Copy link
Owner

alrusdi commented Feb 18, 2015

For consistency please implement this also in "remove" function (lowercase version which is works with copy of object)
Also add an example of usage to README.md

update 'remove' function
Fixed: removing incorrect array value caused by negative array index.
Added: ability to remove any parameter by value.
Update readme with enhanced remove functionality
Small correct to readme
@sanex3339
Copy link
Contributor Author

Done. I tested updated functions with:

  • normal second argument value;
  • empty second argument value;
  • bool (true) second argument value;
  • without second argument.

alrusdi added a commit that referenced this pull request Feb 20, 2015
Added ability to remove array value
@alrusdi alrusdi merged commit 449fb55 into alrusdi:master Feb 20, 2015
@alrusdi
Copy link
Owner

alrusdi commented Feb 20, 2015

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants