Issue in “cron import” tab

  • Resolved alex.gav

    (@alexgav)


    6 years, 8 months ago

    Hi it looks like there is an issue in the “cron import” tab.
    The checkbox for “Update roles for existing users?” shows incorrect (it is always unchecked). I have checked the code and it looks like the is an issue in line 137
    <input type="checkbox" name="update-roles-existing-users" value="yes" <?php if( $update_roles_existing_users == 'yes' ) echo "checked='checked'"; ?>/>
    The value for $update_roles_existing_users should be checked for 1 since this is what you are storing in DB.

    Can you also please fix:

    "PHP message: PHP Warning:  explode() expects parameter 2 to be string, array given in
 /wp-content/plugins/import-users-from-csv-with-meta/importer.php on line 210PHP message: PHP Warning:  array_walk() expects paramete
r 1 to be array, null given in /wp-content/plugins/import-users-from-csv-with-meta/importer.php on line 211

    Thank you for the plugin!
    Alex.

    • This topic was modified 6 years, 8 months ago by alex.gav.
Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author Javier Carazo

    (@carazo)

    6 years, 8 months ago

    @alexgav,

    I have just fixed the problems you reported, please check and if there is something wrong, tell me again.

    Thank you very much for debugging!!

    Thread Starter alex.gav

    (@alexgav)

    6 years, 8 months ago

    Thanks for the quick fix!
    The update roles option show correct now.
    Now I am checking the functionality for update users using cron and there is an issue with roles update, roles not getting updated, while using manual update with the same file (using same file path on server) working good.
    Please let me know if you need some more info to fix the issue.

    Plugin Author Javier Carazo

    (@carazo)

    6 years, 8 months ago

    I have just made a change into the code.

    Can you check?

    Thread Starter alex.gav

    (@alexgav)

    6 years, 8 months ago

    Thanks!
    After updating the plugin the roles update is working.

    • This reply was modified 6 years, 8 months ago by alex.gav.
    • This reply was modified 6 years, 8 months ago by alex.gav.
    Thread Starter alex.gav

    (@alexgav)

    6 years, 8 months ago

    One more thought,
    For me and probably for other users wit would be helpful to have more options for
    deleting users that are not present in the CSV. I would like to have list of roles in which users should not be removed.

    Thanks again for the plugin!

    Plugin Author Javier Carazo

    (@carazo)

    6 years, 8 months ago

    @alexgav,

    Thanks! Please, if you have one minute leave a good review or make a little donation to keep this plugin alive with this good support 🙂

    Respect to new features, we can do it if you support the develop. You can pay what it will take to do it or you can send us the code to include into the plugin.

Viewing 6 replies - 1 through 6 (of 6 total)

The topic ‘Issue in “cron import” tab’ is closed to new replies.