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.

Sign up for GitHub

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

Jump to bottom

Fix notices appearing on form list table #1319

Closed
ravinderk opened this issue Dec 8, 2016 · 0 comments
Closed

Fix notices appearing on form list table #1319

ravinderk opened this issue Dec 8, 2016 · 0 comments

Comments

@ravinderk
Copy link
Collaborator

Issue Overview

There are a couple of notice are coming on form table list for a multi type form which has only one level ( all field of this level should be empty, especially amount field ).

Notice: Undefined index: in /wp-content/plugins/Give/includes/forms/functions.php on line 707
Notice: Undefined variable: max_id in /wp-content/plugins/Give/includes/forms/functions.php on line 707
Notice: Undefined index: in /wp-content/plugins/Give/includes/forms/functions.php on line 660
Notice: Undefined variable: min_id in /wp-content/plugins/Give/includes/forms/functions.php on line 660

Give version: latest
ravinderk added a commit to ravinderk/Give that referenced this issue Dec 8, 2016
@ravinderk
Fix impress-org#1319
390dd45
@ravinderk ravinderk mentioned this issue Dec 8, 2016
Closed
3 tasks
ravinderk added a commit to ravinderk/Give that referenced this issue Dec 8, 2016
@ravinderk
Fix impress-org#1319
d0a070e
@ravinderk ravinderk mentioned this issue Dec 8, 2016
Merged
3 tasks
DevinWalker pushed a commit that referenced this issue Dec 8, 2016
Merge pull request #1321 from ravinderk/issues/1319
c516c4d 
Fix #1319
DevinWalker pushed a commit that referenced this issue Dec 8, 2016
Merge branch 'release/1.7.1' into release/1.8
27b3651 
* release/1.7.1:
  Fixed border added by WP 1.7
  Add fs-extra requirement so "gulp watch" works out of the box after "npm install"
  Add missing  param
  Fix #1319

# Conflicts:
#	assets/css/give-admin-rtl.min.css
#	assets/css/give-admin.min.css
#	assets/scss/admin/welcome.scss
#	assets/sourcemaps/give-admin.css.map
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ravinderk