Completed
Push — 2.0 ( c8eb0d...914f97 )
by Raphaël
15:51 queued 12:54
created
src/ZohoDatabaseHelper.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -16,6 +16,7 @@
 block discarded – undo
16 16
      * Computes the name of the table based on the DAO plural module name.
17 17
      *
18 18
      * @param AbstractZohoDao $dao
19
+     * @param string $prefix
19 20
      *
20 21
      * @return string
21 22
      */
Please login to merge, or discard this patch.
src/ZohoDatabaseCopier.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -6,7 +6,6 @@
 block discarded – undo
6 6
 use Psr\Log\LoggerInterface;
7 7
 use Psr\Log\NullLogger;
8 8
 use Wabel\Zoho\CRM\AbstractZohoDao;
9
-use Wabel\Zoho\CRM\Request\Response;
10 9
 
11 10
 /**
12 11
  * This class is in charge of synchronizing one table of your database with Zoho records.
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -129,9 +129,9 @@
 block discarded – undo
129 129
             // Let's get the last modification date:
130 130
             $tableDetail = $this->connection->getSchemaManager()->listTableDetails($tableName);
131 131
             //We use createdTime instead of lastActivityTime when the field doesn't exist in order to follow changes.
132
-            if($tableDetail->hasColumn('lastActivityTime')){
132
+            if ($tableDetail->hasColumn('lastActivityTime')) {
133 133
                 $lastActivityTime = $this->connection->fetchColumn('SELECT MAX(lastActivityTime) FROM '.$tableName);
134
-            } else{
134
+            } else {
135 135
                 $lastActivityTime = $this->connection->fetchColumn('SELECT MAX(createdTime) FROM '.$tableName);
136 136
             }
137 137
             if ($lastActivityTime !== null) {
Please login to merge, or discard this patch.
src/ZohoDatabaseModelSync.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -7,7 +7,6 @@
 block discarded – undo
7 7
 use Psr\Log\LoggerInterface;
8 8
 use Psr\Log\NullLogger;
9 9
 use Wabel\Zoho\CRM\AbstractZohoDao;
10
-use Wabel\Zoho\CRM\Request\Response;
11 10
 
12 11
 /**
13 12
  * This class is in charge of synchronizing one table MODEL with Zoho.
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -86,15 +86,15 @@  discard block
 block discarded – undo
86 86
 
87 87
         $flatFields = ZohoDatabaseHelper::getFlatFields($dao->getFields());
88 88
         //@Temporary fix to use Mysql5.7 not strict
89
-        $table->addColumn('uid', 'string', ['length' => 36,'notnull'=>false]);
90
-        $table->addColumn('id', 'string', ['length' => 100,'notnull'=>false]);
89
+        $table->addColumn('uid', 'string', ['length' => 36, 'notnull'=>false]);
90
+        $table->addColumn('id', 'string', ['length' => 100, 'notnull'=>false]);
91 91
         $table->addUniqueIndex(['id']);
92 92
         $table->setPrimaryKey(['uid']);
93 93
 
94 94
         foreach ($flatFields as $field) {
95 95
             $columnName = $field['name'];
96 96
             //It seems sometime we can have the same field twice in the list of fields from the API.
97
-            if($table->hasColumn($columnName)){
97
+            if ($table->hasColumn($columnName)) {
98 98
                 continue;
99 99
             }
100 100
 
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
         $table = $schema->createTable($tableName);
203 203
 
204 204
         $flatFields = $users->getUserFields();
205
-        $table->addColumn('id', 'string', ['length' => 100,'notnull'=>false]);
205
+        $table->addColumn('id', 'string', ['length' => 100, 'notnull'=>false]);
206 206
         $table->setPrimaryKey(['id']);
207 207
         foreach ($flatFields as $field) {
208 208
             if (in_array($field, ['id'])) {
Please login to merge, or discard this patch.
src/LocalChangesTracker.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
         $localDelete = $schema->createTable('local_delete');
50 50
         $localDelete->addColumn('table_name', 'string', ['length' => 100]);
51 51
         $localDelete->addColumn('uid', 'string', ['length' => 36]);
52
-        $localDelete->addColumn('id', 'string', ['length' => 100,'notnull'=>false]);
52
+        $localDelete->addColumn('id', 'string', ['length' => 100, 'notnull'=>false]);
53 53
         $localDelete->setPrimaryKey(array('table_name', 'uid'));
54 54
         $localDelete->addUniqueIndex(['id', 'table_name']);
55 55
 
Please login to merge, or discard this patch.
src/ZohoSyncDatabaseCommand.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
         $this->zohoDatabaseModelSync = $zohoDatabaseModelSync;
94 94
         $this->zohoDatabaseCopier = $zohoDatabaseCopier;
95 95
         $this->zohoDatabaseSync = $zohoDatabaseSync;
96
-        $this->zohoEntitiesGenerator =  $zohoEntitiesGenerator;
96
+        $this->zohoEntitiesGenerator = $zohoEntitiesGenerator;
97 97
         $this->zohoClient = $zohoClient;
98 98
         $this->pathZohoDaos = $pathZohoDaos;
99 99
         $this->namespaceZohoDaos = $namespaceZohoDaos;
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
      */
183 183
     private function getListFieldName(AbstractZohoDao $zohoDao)
184 184
     {
185
-        $fieldNames= array();
185
+        $fieldNames = array();
186 186
         foreach ($zohoDao->getFields() as $fieldsDescriptor) {
187 187
             foreach (array_values($fieldsDescriptor) as $fieldDescriptor) {
188 188
                 $fieldNames[] = $fieldDescriptor['name'];
Please login to merge, or discard this patch.
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -194,7 +194,6 @@
 block discarded – undo
194 194
     
195 195
     /**
196 196
      * Regerate Zoho Daos
197
-     * @param InputInterface $input
198 197
      * @param OutputInterface $output
199 198
      */
200 199
     private function regenerateZohoDao(OutputInterface $output)
Please login to merge, or discard this patch.
src/ZohoDatabasePusher.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -126,8 +126,8 @@  discard block
 block discarded – undo
126 126
         //To optimize your API usage, get maximum 200 records with each request and insert, update or delete maximum 100 records with each request.
127 127
         $offsetTodo = 0;
128 128
         $doneBeans = 0;
129
-        while($doneBeans < $countItems){
130
-            $zohoBeansToSend = array_slice($zohoBeans,$offsetTodo,$this->apiLimitInsertUpdateDelete,true);
129
+        while ($doneBeans < $countItems) {
130
+            $zohoBeansToSend = array_slice($zohoBeans, $offsetTodo, $this->apiLimitInsertUpdateDelete, true);
131 131
             $offsetTodo += $this->apiLimitInsertUpdateDelete;
132 132
             $doneBeans += count($zohoBeansToSend);
133 133
         }
@@ -140,13 +140,13 @@  discard block
 block discarded – undo
140 140
                     // ID not exist we can update the new row with the Zoho ID
141 141
                     $this->connection->beginTransaction();
142 142
                     $this->connection->update($tableName, ['id' => $zohoBean->getZohoId()], ['uid' => $uid]);
143
-                    $this->connection->delete('local_insert', ['table_name'=>$tableName, 'uid' => $uid ]);
143
+                    $this->connection->delete('local_insert', ['table_name'=>$tableName, 'uid' => $uid]);
144 144
                     $this->connection->commit();
145 145
                 } else {
146 146
                     //ID already exist we need to delete the duplicate row.
147 147
                     $this->connection->beginTransaction();
148
-                    $this->connection->delete($tableName, ['uid' => $uid ]);
149
-                    $this->connection->delete('local_insert', ['table_name'=>$tableName, 'uid' => $uid ]);
148
+                    $this->connection->delete($tableName, ['uid' => $uid]);
149
+                    $this->connection->delete('local_insert', ['table_name'=>$tableName, 'uid' => $uid]);
150 150
                     $this->connection->commit();
151 151
                 }
152 152
             }
Please login to merge, or discard this patch.