Develop (#383)

* Duplicate handling.

* Added page of list with active duplicates.

* Full IP-check for confirmed duplicates.

* Simplification.

* Removed comment.

* Small update.

* Added database migration.

* Added unit test.

* track observers

* move comment

* update image.php

* fix null pointer (#379)

* Develop (#380) (#381)

* Duplicate handling.

* Added page of list with active duplicates.

* Full IP-check for confirmed duplicates.

* Simplification.

* Removed comment.

* Small update.

* Added database migration.

* Added unit test.

* track observers

* move comment

* update image.php

* fix null pointer (#379)

---------

Co-authored-by: Sneer <[email protected]>
Co-authored-by: Sneer-ra2 <[email protected]>

* add more logs (#382)

* add more logs

* update log

* Fix for createPlayer.

---------

Co-authored-by: Sneer <[email protected]>
Co-authored-by: Sneer-ra2 <[email protected]>

Author: alexp8

cncnet-api/app/Http/Controllers/ApiPlayerController.php

@@ -10,6 +10,7 @@
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
 use Illuminate\Validation\ValidationException;
+use Illuminate\Support\Facades\Log;
 
 class ApiPlayerController extends Controller
 {
@@ -73,6 +74,7 @@ public function getUsernames(Request $request)
         }
         catch (Exception $ex)
         {
+            Log::error("Exception in ApiPlayerController@getUsernames", ['exception' => $ex]);
             return response()->json(["message" => "Something went wrong"], 500);
         }
     }
@@ -100,7 +102,7 @@ public function createPlayer(Request $request)
 
             # Check ladder exists
             $ladder = \App\Models\Ladder::where("abbreviation", '=', $request->ladderAbbrev)->first();
-            $ladderHistory = $ladder->currentHistory;
+            $ladderHistory = $ladder->currentHistory();
             if ($ladder === null || $ladderHistory === null)
             {
                 return response()->json(["message" => "Ladder does not exist"], 400);
@@ -126,10 +128,12 @@ public function createPlayer(Request $request)
         }
         catch (ValidationException $ex)
         {
+            Log::warning("ValidationException in ApiPlayerController@createPlayer", ['errors' => $ex->errors()]);
             return response()->json(["message" => $ex->getMessage()], 400);
         }
         catch (Exception $ex)
         {
+            Log::error("Exception in ApiPlayerController@createPlayer", ['exception' => $ex]);
             return response()->json(["message" => "Something went wrong"], 500);
         }
     }
@@ -221,10 +225,12 @@ public function togglePlayerStatus(Request $request)
         }
         catch (ValidationException $ex)
         {
+            Log::warning("ValidationException in ApiPlayerController@togglePlayerStatus", ['errors' => $ex->errors()]);
             return response()->json(["message" => $ex->getMessage()], 400);
         }
         catch (Exception $ex)
         {
+            Log::error("Exception in ApiPlayerController@togglePlayerStatus", ['exception' => $ex]);
             return response()->json(["message" => "Something went wrong"], 500);
         }
     }