Parse sync fields (#84)

* Parse sync fields

+ Properly use case mapping on sync related fields.

* Apply fixes from StyleCI (#85)

Co-authored-by: Craig Smith <[email protected]>

Co-authored-by: Craig Smith <[email protected]>
Co-authored-by: Craig Smith <[email protected]>

Author: samatcd

src/Contracts/Parser.php

@@ -147,11 +147,10 @@ protected function parseJoinSorts(Collection $sorts)
 
             $withTable = $relation->getRelated()->getTable();
 
-            $withTableName = strpos($withTable, ".") === false ? $withConnection . '.' . $withTable : $withTable;
+            $withTableName = strpos($withTable, '.') === false ? $withConnection.'.'.$withTable : $withTable;
 
             $this->repository->join($withTableName, "{$withTableName}.{$foreignKey}", "{$currentTable}.{$localKey}");
             $this->repository->orderBy("{$withTableName}.{$key}", $sortD);
-
         }
     }
 

src/Contracts/Relationships.php

@@ -183,13 +183,13 @@ protected function processBelongsToManyRelation($relation, array $relatedRecords
         $relatedKey = $relation->getRelatedKeyName();
 
         $model = $relation->getRelated();
-        $sync = collect();
+        $pivots = $relation->getPivotColumns();
 
-        $pivots =  $relation->getPivotColumns();
-        $detach = filter_var(request()->get('sync')[$with] ?? false, FILTER_VALIDATE_BOOLEAN);
+        $syncWithRelated = Helpers::snakeCaseArrayKeys(request()->get('sync') ?? []);
+        $detach = filter_var($syncWithRelated[Helpers::snake($with)] ?? false, FILTER_VALIDATE_BOOLEAN);
+        $sync = collect();
 
         foreach ($relatedRecords as $relatedRecord) {
-
             if (! isset($relatedRecord[$parentKey])) {
                 $relatedRecord[$parentKey] = $item->getAttribute($relatedKey);
             }
@@ -204,17 +204,14 @@ protected function processBelongsToManyRelation($relation, array $relatedRecords
             }
 
             $pvals = [];
-            if($pivots){
-
-                foreach($pivots as $pivot)
-                {
-                    if(isset($relatedRecord[$pivot])){
+            if ($pivots) {
+                foreach ($pivots as $pivot) {
+                    if (isset($relatedRecord[$pivot])) {
                         $pvals[$pivot] = $relatedRecord[$pivot];
                     }
                 }
             }
             $sync->put($record->getKey(), $pvals);
-
         }
 
         $relation->sync($sync->toArray(), $detach);

src/Helpers.php

@@ -162,17 +162,19 @@ public static function filterFieldsFromRequest($request, ?array $defaultFields,
     protected static function fieldsFromPutPost($request, $fields): array
     {
         $method = $request->method();
-        if (! in_array($method, ['PUT','POST','PATCH'])) {
+        if (! in_array($method, ['PUT', 'POST', 'PATCH'])) {
             return [];
         }
+
         return array_values(collect($request->all())->filter(function ($item, $key) use ($fields) {
             if (in_array($key, $fields)) {
                 return false;
             }
             if (! is_array($item) && ! is_object($key)) {
                 return false;
             }
-                return true;
+
+            return true;
         })->map(function ($item, $key) {
             return $key;
         })->toArray());

src/Http/Api/Contracts/HasResponse.php

@@ -243,7 +243,6 @@ protected function errorNotImplemented($message = 'Not implemented', ?array $err
         throw new HttpException(501, $message);
     }
 
-
     protected function handleIndexResponse($items)
     {
         return $this->respondWithMany($items);