Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

angular.copy does not correctly duplicate RegExp #3473

Closed
AndyHitchman opened this issue Aug 6, 2013 · 0 comments
Closed

angular.copy does not correctly duplicate RegExp #3473

AndyHitchman opened this issue Aug 6, 2013 · 0 comments

Comments

@AndyHitchman
Copy link
Contributor

Similarly to issue #49 (copying Date objects), angular.copy does not correctly duplicate RegExp objects. The result is an empty object '{}'.
AndyHitchman added a commit to AndyHitchman/angular.js that referenced this issue Aug 6, 2013
@AndyHitchman
fix(angular.copy): change angular.copy to correcly clone RegExp
89fbd5d 
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
@AndyHitchman AndyHitchman mentioned this issue Aug 6, 2013
Closed
IgorMinar pushed a commit that referenced this issue Aug 9, 2013
@AndyHitchman @IgorMinar
fix(angular.copy): change angular.copy to correcly clone RegExp
5cca077 
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
Closes #3474
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(angular.copy): change angular.copy to correcly clone RegExp AndyHitchman/angular.js
1 participant
@AndyHitchman