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

Getting unexpected error message while using tools #2744

Closed
ravinderk opened this issue Jan 30, 2018 · 3 comments
Closed

Getting unexpected error message while using tools #2744

ravinderk opened this issue Jan 30, 2018 · 3 comments
Assignees
@raftaar1191

Comments

@ravinderk
Copy link
Collaborator

Issue Overview

9qdpb0kswv

Expected Behavior

Either work or show an appropriate error message.

Current Behavior

Getting error when using any tools.
@ravinderk ravinderk added the bug label Jan 30, 2018
@ravinderk ravinderk added this to the 2.0.2 milestone Jan 30, 2018
@mehul0810
Copy link
Contributor

@ravinderk @raftaar1191 While performing Delete Test Payments, it is working fine and deleted donations and when there is no donation then also it show success message.

While performing Delete Test Donors and Payments, it is also working fine. But, when there is no payments, it shows an error message which is not related to it No data found for export parameters.

raftaar1191 added a commit to raftaar1191/Give that referenced this issue Jan 30, 2018
@raftaar1191
Fix table name error impress-org#2744
3be37b3
raftaar1191 added a commit to raftaar1191/Give that referenced this issue Jan 30, 2018
@raftaar1191
Change function impress-org#2744
92beec4
@raftaar1191 raftaar1191 mentioned this issue Jan 30, 2018
Closed
raftaar1191 added a commit to raftaar1191/Give that referenced this issue Jan 30, 2018
@raftaar1191
Add export true variable impress-org#2744
04eb855
raftaar1191 added a commit to raftaar1191/Give that referenced this issue Jan 30, 2018
@raftaar1191
Add constructor to make is_writable true for import impress-org#2744
00763ed
raftaar1191 added a commit to raftaar1191/Give that referenced this issue Jan 30, 2018
@raftaar1191
Add constructor parameter impress-org#2744
09df95e
@raftaar1191 raftaar1191 mentioned this issue Jan 30, 2018
Merged
3 tasks
@raftaar1191
Copy link
Contributor

@DevinWalker @ravinderk
I did not fix this comment that is being made by @mehul0810 because we are going to refactor the Data & export ajax code. This issue is already being created by @ravinderk ( did not find the issues link ).

Will fixed the comment at the time of refactoring the code of data and export

@raftaar1191
Copy link
Contributor

Fix DB error at the time of deleting the donor after the Give Upgrade is completed

[30-Jan-2018 18:48:47 UTC] WordPress database error Unknown column 'customer_id' in 'where clause' for query DELETE FROM `wpgive_give_donormeta` WHERE `customer_id` = 37 made by do_action('wp_ajax_give_do_ajax_export'), WP_Hook->do_action, WP_Hook->apply_filters, give_do_ajax_export, Give_Tools_Import_Donors->process_step, Give_Tools_Import_Donors->get_data, Give_DB_Donors->delete_by_user_id, Give_DB_Donor_Meta->delete_all_meta

DevinWalker added a commit that referenced this issue Jan 31, 2018
@DevinWalker
Merge branch 'release/2.0.3' into release/2.1
8c140db 
* release/2.0.3: (62 commits)
  Ran gulp
  Added change log for 2.0.2 and bumped version
  Added give_v201_create_tables() to 2.0.1 upgrades just to be super sure tables are created properly.
  Use direct sql queries for upgrade instead of WP_Query
  Remove code repetation #2737
  Fix price level #2737
  Fix coding standard #2737
  Remove meta update when form is alredy being created #2737
  Add give_get_total_post_type_count fn pass2
  Add give_get_total_post_type_count fn
  Revert "Add/Update Donation goal meta on donation form update or when donating #2250"
  Use get_posts instead of WP_Query in updates
  Add helper fn give_get_total_post_type_count
  Change fn order in file give_v202_add_form_goal_meta_callback
  Reverted links to only published status because it was causing link to go to blank entry when donor had only abandoned or pending donations
  Add constructor parameter #2744
  Add constructor to make is_writable true for import #2744
  Run upgrade after 2.0.1 upgrade removed from option
  Rerun  2.0.1 update into 2.0.2
  Change delete donor key#2744
  ...
DevinWalker added a commit that referenced this issue Jan 31, 2018
@DevinWalker
Merge branch 'release/2.1' into issue/339
540ab35 
* release/2.1: (101 commits)
  Ran gulp
  Added change log for 2.0.2 and bumped version
  Added give_v201_create_tables() to 2.0.1 upgrades just to be super sure tables are created properly.
  Use direct sql queries for upgrade instead of WP_Query
  Remove code repetation #2737
  Fix price level #2737
  Fix coding standard #2737
  Remove meta update when form is alredy being created #2737
  Add give_get_total_post_type_count fn pass2
  Add give_get_total_post_type_count fn
  Revert "Add/Update Donation goal meta on donation form update or when donating #2250"
  Use get_posts instead of WP_Query in updates
  Add helper fn give_get_total_post_type_count
  Change fn order in file give_v202_add_form_goal_meta_callback
  Reverted links to only published status because it was causing link to go to blank entry when donor had only abandoned or pending donations
  Add constructor parameter #2744
  Add constructor to make is_writable true for import #2744
  Run upgrade after 2.0.1 upgrade removed from option
  Rerun  2.0.1 update into 2.0.2
  Change delete donor key#2744
  ...

# Conflicts:
#	assets/css/give-admin.css
#	assets/js/admin/admin-forms.js
#	assets/js/admin/admin-scripts.js
#	assets/js/frontend/give-donations.js
#	assets/js/frontend/give.min.js
#	assets/sourcemaps/give-admin.css.map
#	includes/scripts.php
#	readme.txt
#	tests/unit-tests/tests-scripts.php
DevinWalker added a commit that referenced this issue Jan 31, 2018
@DevinWalker
Merge branch 'release/2.1' into feature/gutenberg-donation-form-block
d34a67f 
* release/2.1: (102 commits)
  Updated changelog md
  Ran gulp
  Added change log for 2.0.2 and bumped version
  Added give_v201_create_tables() to 2.0.1 upgrades just to be super sure tables are created properly.
  Use direct sql queries for upgrade instead of WP_Query
  Remove code repetation #2737
  Fix price level #2737
  Fix coding standard #2737
  Remove meta update when form is alredy being created #2737
  Add give_get_total_post_type_count fn pass2
  Add give_get_total_post_type_count fn
  Revert "Add/Update Donation goal meta on donation form update or when donating #2250"
  Use get_posts instead of WP_Query in updates
  Add helper fn give_get_total_post_type_count
  Change fn order in file give_v202_add_form_goal_meta_callback
  Reverted links to only published status because it was causing link to go to blank entry when donor had only abandoned or pending donations
  Add constructor parameter #2744
  Add constructor to make is_writable true for import #2744
  Run upgrade after 2.0.1 upgrade removed from option
  Rerun  2.0.1 update into 2.0.2
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@raftaar1191 raftaar1191

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ravinderk @mehul0810 @raftaar1191