$location.search(search, [paramValue]) documentation now reflects that paramValue can be a Number

[johnnyreilly]

I've amended the documentation to reflect that paramValue can also accept a Number as the value to set in the same manner it currently accepts a string. I'm aware of it being used like this at present (without issue that I'm aware of) and so this PR just makes it a recognised usage of the API.

[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.

[johnnyreilly]

Hi @mary-poppins (have you become self-aware yet? 😄),

I've just signed the CLA (electronically) . I hope all is in order now? Let me know if not...

"Let's go fly a kite..."

[pkozlowski-opensource]

If we want to make non-string arguments "official" part of the public API then I guess we should merge #7161 as well that enables numbers for the 'path' part

[johnnyreilly]

Sounds sensible @pkozlowski-opensource - originally this came up as I was updating the Definitely Typed angular typings and noticed that Number wasn't officially part of the API.

Accordingly I've not included Number in the DT typings for now as we seek to be official-API-compliant. But I can't think of a reason this shouldn't be part of the official API (hence the PR). If this is accepted I'll plan to update the DT typings again to include a search paramValue Number overload.

I suppose the difference between this and #7161 is that this is only a doc change - the code isn't touched.

[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).

[johnnyreilly]

Thanks @mary-poppins!

[jeffbcross]

Landed 48e66cf
Thanks!