WordPress.org

Ready to get started?Download WordPress

Forums

NS Cloner - Site Copier
[resolved] Capitalization in new_site_name field causes significant failure (4 posts)

  1. Aram Zucker-Scharff
    Member
    Posted 1 year ago #

    If the user places a string with capitalization in it in the new_site_name field, the cloning process will place that capitalized string into the database (which is very bad to begin with). But it becomes even worse if the user than attempts to clone the resulting site. That site will be duplicated with any new URL placed in the subsequent attempts' new_site_name field not properly propagating to the database, giving all subsequent sites the identical URL in the database (with the same bad caps in the database) and the same occurring for any sites cloned from those sites.

    In short: disaster.

    There are good and better ways to solve this issue, but since I just want to resolve the problem ASAP, i just created a patch to de-cap all letters in the resolved field when processing for cloning.

    I suggest that the changes I've made at https://github.com/AramZS/ns-cloner-site-copier/commit/11102efcf72bb88b0b848e1c0d91ca11cf9cb838 (or some other better validation check at the user level) be merged into the next patch for the plugin.

    Thanks for the great plugin!

    http://wordpress.org/extend/plugins/ns-cloner-site-copier/

  2. Never Settle
    Member
    Plugin Author

    Posted 1 year ago #

    And very many thanks for your input! We'll take a look at those recommendations and merge them at the soonest opportunity. Very much appreciated.

    I honestly would have never thought of using caps in a site name. It's essentially a subdomain or subfolder and as part of a URL I would never use uppercase. But fantastic catch and always better to be overprotective of bad input.

    All the best!

    PS
    (Did you find the logs? Or is there still an issue there?) they should be getting generated under /logs in the plugin directory.

  3. Never Settle
    Member
    Plugin Author

    Posted 11 months ago #

    Just cleaning up old threads and wanted to let you know we've closed the loop on this with the latest release just pushed. Thanks again!

    You can read more about the new version here:
    http://neversettle.it/ns-cloner-plugin-for-wordpress-can-clone-main-site/

  4. Aram Zucker-Scharff
    Member
    Posted 3 months ago #

    Great, thanks!

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic