-
Notifications
You must be signed in to change notification settings - Fork 192
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
fix(admin-form): 2.2 ensure name title prefixes can be removed once saved #3466
Comments
@mehul0810 I got PHP notice when save empty value into global setting. Please take a look.
|
@mehul0810 I am seeing double quotes in attribute, please check: |
FindingGetting PHP notices in Donation Form Page too |
Debug Log
[16-Jul-2018 04:33:47 UTC] PHP Warning: array_combine() [function.array-combine]: Both parameters should have at least 1 element in /app/public/wp-content/plugins/Give/includes/admin/class-admin-settings.php on line 813
[16-Jul-2018 04:33:47 UTC] PHP Stack trace:
[16-Jul-2018 04:33:47 UTC] PHP 1. {main}() /app/public/wp-admin/edit.php:0
[16-Jul-2018 04:33:47 UTC] PHP 2. require_once() /app/public/wp-admin/edit.php:10
[16-Jul-2018 04:33:47 UTC] PHP 3. do_action() /app/public/wp-admin/admin.php:224
[16-Jul-2018 04:33:47 UTC] PHP 4. WP_Hook->do_action() /app/public/wp-includes/plugin.php:453
[16-Jul-2018 04:33:47 UTC] PHP 5. WP_Hook->apply_filters() /app/public/wp-includes/class-wp-hook.php:310
[16-Jul-2018 04:33:47 UTC] PHP 6. call_user_func_array:{/app/public/wp-includes/class-wp-hook.php:286}() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 7. Give_Admin_Settings->output() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 8. include() /app/public/wp-content/plugins/Give/includes/admin/class-admin-settings.php:263
[16-Jul-2018 04:33:47 UTC] PHP 9. do_action() /app/public/wp-content/plugins/Give/includes/admin/views/html-admin-settings.php:109
[16-Jul-2018 04:33:47 UTC] PHP 10. WP_Hook->do_action() /app/public/wp-includes/plugin.php:453
[16-Jul-2018 04:33:47 UTC] PHP 11. WP_Hook->apply_filters() /app/public/wp-includes/class-wp-hook.php:310
[16-Jul-2018 04:33:47 UTC] PHP 12. call_user_func_array:{/app/public/wp-includes/class-wp-hook.php:286}() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 13. Give_Settings_Page->output() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 14. Give_Admin_Settings::output_fields() /app/public/wp-content/plugins/Give/includes/admin/abstract-admin-settings-page.php:230
[16-Jul-2018 04:33:47 UTC] PHP 15. array_combine() /app/public/wp-content/plugins/Give/includes/admin/class-admin-settings.php:813
[16-Jul-2018 04:33:47 UTC] PHP Warning: array_merge() [function.array-merge]: Argument #2 is not an array in /app/public/wp-content/plugins/Give/includes/admin/class-admin-settings.php on line 813
[16-Jul-2018 04:33:47 UTC] PHP Stack trace:
[16-Jul-2018 04:33:47 UTC] PHP 1. {main}() /app/public/wp-admin/edit.php:0
[16-Jul-2018 04:33:47 UTC] PHP 2. require_once() /app/public/wp-admin/edit.php:10
[16-Jul-2018 04:33:47 UTC] PHP 3. do_action() /app/public/wp-admin/admin.php:224
[16-Jul-2018 04:33:47 UTC] PHP 4. WP_Hook->do_action() /app/public/wp-includes/plugin.php:453
[16-Jul-2018 04:33:47 UTC] PHP 5. WP_Hook->apply_filters() /app/public/wp-includes/class-wp-hook.php:310
[16-Jul-2018 04:33:47 UTC] PHP 6. call_user_func_array:{/app/public/wp-includes/class-wp-hook.php:286}() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 7. Give_Admin_Settings->output() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 8. include() /app/public/wp-content/plugins/Give/includes/admin/class-admin-settings.php:263
[16-Jul-2018 04:33:47 UTC] PHP 9. do_action() /app/public/wp-content/plugins/Give/includes/admin/views/html-admin-settings.php:109
[16-Jul-2018 04:33:47 UTC] PHP 10. WP_Hook->do_action() /app/public/wp-includes/plugin.php:453
[16-Jul-2018 04:33:47 UTC] PHP 11. WP_Hook->apply_filters() /app/public/wp-includes/class-wp-hook.php:310
[16-Jul-2018 04:33:47 UTC] PHP 12. call_user_func_array:{/app/public/wp-includes/class-wp-hook.php:286}() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 13. Give_Settings_Page->output() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 14. Give_Admin_Settings::output_fields() /app/public/wp-content/plugins/Give/includes/admin/abstract-admin-settings-page.php:230
[16-Jul-2018 04:33:47 UTC] PHP 15. array_merge() /app/public/wp-content/plugins/Give/includes/admin/class-admin-settings.php:813
[16-Jul-2018 04:33:47 UTC] PHP Warning: Invalid argument supplied for foreach() in /app/public/wp-content/plugins/Give/includes/admin/class-admin-settings.php on line 828
[16-Jul-2018 04:33:47 UTC] PHP Stack trace:
[16-Jul-2018 04:33:47 UTC] PHP 1. {main}() /app/public/wp-admin/edit.php:0
[16-Jul-2018 04:33:47 UTC] PHP 2. require_once() /app/public/wp-admin/edit.php:10
[16-Jul-2018 04:33:47 UTC] PHP 3. do_action() /app/public/wp-admin/admin.php:224
[16-Jul-2018 04:33:47 UTC] PHP 4. WP_Hook->do_action() /app/public/wp-includes/plugin.php:453
[16-Jul-2018 04:33:47 UTC] PHP 5. WP_Hook->apply_filters() /app/public/wp-includes/class-wp-hook.php:310
[16-Jul-2018 04:33:47 UTC] PHP 6. call_user_func_array:{/app/public/wp-includes/class-wp-hook.php:286}() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 7. Give_Admin_Settings->output() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 8. include() /app/public/wp-content/plugins/Give/includes/admin/class-admin-settings.php:263
[16-Jul-2018 04:33:47 UTC] PHP 9. do_action() /app/public/wp-content/plugins/Give/includes/admin/views/html-admin-settings.php:109
[16-Jul-2018 04:33:47 UTC] PHP 10. WP_Hook->do_action() /app/public/wp-includes/plugin.php:453
[16-Jul-2018 04:33:47 UTC] PHP 11. WP_Hook->apply_filters() /app/public/wp-includes/class-wp-hook.php:310
[16-Jul-2018 04:33:47 UTC] PHP 12. call_user_func_array:{/app/public/wp-includes/class-wp-hook.php:286}() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 13. Give_Settings_Page->output() /app/public/wp-includes/class-wp-hook.php:286
[16-Jul-2018 04:33:47 UTC] PHP 14. Give_Admin_Settings::output_fields() /app/public/wp-content/plugins/Give/includes/admin/abstract-admin-settings-page.php:230
|
Slack Call Summary Participants: @mehul0810 @raftaar1191 |
fix(admin-form): 2.2 ensure name title prefixes can be removed once saved #3466
@Mehul I am not able to save custom title prefix from form metabox |
debug.log
|
Finding@mehul0810 still getting the error but this time its diff. This issue exists in PHP version 5.3 Debug log
|
Revert "fix(admin-form): 2.2 ensure name title prefixes can be removed once saved #3466"
fix(admin-form): 2.2 ensure name title prefixes can be removed once saved #3466
* release/2.2: fix(admin-tools): return array data type fix(form): stop comment moderation emails fix(admin-tools): fix PHP notices fix(admin-donor): added first option of title prefix to disabled fix(admin-setting): fix error with admin setting title prefix fix(admin-form): improve code using sprintf and double quotes fix(admin-setting): fix admin setting title prefix notice fix(admin-setting): remove unnecessary double quotes fix(admin-form): fix empty title prefixes saving issue Revert "fix(admin-form): 2.2 ensure name title prefixes can be removed once saved #3466" fix(admin-form): fix php notice related when value is not array
User Story
As a Site Admin, I want to remove the name title prefixes because I don't like them anymore.
Current Behavior
I currently try to remove them with the "x" provided and when I save the form they don't go away in either the backend or the front-end
Expected Behavior
I expect to use the "x" to remove them.
Steps to Reproduce
Visuals
Screencast of the issue
https://screencast-o-matic.com/watch/cFilY8FAdY
Related
Found during testing 2.2
Environment
WordPress System Info
### WordPress Environment ###Home URL: http://betagive.local
Site URL: http://betagive.local
WP Version: 4.9.7
WP Multisite: –
WP Memory Limit: 256 MB
WP Debug Mode: ✔
WP Cron: ✔
Language: en_US
Permalink Structure: /%postname%/
Show on Front: posts
Table Prefix Length: wp_
Table Prefix Length: 3
Table Prefix Status: Acceptable
Admin AJAX: Accessible
Registered Post Statuses: publish, future, draft, pending, private, trash, auto-draft, inherit, request-pending, request-confirmed, request-failed, request-completed, refunded, failed, revoked, cancelled, abandoned, processing, preapproval, give_subscription
Server Environment
Hosting Provider: DBH: localhost, SRV: betagive.local
TLS Connection: Connection uses TLS 1.2
TLS Connection: Probably Okay
Server Info: Apache/2.4.10 (Debian)
PHP Version: 7.0.3
PHP Post Max Size: 1,000 MB
PHP Time Limit: 900
PHP Max Input Vars: 3000
PHP Max Upload Size: 1,000 MB
cURL Version: ❌ 7.38.0, OpenSSL/1.0.1t - We recommend a minimum cURL version of 7.40.
SUHOSIN Installed: –
MySQL Version: 5.6.34
Default Timezone is UTC: ✔
fsockopen/cURL: ✔
SoapClient: ✔
DOMDocument: ✔
gzip: ✔
GD Graphics Library: ✔
Multibyte String: ✔
Remote Post: ✔
Remote Get: ✔
Give Configuration
Give Version: 2.2.0
Give Cache: Enabled
Database Updates: All DB Updates Completed.
Give Cache: Enabled
Give Cache: ✔New Donation✔Donation Receipt✔New Offline Donation✔Offline Donation Instructions✔New User Registration✔User Registration Information✔Email access✔Daily Email Report✔Weekly Email Report✔Monthly Email Report❌Renewal Receipt Email❌Subscription Cancelled Email❌Subscription Completed Email❌Subscription Reminder Email✔Subscriptions Email Access
Upgraded From: 2.2.0
Test Mode: Enabled
Currency Code: USD
Currency Position: Before
Decimal Separator: .
Thousands Separator: ,
Success Page: http://betagive.local/donation-confirmation/
Failure Page: http://betagive.local/donation-failed/
Donation History Page: http://betagive.local/donation-history/
Give Forms Slug: /donations/
Enabled Payment Gateways: Test Donation, PayPal Standard, Offline Donation
Default Payment Gateway: Test Donation
PayPal IPN Verification: Enabled
PayPal IPN Notifications: N/A
Donor Email Access: Enabled
Active Give Add-ons
Give - Currency Switcher: ❌ Unlicensed – by WordImpress – 1.2.1
Other Active Plugins
Email Cop: by Ashfame – 0.1.1
Give - Authorize.net Gateway: by WordImpress – 1.4.2
Give - Email Reports: by WordImpress – 1.1.2
Give - Google Analytics Donation Tracking: by WordImpress – 1.1.4
Give - Hook Helper: by Ravinder Kumar – 1.0
Give - MailChimp: by WordImpress – 1.4.1
Give - Manual Donations: by WordImpress – 1.4.1
Give - PayPal Pro Gateway: by WordImpress – 1.1.6
Give - PDF Receipts: by WordImpress – 2.3
Give - Per Form Gateways: by WordImpress – 1.0.1
Give - Recurring Donations: by WordImpress – 1.7.1
Give - Stripe Gateway: by WordImpress – 2.0.7
Give - Tributes: by WordImpress – 1.4.2
Give - Zapier: by WordImpress – 1.2.1
Inactive Plugins
Give - Fee Recovery: by WordImpress – 1.7.0
Give - Form Field Manager: by WordImpress – 1.4
Give - Gift Aid: by WordImpress – 1.1.4
Active MU Plugins
Bypass Login: by Stephen Carnam – 1.1.0
Local by Flywheel Relative URL (for Live Links): by Flywheel – 1.0
Theme
Name: Twenty Seventeen
Version: 1.6
Author URL: https://wordpress.org/
Child Theme: No – If you're modifying Give on a parent theme you didn't build personally, then we recommend using a child theme. See: How to Create a Child Theme
Operating System
Browser
The text was updated successfully, but these errors were encountered: