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

refactor(ngSwitch):delete default directive restriction #10987

Closed
wants to merge 1 commit into from

Conversation

diegofss11
Copy link
Contributor

Delete the directive restriction to avoid code duplication since the default is already applied

This line is uneccesary because the default value is set inside the registerDirective function:

directive.index = index;
directive.name = directive.name || name;
directive.require = directive.require || (directive.controller && directive.name);
directive.restrict = directive.restrict || 'EA';

PR baked during http://www.meetup.com/AngularJS-BH/events/220033641/

delete the directive restriction to avoid code duplication since the default is already applied
@pkozlowski-opensource pkozlowski-opensource self-assigned this Feb 20, 2015
pkozlowski-opensource pushed a commit to pkozlowski-opensource/angular.js that referenced this pull request Feb 20, 2015
delete the directive restriction to avoid code duplication since the default is already applied

Closes angular#10987
netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
delete the directive restriction to avoid code duplication
since the default is already applied

Closes angular#10987
Closes angular#11119
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants