Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #785 URL query for node is not working for paths with non-numbers. #788

Merged
merged 1 commit into from
Jan 16, 2016

Conversation

kecso
Copy link
Member

@kecso kecso commented Jan 15, 2016

Now a library is used for uri query string parsing.
The node parameter of the query is not transformed to lower case as it can have both upper and lower case characters now.

now a library is used for uri query string parse
the node parameter of the query is not transformed to lower caps, as it can have both now
@@ -78,6 +78,8 @@ require.config({

moment: 'lib/moment/moment.min',

urlparse:'lib/purl/purl.min',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add space next time.

@lattmann lattmann closed this Jan 16, 2016
@lattmann lattmann reopened this Jan 16, 2016
lattmann added a commit that referenced this pull request Jan 16, 2016
…er_case

Fixes #785 URL query for node is not working for paths with non-numbers.
@lattmann lattmann merged commit 1622594 into master Jan 16, 2016
@lattmann lattmann deleted the issue/785_url_query_node_path_lower_case branch January 16, 2016 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants