Wabel / zoho-crm-copy-db

src/ZohoDatabaseHelper.php

Doc Comments +1 added lines

@@ -16,6 +16,7 @@
      * Computes the name of the table based on the DAO plural module name.
      *
      * @param AbstractZohoDao $dao
+     * @param string $prefix
      *
      * @return string
      */

src/ZohoDatabaseCopier.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
 use Wabel\Zoho\CRM\AbstractZohoDao;
-use Wabel\Zoho\CRM\Request\Response;
 
 /**
  * This class is in charge of synchronizing one table of your database with Zoho records.

Spacing +7 added lines, -7 removed lines

@@ -102,7 +102,7 @@ 
                         $object = $user->{$fieldMethod}();
                         $data[$column->getName()] = $object->getName();
                     }
-                    elseif($column->getName() === 'Currency') {
+                    elseif ($column->getName() === 'Currency') {
                         //Todo: Do a pull request about \ZCRMUser::geCurrency() to \ZCRMUser::getCurrency()
                         $data[$column->getName()] = $user->geCurrency();
                     }
@@ -114,14 +114,14 @@ 
             $select->execute(['id' => $user->getId()]);
             $result = $select->fetch(\PDO::FETCH_ASSOC);
             if ($result === false && $data) {
-                $this->logger->debug("Inserting record with ID '" . $user->getId() . "'.");
+                $this->logger->debug("Inserting record with ID '".$user->getId()."'.");
 
                 $data['id'] = $user->getId();
                 $types['id'] = 'string';
 
                 $this->connection->insert($tableName, $data, $types);
-            } elseif($data) {
-                $this->logger->debug("Updating record with ID '" . $user->getId() . "'.");
+            } elseif ($data) {
+                $this->logger->debug("Updating record with ID '".$user->getId()."'.");
                 $identifier = ['id' => $user->getId()];
                 $types['id'] = 'string';
                 $this->connection->update($tableName, $data, $identifier, $types);
@@ -149,10 +149,10 @@ 
             // Let's get the last modification date:
             $tableDetail = $this->connection->getSchemaManager()->listTableDetails($tableName);
             $lastActivityTime = null;
-            if($tableDetail->hasColumn('modifiedTime')) {
+            if ($tableDetail->hasColumn('modifiedTime')) {
                 $lastActivityTime = $this->connection->fetchColumn('SELECT MAX(modifiedTime) FROM '.$tableName);
             }
-            if(!$lastActivityTime && $tableDetail->hasColumn('createdTime')) {
+            if (!$lastActivityTime && $tableDetail->hasColumn('createdTime')) {
                 $lastActivityTime = $this->connection->fetchColumn('SELECT MAX(createdTime) FROM '.$tableName);
             }
 
@@ -189,7 +189,7 @@ 
                     continue;
                 } else {
                     $field = $dao->getFieldFromFieldName($column->getName());
-                    if(!$field) {
+                    if (!$field) {
                         continue;
                     }
                     $getterName = $field->getGetter();

Braces +3 added lines, -6 removed lines

@@ -95,18 +95,15 @@
                         && array_key_exists('id', $user->{$fieldMethod}())
                     ) {
                         $data[$column->getName()] = $user->{$fieldMethod}()['name'];
-                    }
-                    elseif (method_exists($user, $fieldMethod)
+                    } elseif (method_exists($user, $fieldMethod)
                         && is_object($user->{$fieldMethod}()) && method_exists($user->{$fieldMethod}(), 'getName')
                     ) {
                         $object = $user->{$fieldMethod}();
                         $data[$column->getName()] = $object->getName();
-                    }
-                    elseif($column->getName() === 'Currency') {
+                    } elseif($column->getName() === 'Currency') {
                         //Todo: Do a pull request about \ZCRMUser::geCurrency() to \ZCRMUser::getCurrency()
                         $data[$column->getName()] = $user->geCurrency();
-                    }
-                    else {
+                    } else {
                         continue;
                     }
                 }

src/LocalChangesTracker.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
         $localDelete = $schema->createTable('local_delete');
         $localDelete->addColumn('table_name', 'string', ['length' => 100]);
         $localDelete->addColumn('uid', 'string', ['length' => 36]);
-        $localDelete->addColumn('id', 'string', ['length' => 100,'notnull'=>false]);
+        $localDelete->addColumn('id', 'string', ['length' => 100, 'notnull'=>false]);
         $localDelete->setPrimaryKey(array('table_name', 'uid'));
         $localDelete->addUniqueIndex(['id', 'table_name']);
 

src/ZohoDatabasePusher.php

Doc Comments -1 removed lines

@@ -202,7 +202,6 @@
      * Insert data to bean in order to update zoho records.
      *
      * @param ZohoBeanInterface $zohoBean
-     * @param array             $fieldsMatching
      * @param type              $columnName
      * @param type              $valueDb
      */

Switch Indentation +20 added lines, -20 removed lines

@@ -230,26 +230,26 @@
     private function formatValueToBeans($type, $value)
     {
         switch ($type) {
-        case 'date':
-            $value = \DateTime::createFromFormat('Y-m-d', $value)?:null;
-            break;
-        case 'datetime':
-            $value = \DateTime::createFromFormat('Y-m-d H:i:s', $value)?:null;
-            break;
-        case 'boolean' :
-            $value = (bool) $value;
-            break;
-        case 'percent' :
-            $value = (int) $value;
-            break;
-        case 'double' :
-            $value = number_format($value, 2);
-            break;
-        case 'multiselectlookup':
-        case 'multiuserlookup':
-        case 'multiselectpicklist':
-            $value = explode(';', $value);
-            break;
+            case 'date':
+                $value = \DateTime::createFromFormat('Y-m-d', $value)?:null;
+                break;
+            case 'datetime':
+                $value = \DateTime::createFromFormat('Y-m-d H:i:s', $value)?:null;
+                break;
+            case 'boolean' :
+                $value = (bool) $value;
+                break;
+            case 'percent' :
+                $value = (int) $value;
+                break;
+            case 'double' :
+                $value = number_format($value, 2);
+                break;
+            case 'multiselectlookup':
+            case 'multiuserlookup':
+            case 'multiselectpicklist':
+                $value = explode(';', $value);
+                break;
         }
 
         return $value;

Spacing +11 added lines, -11 removed lines

@@ -47,7 +47,7 @@ 
             $this->logger = $logger;
         }
         $this->apiLimitInsertUpdateDelete = $apiLimitInsertUpdateDelete;
-        if($apiLimitInsertUpdateDelete === null) {
+        if ($apiLimitInsertUpdateDelete === null) {
             $this->apiLimitInsertUpdateDelete = 100;
         }
     }
@@ -81,7 +81,7 @@ 
     {
         $localTable = $update ? 'local_update' : 'local_insert';
         $tableName = ZohoDatabaseHelper::getTableName($zohoDao, $this->prefix);
-        do{
+        do {
             $rowsDeleted = [];
             //@see https://www.zoho.com/crm/help/api/v2/#ra-update-records
             //To optimize your API usage, get maximum 200 records with each request and insert, update or delete maximum 100 records with each request.
@@ -128,11 +128,11 @@ 
                     $rowsDeleted[] = $row['uid'];
                 }
             }
-            if($zohoBeans) {
+            if ($zohoBeans) {
                 $this->sendDataToZohoCleanLocal($zohoDao, $zohoBeans, $rowsDeleted, $update);
             }
             $countToPush = $this->countElementInTable($zohoDao, $update);
-        } while($countToPush > 0);
+        } while ($countToPush > 0);
     }
 
     /**
@@ -141,7 +141,7 @@ 
      * @param string[]            $rowsDeleted
      * @param bool                $update
      */
-    private function sendDataToZohoCleanLocal(AbstractZohoDao $zohoDao, array $zohoBeans,$rowsDeleted, $update = false)
+    private function sendDataToZohoCleanLocal(AbstractZohoDao $zohoDao, array $zohoBeans, $rowsDeleted, $update = false)
     {
         $tableName = ZohoDatabaseHelper::getTableName($zohoDao, $this->prefix);
         $zohoDao->save($zohoBeans);
@@ -153,13 +153,13 @@ 
                     // ID not exist we can update the new row with the Zoho ID
                     $this->connection->beginTransaction();
                     $this->connection->update($tableName, ['id' => $zohoBean->getZohoId()], ['uid' => $uid]);
-                    $this->connection->delete('local_insert', ['table_name'=>$tableName, 'uid' => $uid ]);
+                    $this->connection->delete('local_insert', ['table_name'=>$tableName, 'uid' => $uid]);
                     $this->connection->commit();
                 } else {
                     //ID already exist we need to delete the duplicate row.
                     $this->connection->beginTransaction();
-                    $this->connection->delete($tableName, ['uid' => $uid ]);
-                    $this->connection->delete('local_insert', ['table_name'=>$tableName, 'uid' => $uid ]);
+                    $this->connection->delete($tableName, ['uid' => $uid]);
+                    $this->connection->delete('local_insert', ['table_name'=>$tableName, 'uid' => $uid]);
                     $this->connection->commit();
                 }
             }
@@ -187,7 +187,7 @@ 
     {
         foreach ($row as $columnName => $columnValue) {
             $fieldMethod = $dao->getFieldFromFieldName($columnName);
-            if(!in_array($columnName, EntitiesGeneratorService::$defaultDateFields) && $fieldMethod
+            if (!in_array($columnName, EntitiesGeneratorService::$defaultDateFields) && $fieldMethod
                 && (!in_array($columnName, ['id', 'uid'])) && !is_null($columnValue)
             ) {
                 $type = $fieldMethod->getType();
@@ -231,10 +231,10 @@ 
     {
         switch ($type) {
         case 'date':
-            $value = \DateTime::createFromFormat('Y-m-d', $value)?:null;
+            $value = \DateTime::createFromFormat('Y-m-d', $value) ?: null;
             break;
         case 'datetime':
-            $value = \DateTime::createFromFormat('Y-m-d H:i:s', $value)?:null;
+            $value = \DateTime::createFromFormat('Y-m-d H:i:s', $value) ?: null;
             break;
         case 'boolean' :
             $value = (bool) $value;

src/ZohoSyncDatabaseCommand.php

Doc Comments -1 removed lines

@@ -190,7 +190,6 @@
     /**
      * Regerate Zoho Daos
      *
-     * @param InputInterface  $input
      * @param OutputInterface $output
      */
     private function regenerateZohoDao(OutputInterface $output)

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use Logger\Formatters\DateTimeFormatter;
 use Mouf\Utils\Log\Psr\MultiLogger;
 use Wabel\Zoho\CRM\Request\Response;
-use zcrmsdk\crm\utility\ZCRMConfigUtil;
 
 class ZohoSyncDatabaseCommand extends Command
 {

Spacing +5 added lines, -5 removed lines

@@ -101,7 +101,7 @@ 
         $this->zohoDatabaseModelSync = $zohoDatabaseModelSync;
         $this->zohoDatabaseCopier = $zohoDatabaseCopier;
         $this->zohoDatabaseSync = $zohoDatabaseSync;
-        $this->zohoEntitiesGenerator =  $zohoEntitiesGenerator;
+        $this->zohoEntitiesGenerator = $zohoEntitiesGenerator;
         $this->zohoClient = $zohoClient;
         $this->pathZohoDaos = $pathZohoDaos;
         $this->namespaceZohoDaos = $namespaceZohoDaos;
@@ -137,7 +137,7 @@ 
             if ($input->getOption('log-path') && $input->getOption('clear-logs')) {
                 $output->writeln('Clearing logs...');
                 $path = $input->getOption('log-path');
-                $logFile = $path . '/ZCRMClientLibrary.log';
+                $logFile = $path.'/ZCRMClientLibrary.log';
                 if (file_exists($logFile)) {
                     if (is_writable($logFile)) {
                         if (file_put_contents($logFile, '') === false) {
@@ -151,7 +151,7 @@ 
                 }
             }
 
-            if(!$input->getOption('limit')) {
+            if (!$input->getOption('limit')) {
                 ini_set('memory_limit', '-1');
             }
 
@@ -176,7 +176,7 @@ 
             if ($input->getOption('log-path') && $input->getOption('dump-logs')) {
                 $output->writeln('Dumping logs...');
                 $path = $input->getOption('log-path');
-                $logFile = $path . '/ZCRMClientLibrary.log';
+                $logFile = $path.'/ZCRMClientLibrary.log';
                 if (file_exists($logFile)) {
                     if (is_readable($logFile)) {
                         $output->writeln(file_get_contents($logFile));
@@ -296,7 +296,7 @@ 
     {
         $output->writeln('Start to synchronize Zoho data into Zoho CRM.');
         foreach ($this->zohoDaos as $zohoDao) {
-            if($zohoDao->getFieldFromFieldName('createdTime')) {
+            if ($zohoDao->getFieldFromFieldName('createdTime')) {
                 $this->zohoDatabaseSync->pushToZoho($zohoDao);
             }
         }

src/ZohoDatabaseModelSync.php

Switch Indentation +91 added lines, -91 removed lines

@@ -102,80 +102,80 @@ 
             $options = [];
             // Note: full list of types available here: https://www.zoho.com/crm/help/customization/custom-fields.html
             switch ($field->getType()) {
-            case 'fileupload':
-                $type = 'string';
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 255;
-                break;
-            case 'lookup':
-                $type = 'string';
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 100;
-                $index = true;
-                break;
-            case 'userlookup':
-            case 'ownerlookup':
-                $type = 'string';
-                $index = true;
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 25;
-                break;
-            case 'formula':
-                // Note: a Formula can return any type, but we have no way to know which type it returns...
-                $type = 'string';
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 100;
-                break;
-            case 'datetime':
-                $type = 'datetime';
-                break;
-            case 'date':
-                $type = 'date';
-                break;
-            case 'boolean':
-                $type = 'boolean';
-                break;
-            case 'textarea':
-                $type = 'text';
-                break;
-            case 'bigint':
-                $type = 'bigint';
-                break;
-            case 'phone':
-            case 'text':
-            case 'url':
-            case 'email':
-            case 'picklist':
-            case 'website':
-                $type = 'string';
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 255;
-                break;
-            case 'multiselectlookup':
-            case 'multiuserlookup':
-            case 'multiselectpicklist':
-                $type = 'text';
-                break;
-            case 'percent':
-                $type = 'integer';
-                break;
-            case 'double':
-                $type = 'float';
-                break;
-            case 'autonumber':
-            case 'integer':
-                $type = 'integer';
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 255;
-                break;
-            case 'currency':
-            case 'decimal':
-                $type = 'decimal';
-                $options['scale'] = 2;
-                break;
-            case 'consent_lookup':
-            case 'profileimage':
-            case 'ALARM':
-            case 'RRULE':
-            case 'event_reminder':
-                continue 2;
+                case 'fileupload':
+                    $type = 'string';
+                    $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 255;
                     break;
-            default:
-                throw new \RuntimeException('Unknown type "'.$field->getType().'"');
+                case 'lookup':
+                    $type = 'string';
+                    $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 100;
+                    $index = true;
+                    break;
+                case 'userlookup':
+                case 'ownerlookup':
+                    $type = 'string';
+                    $index = true;
+                    $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 25;
+                    break;
+                case 'formula':
+                    // Note: a Formula can return any type, but we have no way to know which type it returns...
+                    $type = 'string';
+                    $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 100;
+                    break;
+                case 'datetime':
+                    $type = 'datetime';
+                    break;
+                case 'date':
+                    $type = 'date';
+                    break;
+                case 'boolean':
+                    $type = 'boolean';
+                    break;
+                case 'textarea':
+                    $type = 'text';
+                    break;
+                case 'bigint':
+                    $type = 'bigint';
+                    break;
+                case 'phone':
+                case 'text':
+                case 'url':
+                case 'email':
+                case 'picklist':
+                case 'website':
+                    $type = 'string';
+                    $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 255;
+                    break;
+                case 'multiselectlookup':
+                case 'multiuserlookup':
+                case 'multiselectpicklist':
+                    $type = 'text';
+                    break;
+                case 'percent':
+                    $type = 'integer';
+                    break;
+                case 'double':
+                    $type = 'float';
+                    break;
+                case 'autonumber':
+                case 'integer':
+                    $type = 'integer';
+                    $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 255;
+                    break;
+                case 'currency':
+                case 'decimal':
+                    $type = 'decimal';
+                    $options['scale'] = 2;
+                    break;
+                case 'consent_lookup':
+                case 'profileimage':
+                case 'ALARM':
+                case 'RRULE':
+                case 'event_reminder':
+                    continue 2;
+                        break;
+                default:
+                    throw new \RuntimeException('Unknown type "'.$field->getType().'"');
             }
 
 
@@ -222,24 +222,24 @@ 
             $length = null;
             $index = false;
             switch ($field) {
-            case 'zuid':
-                $type = 'string';
-                $length = 100;
-                $index = true;
-                break;
-            case 'name':
-            case 'email':
-                $type = 'string';
-                $length = 255;
-                $index = true;
-                break;
-            case 'phone':
-            case 'website':
-                $type = 'text';
-                break;
-            default:
-                $type = 'string';
-                $length = 100;
+                case 'zuid':
+                    $type = 'string';
+                    $length = 100;
+                    $index = true;
+                    break;
+                case 'name':
+                case 'email':
+                    $type = 'string';
+                    $length = 255;
+                    $index = true;
+                    break;
+                case 'phone':
+                case 'website':
+                    $type = 'text';
+                    break;
+                default:
+                    $type = 'string';
+                    $length = 100;
             }
 
             $options = [];

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
 use Wabel\Zoho\CRM\AbstractZohoDao;
-use Wabel\Zoho\CRM\Request\Response;
 use zcrmsdk\crm\setup\users\ZCRMUser;
 
 /**

Spacing +10 added lines, -10 removed lines

@@ -86,15 +86,15 @@ 
         $table = $schema->createTable($tableName);
 
         //@Temporary fix to use Mysql5.7 not strict
-        $table->addColumn('uid', 'string', ['length' => 36,'notnull'=>false]);
-        $table->addColumn('id', 'string', ['length' => 100,'notnull'=>false]);
+        $table->addColumn('uid', 'string', ['length' => 36, 'notnull'=>false]);
+        $table->addColumn('id', 'string', ['length' => 100, 'notnull'=>false]);
         $table->addUniqueIndex(['id']);
         $table->setPrimaryKey(['uid']);
 
         foreach ($dao->getFields() as $field) {
             $columnName = $field->getName();
             //It seems sometime we can have the same field twice in the list of fields from the API.
-            if($table->hasColumn($columnName)) {
+            if ($table->hasColumn($columnName)) {
                 continue;
             }
 
@@ -105,23 +105,23 @@ 
             switch ($field->getType()) {
             case 'fileupload':
                 $type = 'string';
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 255;
+                $length = $field->getMaxlength() && $field->getMaxlength() > 0 ? $field->getMaxlength() : 255;
                 break;
             case 'lookup':
                 $type = 'string';
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 100;
+                $length = $field->getMaxlength() && $field->getMaxlength() > 0 ? $field->getMaxlength() : 100;
                 $index = true;
                 break;
             case 'userlookup':
             case 'ownerlookup':
                 $type = 'string';
                 $index = true;
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 25;
+                $length = $field->getMaxlength() && $field->getMaxlength() > 0 ? $field->getMaxlength() : 25;
                 break;
             case 'formula':
                 // Note: a Formula can return any type, but we have no way to know which type it returns...
                 $type = 'string';
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 100;
+                $length = $field->getMaxlength() && $field->getMaxlength() > 0 ? $field->getMaxlength() : 100;
                 break;
             case 'datetime':
                 $type = 'datetime';
@@ -145,7 +145,7 @@ 
             case 'picklist':
             case 'website':
                 $type = 'string';
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 255;
+                $length = $field->getMaxlength() && $field->getMaxlength() > 0 ? $field->getMaxlength() : 255;
                 break;
             case 'multiselectlookup':
             case 'multiuserlookup':
@@ -161,7 +161,7 @@ 
             case 'autonumber':
             case 'integer':
                 $type = 'integer';
-                $length = $field->getMaxlength() && $field->getMaxlength() > 0?$field->getMaxlength() : 255;
+                $length = $field->getMaxlength() && $field->getMaxlength() > 0 ? $field->getMaxlength() : 255;
                 break;
             case 'currency':
             case 'decimal':
@@ -213,7 +213,7 @@ 
         $schema = new Schema();
         $table = $schema->createTable($tableName);
 
-        $table->addColumn('id', 'string', ['length' => 100,'notnull'=>false]);
+        $table->addColumn('id', 'string', ['length' => 100, 'notnull'=>false]);
         $table->setPrimaryKey(['id']);
         foreach (ZCRMUser::$defaultKeys as $field) {
             if ($field === 'id') {

src/Log/ZohoSyncSQLLogger.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\Logging\SQLLogger;
-use Doctrine\DBAL\SQLParserUtils;
 use function PHPSTORM_META\type;
 use Psr\Log\LoggerInterface;
 

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
         $query = null;
         if ($params && $this->authorizedSQLWithParams) {
             $this->logger->debug($sql.' -- Params : {params} - Types : {types}', ['params' => json_encode($params), 'types' => json_encode($types)]);
-        } elseif($this->authorizedSQLWithoutParams) {
+        } elseif ($this->authorizedSQLWithoutParams) {
             $this->logger->debug($sql);
         }
     }