only use window.jQuery if it supports required jQuery.fn.on

[snapwich]

This is a safe and simple fix to the legacy jQuery situation referenced in #2163 until the API in issue #608 is refined and implemented.

[mary-poppins]

Thanks for the PR! Please check the items below to help us merge this faster. See the contributing docs for more information.

• Uses the issue template (#6376)

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

[mary-poppins]

I'm sorry, but I wasn't able to verify your Contributor License Agreement (CLA) signature. CLA signature is required for any code contributions to AngularJS.

Please sign our CLA and ensure that the CLA signature email address and the email address in this PR's commits match.

If you signed the CLA as a corporation, please let us know the company's name.

Thanks a bunch!

PS: If you signed the CLA in the past then most likely the email addresses don't match. Please sign the CLA again or update the email address in the commit of this PR.
PS2: If you are a Googler, please sign the CLA as well to simplify the CLA verification process.

[snapwich]

There should be a CLA for this commit now

[mary-poppins]

CLA signature verified! Thank you!

Someone from the team will now triage your PR and it will be processed based on the determined priority (doc updates and fixes with tests are prioritized over other changes).

[tomdcc]

+1 Another case here of wanting to use angular in an existing website with an old jquery already present - this seems like the safest option.

[mgallag]

+1 I'm trying to drop an angular app into a large site that uses old jQuery. This change is simple and solves our problem.

[mgol]

We'll likely switch to jQuery 2.x soon in Angular 1.3; see #7311 for progress in that regard.

[mgol]

(though API is compatible so this PR should be OK)

[tomdcc]

It would be nice if this patch could be applied to the 1.2.x branch at least. If there's interest in applying it there I can submit a PR against 1.2.x.

[mgol]

angular -> Angular

[mgol]

@tomdcc No need for a separate PR, this will do just fine for 1.2 as well.

[mgol]

@RSNApp The commit message should start with fix(jqLite):; I'd also shorten the commit title a little, e.g.:

fix(jqLite): use jQuery only if jQuery.fn.on present

Make Angular not load jQuery older than 1.7 which doesn't implement
the `on` method.

[mgol]

@RSNApp Also, if you could rebase to the latest master, that would be great.

(EDIT: corrected the nick)

[tomdcc]

I didn't submit this PR, so I don't think I can rebase it - would have to submit a new one I think, or submit a PR to the @RSNApp branch and have him/her accept it. Let me know if you want a new one.

[mgol]

@tomdcc sorry, I meant to ping @RSNApp. :)

[snapwich]

I've rebased to master and updated the commit message.

[iPirat]

didnt find that one, when I opened #7300
Would be glad to see in the next 1.2.x Version.

Thanks!

[mgol]

Thanks @RSNApp! Landed for both 1.3 & 1.2.