Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
This repository was archived by the owner on Apr 12, 2024. It is now read-only.
/angular.jsPublic archive

(fix): merge URI query into fragment in non-html5 $location mode.#15865

Open
ed-at-work wants to merge1 commit intoangular:master
base:master
Choose a base branch
Loading
fromed-at-work:issue-#15856

Conversation

ed-at-work
Copy link

Angular does not currently include the non-fragment query portion
Of URLs in non-html5 $location mode. This commit merges the fragment
Query, if present, with the non-fragment query if it exists.

See#15856

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix.

What is the current behavior? (You can also link to an open issue here)
AngularJS's $location service treats URLs differently depending on whether or not the browser supports the HTML5 history API or requires the use of the hashbang mode. URLs should be parsed into the same object regardless of browser mode. In the current behaviour, query parameters are lost.

What is the new behavior (if this is a feature change)?
In non-HTML5 hashbang mode, the query parameters that appear prior to the fragment identifier (#) are copied into the fragment identifier before theparseAppUrl function is called. Therefore, the query parameters outside of the fragment are included in both html5 and non-html5 modes.

Does this PR introduce a breaking change?
Not really.

Please check if the PR fulfills these requirements

Other information:
None.

Angular does not currently include the non-fragment query portionOf URLs in non-html5 $location mode. This commit merges the fragmentQuery, if present, with the non-fragment query if it exists.Seeangular#15856
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.
Reviewers
No reviews
Assignees
No one assigned
Projects
None yet
Milestone
Ice Box
Development

Successfully merging this pull request may close these issues.

3 participants
@ed-at-work@Narretz@googlebot

[8]ページ先頭

©2009-2025 Movatter.jp