-
Couldn't load subscription status.
- Fork 27.3k
fix(angular.copy): change angular.copy to correcly clone RegExp #3474
Conversation
angular.copy previously copied RegExp as an empty object. Change detects RegExp instance and clones into new RegExp. This change is based on a previous fix to allow Date to be copied. Closes angular#3473
|
Thanks for the PR!
If you need to make changes to your pull request, you can update the commit with Thanks again for your help! |
|
Please note that unit tests reported a failure before (and after) my commit: |
I have just signed CLA. Name is Andrew Hitchman. |
|
@AndyHitchman I'm afraid that we can't find you CLA signature... Could you please verify and provide the exact name / e-mail you've used to sign-in the CLA? Thnx! |
|
Resigned CLA with name Andrew Hitchman and email andy at hitchman dot net |
|
CLA verified |
|
Hi, I tried to run unit test on g3_v1_x branch but it failed due to similar issue. Can you please help?
|
|
What time zone are you in? |
|
Hi, I am in Australia/Melbourne (UTC/GMT +10 hours). Thanks On 26/08/2013, at 1:38 PM, Igor Minar notifications@github.com wrote:
|
angular.copy previously copied RegExp as an empty object. Change detects
RegExp instance and clones into new RegExp. This change is based on a previous
fix to allow Date to be copied.
Closes #3473