Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New AngularJS directive for AJAX forms #6399

Merged
merged 3 commits into from Oct 7, 2014
Merged

New AngularJS directive for AJAX forms #6399

merged 3 commits into from Oct 7, 2014

Conversation

diosmosis
Copy link
Member

As title, an angular directive that automates logic for AJAX forms.

Contains two changes to core/Config.php:

  • First is bug fix caused by typo. See commit 773160a
  • Second is modification of old default behavior. The bug caused new default behavior which is now relevant and to be kept. See commit 79496d4 . This change will return an empty array if a config section doesn't exist.

Rationale for second change: Plugins may require configuration options to be set in INI config. They will not be able to modify global.ini.php so when they first access a non-existant config section, an error will throw. This effectively bars plugins from using INI config.

diosmosis added 3 commits October 7, 2014 12:36
…fig sections to contain old superuser bridge info on first access).
…y array() when config section doesn't exist in INI config).
diosmosis pushed a commit that referenced this pull request Oct 7, 2014
Added angular directive that automates logic for AJAX forms.
@diosmosis diosmosis merged commit ef2e342 into master Oct 7, 2014
@diosmosis diosmosis deleted the angular_forms branch October 7, 2014 20:07
@mattab mattab changed the title angular directive for AJAX forms New AngularJS directive for AJAX forms Oct 14, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants