Offset Limit is not working after the first attempt

Hi,

Seems that memory usage is becoming a more common problem – perhaps due to a change in BP.. there have been some other fixes suggested, when I have time I’ll take a look into these.

Q

Okay, I see the problem with the offset. Line 278 of export-user-data.php has:

$args['number'] = $limit_total - $limit_offset;

So when you have an offset of 100 and a total of 100 $args[‘number’] is set to 0, so no users are found.

That line needs to be changed to

$args['number'] = $limit_total;

Thanks @cwjordan – seems like a stack of changes and updates to review – I’ll try to make time as soon as possible.

Q

I thought I’d drop this note in this topic also, as this includes a fix for the offset issue:

—————-

First push of 0.9.6 is up on github for testing:

https://github.com/qstudio/export-user-data

There are a few bits missing and a few changes, mostly to the saving of previous export data:

– I moved this from the wp_options table to the wp_usermeta – this will allow several users to store data on the same install
– I moved some of the saving and retrieving options around in the layout and reformatted the methods a little, to match the rest of the plugin
– I had to comment out the array data sanitizing lines for now, noted with an @todo – @cwjordan, please can you take a look at this, as it’s including the saved array as a key within itself?

I tested a 1500 member export and all the saving and loading functions, but most testing is required before this goes up on WP.

Any questions, just ask.

Q

I plan to release this tomorrow, unless anyone how spotted any issues in the github version.

Q