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

Problems with s2Member Export After 20170221 Update #1055

Closed
patdumond opened this Issue Feb 21, 2017 · 6 comments

Comments

Projects
None yet
3 participants
@patdumond

patdumond commented Feb 21, 2017

EXPLANATION OF THE ISSUE

I updated both s2Member and s2Member Pro to 170221 today on a couple of client sites: one is a developmental site and the other is a production site. They are pretty close to having the same configurations, but they are not "in sync". On the production site, when I run an s2Member Export after the update all the data from the "First Payment Date" column on is blank in the export. The data exists in the database. I can export it using the "Export User Data" plugin, but it is pretty much unusable because EUD doesn't unserialize the arrays or change the timestamps into dates readable by Excel, etc...

On the developmental site, however, the s2Member Export still works fine, so it isn't simply an issue of the update. To complicate matters, the production site was migrated from one hosting provider (A2) to another (SiteGround) yesterday. I did an s2Member IMPORT yesterday and THAT worked fine. (Pre-update obviously.)

I'm looking for any help I can get to figure out what caused this and how to fix it. This client relies heavily on being able to export and import s2Member data at will.

STEPS TO REPRODUCE THE ISSUE

I'm not sure. ;)

BEHAVIOR THAT I EXPECTED

I expected that the s2Member data would be exported in the s2Member Export (since it is being exported, but not made presentable) by the Export User Data plugin and clearly exists in the database.

This isn't a clean installation, but it worked with this configuration before moving the site to SiteGround and doing the update this morning. The developmental site is running the same theme and plugins and it is STILL working. The developmental site is ALSO on SiteGround, but it has ALWAYS been on SiteGround.

Thanks in advance for any help you can give me in figuring this out.

@jaswrks

This comment has been minimized.

Show comment
Hide comment
@jaswrks

jaswrks Feb 22, 2017

Member

all the data from the "First Payment Date" column on is blank in the export

Attempting to reproduce this.

Member

jaswrks commented Feb 22, 2017

all the data from the "First Payment Date" column on is blank in the export

Attempting to reproduce this.

@jaswrks

This comment has been minimized.

Show comment
Hide comment
@jaswrks

jaswrks Feb 22, 2017

Member

I was able to reproduce this using the 'Simple' variation of the export system in s2Member. The 'Advanced' export tool does not seem to be impacted by this bug.

Notice: Array to string conversion in /app/src/wp-content/plugins/s2member-pro/src/includes/classes/exports-simple-in.inc.php on line 124

Call Stack:
    0.0047     433872   1. {main}() /app/src/wp-admin/admin.php:0
    0.0052     446480   2. require_once('/app/src/wp-load.php') /app/src/wp-admin/admin.php:31
    0.0055     459624   3. require_once('/app/src/wp-config.php') /app/src/wp-load.php:37
    0.0070     539728   4. require_once('/app/src/wp-settings.php') /app/src/wp-config.php:41
    0.5487   27539856   5. do_action() /app/src/wp-settings.php:449
    0.5487   27540232   6. WP_Hook->do_action() /app/src/wp-includes/plugin.php:453
    0.5487   27540232   7. WP_Hook->apply_filters() /app/src/wp-includes/class-wp-hook.php:323
    0.5840   27856592   8. c_ws_plugin__s2member_pro_exports_simple::export() /app/src/wp-includes/class-wp-hook.php:298
    0.5860   27992368   9. c_ws_plugin__s2member_pro_exports_simple_in::export_users() /app/src/wp-content/plugins/s2member-pro/src/includes/classes/exports-simple.inc.php:58
Member

jaswrks commented Feb 22, 2017

I was able to reproduce this using the 'Simple' variation of the export system in s2Member. The 'Advanced' export tool does not seem to be impacted by this bug.

Notice: Array to string conversion in /app/src/wp-content/plugins/s2member-pro/src/includes/classes/exports-simple-in.inc.php on line 124

Call Stack:
    0.0047     433872   1. {main}() /app/src/wp-admin/admin.php:0
    0.0052     446480   2. require_once('/app/src/wp-load.php') /app/src/wp-admin/admin.php:31
    0.0055     459624   3. require_once('/app/src/wp-config.php') /app/src/wp-load.php:37
    0.0070     539728   4. require_once('/app/src/wp-settings.php') /app/src/wp-config.php:41
    0.5487   27539856   5. do_action() /app/src/wp-settings.php:449
    0.5487   27540232   6. WP_Hook->do_action() /app/src/wp-includes/plugin.php:453
    0.5487   27540232   7. WP_Hook->apply_filters() /app/src/wp-includes/class-wp-hook.php:323
    0.5840   27856592   8. c_ws_plugin__s2member_pro_exports_simple::export() /app/src/wp-includes/class-wp-hook.php:298
    0.5860   27992368   9. c_ws_plugin__s2member_pro_exports_simple_in::export_users() /app/src/wp-content/plugins/s2member-pro/src/includes/classes/exports-simple.inc.php:58

@jaswrks jaswrks added bug and removed needs testing labels Feb 22, 2017

jaswrks pushed a commit to websharks/s2member-pro that referenced this issue Feb 22, 2017

@jaswrks

This comment has been minimized.

Show comment
Hide comment
@jaswrks

jaswrks Feb 22, 2017

Member

Short-Term Workarounds

  • Option 1. Use 'Advanced' export tool instead of 'Simple' export tool.
  • Option 2. Download this updated file and upload via FTP, allowing it to replace your existing copy of s2member-pro/src/includes/classes/exports-simple-in.inc.php
Member

jaswrks commented Feb 22, 2017

Short-Term Workarounds

  • Option 1. Use 'Advanced' export tool instead of 'Simple' export tool.
  • Option 2. Download this updated file and upload via FTP, allowing it to replace your existing copy of s2member-pro/src/includes/classes/exports-simple-in.inc.php

jaswrks pushed a commit that referenced this issue Feb 22, 2017

jaswsinc
@jaswrks

This comment has been minimized.

Show comment
Hide comment
@jaswrks

jaswrks Feb 22, 2017

Member

Next Release Changelog

  • (s2Member Pro) Bug Fix: Updating PHP syntax in Simple Export tool, for compatibility w/ modern versions of PHP. Props @patdumond for reporting and helping us locate the underlying cause of this problem. See Issue #1055.
Member

jaswrks commented Feb 22, 2017

Next Release Changelog

  • (s2Member Pro) Bug Fix: Updating PHP syntax in Simple Export tool, for compatibility w/ modern versions of PHP. Props @patdumond for reporting and helping us locate the underlying cause of this problem. See Issue #1055.

@jaswrks jaswrks added this to the Next Release milestone Feb 22, 2017

@jaswrks jaswrks self-assigned this Feb 22, 2017

@patdumond

This comment has been minimized.

Show comment
Hide comment
@patdumond

patdumond Feb 22, 2017

@jaswsinc - Thanks for the quick response, but I'm getting a 500 error after patching. Here's the debug.log results:

[22-Feb-2017 08:59:57 UTC] PHP Fatal error:  Uncaught Error: Class 'c_ws_plugin__s2member_pro_exports_simple_in' not found in /home/themilkm/public_html/wp-content/plugins/s2member-pro/src/includes/classes/exports-simple.inc.php:58
Stack trace:
#0 /home/themilkm/public_html/wp-includes/class-wp-hook.php(298): c_ws_plugin__s2member_pro_exports_simple::export('')
#1 /home/themilkm/public_html/wp-includes/class-wp-hook.php(323): WP_Hook->apply_filters(NULL, Array)
#2 /home/themilkm/public_html/wp-includes/plugin.php(453): WP_Hook->do_action(Array)
#3 /home/themilkm/public_html/wp-settings.php(449): do_action('init')
#4 /home/themilkm/public_html/wp-config.php(97): require_once('/home/themilkm/...')
#5 /home/themilkm/public_html/wp-load.php(37): require_once('/home/themilkm/...')
#6 /home/themilkm/public_html/wp-admin/admin.php(31): require_once('/home/themilkm/...')
#7 {main}
  thrown in /home/themilkm/public_html/wp-content/plugins/s2member-pro/src/includes/classes/exports-simple.inc.php on line 58

Update: Doh! I forgot to save the file in Brackets... It was empty... Working now. Thanks again for the quick response. :)

patdumond commented Feb 22, 2017

@jaswsinc - Thanks for the quick response, but I'm getting a 500 error after patching. Here's the debug.log results:

[22-Feb-2017 08:59:57 UTC] PHP Fatal error:  Uncaught Error: Class 'c_ws_plugin__s2member_pro_exports_simple_in' not found in /home/themilkm/public_html/wp-content/plugins/s2member-pro/src/includes/classes/exports-simple.inc.php:58
Stack trace:
#0 /home/themilkm/public_html/wp-includes/class-wp-hook.php(298): c_ws_plugin__s2member_pro_exports_simple::export('')
#1 /home/themilkm/public_html/wp-includes/class-wp-hook.php(323): WP_Hook->apply_filters(NULL, Array)
#2 /home/themilkm/public_html/wp-includes/plugin.php(453): WP_Hook->do_action(Array)
#3 /home/themilkm/public_html/wp-settings.php(449): do_action('init')
#4 /home/themilkm/public_html/wp-config.php(97): require_once('/home/themilkm/...')
#5 /home/themilkm/public_html/wp-load.php(37): require_once('/home/themilkm/...')
#6 /home/themilkm/public_html/wp-admin/admin.php(31): require_once('/home/themilkm/...')
#7 {main}
  thrown in /home/themilkm/public_html/wp-content/plugins/s2member-pro/src/includes/classes/exports-simple.inc.php on line 58

Update: Doh! I forgot to save the file in Brackets... It was empty... Working now. Thanks again for the quick response. :)

@raamdev

This comment has been minimized.

Show comment
Hide comment
@raamdev

raamdev May 24, 2017

Member

s2Member v170524 has been released and includes changes from this GitHub Issue. See the v170524 announcement for further details.


This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference this one (#1055).

Member

raamdev commented May 24, 2017

s2Member v170524 has been released and includes changes from this GitHub Issue. See the v170524 announcement for further details.


This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference this one (#1055).

@raamdev raamdev closed this May 24, 2017

@websharks websharks locked and limited conversation to collaborators May 24, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.