XoopsModules25x / smartpartner

clone.php

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 42 and the first side effect is on line 18.

/*

Usage:

    Copy clone.php in <xoops_root>
    Change current working directory to <xoops_root>
    Update mappings as per new modulename.

    php -q clone.php

*/

// ##########################################################
//  Define your mapping here
// ##########################################################
$patterns = array(
    // first one must be module directory name
    'smartpartner'  => 'smartclient',
    'partner'       => 'client',
    'SMARTPARTNER'  => 'SMARTCLIENT',
    'SmartPartner'  => 'SmartClient',
    'Smart Partner' => 'Smart Client',
    'SPARTNER'      => 'SCLIENT',
    'Smartpartner'  => 'Smartclient',
    'Partner'       => 'Client',
    'Partenaire'    => 'Client',
    'partenaire'    => 'client'
);

$patKeys   = array_keys($patterns);
$patValues = array_values($patterns);

// work around for PHP < 5.0.x
if (!function_exists('file_put_contents')) {
    /**
     * @param      $filename
     * @param      $data
     * @param bool $file_append
     */
    function file_put_contents($filename, $data, $file_append = false)
    {
        $fp = fopen($filename, (!$file_append ? 'w+' : 'a+'));
        if (!$fp) {
            trigger_error('file_put_contents cannot write in file.', E_USER_ERROR);

            return;
        }
        fwrite($fp, $data);
        fclose($fp);
    }
}

// recursive clonning script
/**
 * @param $path
 * @throws
 */
function cloneFileFolder($path)
{
    global $patKeys;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

    global $patValues;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

    $newPath = str_replace($patKeys[0], $patValues[0], $path);

$newPath is not used, you could remove the assignment.

    $newPath = str_replace($patKeys[1], $patValues[1], $path);

    if (is_dir($path)) {
        // create new dir
        //        mkdir($newPath);

67% of this comment could be valid code. Did you maybe forget this after debugging?

        if (!@mkdir($newPath) && !is_dir($newPath)) {

This code seems to be duplicated across your project.

            throw Exception("Couldn't create this directory: " . $newPath);
        }

        // check all files in dir, and process it
        if ($handle = opendir($path)) {
            while ($file = readdir($handle)) {
                if ($file !== '.' && $file !== '..') {
                    cloneFileFolder("$path/$file");
                }
            }
            closedir($handle);
        }
    } else {
        if (preg_match('/(.jpg|.gif|.png|.zip)$/i', $path)) {
            copy($path, $newPath);
        } else {
            // file, read it
            $content = file_get_contents($path);
            $content = str_replace($patKeys, $patValues, $content);
            file_put_contents($newPath, $content);
        }
    }
}

cloneFileFolder('modules/smartpartner');

echo "Happy cloning...\n";
echo 'check directory modules/' . $patterns['smartpartner'] . " for cloned module \n";
echo "Consider modifying new module by editing language/english/modinfo.php and assets/images/logo_module.png manually (if you care)\n";