Fixed api issue

Reviewed-on: #11

app/Http/Controllers/api/GlobalSearchApiController.php

@@ -42,80 +42,30 @@ class GlobalSearchApiController extends Controller
         $perPage = min(max((int) $request->input('per_page', 15), 10), 100);
         $page = (int) $request->input('page', 1);
 
-        $results = collect();
+        $staticResults = collect();   // Arts and Structures (Always at the top)
+        $sortableResults = collect(); // Bylaws, Opinions, Precedents (Sorted if requested)
 
         if (!empty($searchTerm)) {
             $limitPerTable = 100;
 
-            // 1. Search Bylaw Documents
+            // =========================================================
-            $bylaws = BylawDocument::with(['category', 'approvalAuthority', 'promulgationAuthority'])
+            // 1. Search Arts (Exact match to ArtController)
-                ->where('is_published', true)
+            // =========================================================
-                ->where(function ($q) use ($searchTerm) {
-                    $q->where('title', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('description', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('promulgation_number', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('official_gazette_number', 'LIKE', "%{$searchTerm}%")
-                        ->orWhereIn('id', function($subQuery) use ($searchTerm) {
-                            $subQuery->select('document_id')
-                                ->from('bylaw_nodes')
-                                ->where('node_content', 'LIKE', "%{$searchTerm}%");
-                        });
-                })->take($limitPerTable)->get()->map(function($item) {
-                    $formatted = $this->formatBylaw($item);
-                    $formatted['type'] = 'bylaw';
-                    return $formatted;
-                });
-            $results = $results->merge($bylaws);
-
-            // 2. Search Judicial Precedents
-            $judicials = JudicialPrecedent::with('category')
-                ->where(function ($q) use ($searchTerm) {
-                    $q->where('ruling_number', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('subject', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('full_text', 'LIKE', "%{$searchTerm}%");
-                })->take($limitPerTable)->get()->map(function($item) {
-                    $formatted = $this->formatJudicial($item);
-                    $formatted['type'] = 'judicial_precedent';
-                    return $formatted;
-                });
-            $results = $results->merge($judicials);
-
-            // 3. Search Legal Opinions
-            $opinions = LegalOpinion::with('category')
-                ->where(function ($q) use ($searchTerm) {
-                    $q->where('opinion_number', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('subject', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('full_text', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('issuing_authority', 'LIKE', "%{$searchTerm}%");
-                })->take($limitPerTable)->get()->map(function($item) {
-                    $formatted = $this->formatOpinion($item);
-                    $formatted['type'] = 'legal_opinion';
-                    return $formatted;
-                });
-            $results = $results->merge($opinions);
-
-            // 4. Search Arts (DEEP RELATIONAL SEARCH INCLUDED)
             $arts = Art::with(['law.category', 'chapter', 'part', 'volum', 'book', 'section', 'gate'])
                 ->where(function ($q) use ($searchTerm) {
                     $q->where('title', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('text', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('text', 'LIKE', "%{$searchTerm}%");
-                        ->orWhere('number', 'LIKE', "%{$searchTerm}%") // Added Number Search
-                        ->orWhereHas('law', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('volum', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('book', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('section', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('chapter', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('part', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('gate', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); });
                 })
                 ->take($limitPerTable)->get()->map(function($item) use ($searchTerm) {
                     $formatted = $this->formatArt($item, $searchTerm, Art::class);
                     $formatted['type'] = 'art';
                     return $formatted;
                 });
-            $results = $results->merge($arts);
+            $staticResults = $staticResults->concat($arts);
 
-            // 5. Search Laws and Structural Models
+            // =========================================================
+            // 2. Search Laws and Structural Models (Exact match to ArtController)
+            // =========================================================
             $structuralModels = [
                 Law::class, Volum::class, Book::class, Division::class,
                 Section::class, Chapter::class, Part::class, Gate::class,
@@ -135,39 +85,95 @@ class GlobalSearchApiController extends Controller
                     return $formatted;
                 });
 
-                $results = $results->merge($items);
+                $staticResults = $staticResults->concat($items);
-            }
             }
 
-        // --- Apply Dynamic Memory Sorting ---
+            // =========================================================
-        if ($request->has('sort_by')) {
+            // 3. Search Bylaw Documents
+            // =========================================================
+            $bylaws = BylawDocument::with(['category', 'approvalAuthority', 'promulgationAuthority'])
+                ->where('is_published', true)
+                ->where(function ($q) use ($searchTerm) {
+                    $q->where('title', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('description', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('promulgation_number', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('official_gazette_number', 'LIKE', "%{$searchTerm}%")
+                        ->orWhereIn('id', function($subQuery) use ($searchTerm) {
+                            $subQuery->select('document_id')
+                                ->from('bylaw_nodes')
+                                ->where('node_content', 'LIKE', "%{$searchTerm}%");
+                        });
+                })->take($limitPerTable)->get()->map(function($item) {
+                    $formatted = $this->formatBylaw($item);
+                    $formatted['type'] = 'bylaw';
+                    return $formatted;
+                });
+            $sortableResults = $sortableResults->concat($bylaws);
+
+            // =========================================================
+            // 4. Search Judicial Precedents
+            // =========================================================
+            $judicials = JudicialPrecedent::with('category')
+                ->where(function ($q) use ($searchTerm) {
+                    $q->where('ruling_number', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('subject', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('full_text', 'LIKE', "%{$searchTerm}%");
+                })->take($limitPerTable)->get()->map(function($item) {
+                    $formatted = $this->formatJudicial($item);
+                    $formatted['type'] = 'judicial_precedent';
+                    return $formatted;
+                });
+            $sortableResults = $sortableResults->concat($judicials);
+
+            // =========================================================
+            // 5. Search Legal Opinions
+            // =========================================================
+            $opinions = LegalOpinion::with('category')
+                ->where(function ($q) use ($searchTerm) {
+                    $q->where('opinion_number', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('subject', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('full_text', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('issuing_authority', 'LIKE', "%{$searchTerm}%");
+                })->take($limitPerTable)->get()->map(function($item) {
+                    $formatted = $this->formatOpinion($item);
+                    $formatted['type'] = 'legal_opinion';
+                    return $formatted;
+                });
+            $sortableResults = $sortableResults->concat($opinions);
+        }
+
+        // --- Apply Dynamic Memory Sorting ONLY to Legal Docs ---
+        if ($request->has('sort_by') && $request->input('sort_by') != null) {
             $sortBy = $request->input('sort_by');
             $sortOrder = $request->input('sort', 'desc');
 
             if ($sortBy === 'number') {
                 $sortCallback = function ($item) {
-                    return (string) ($item['promulgation_number'] ?? $item['ruling_number'] ?? $item['opinion_number'] ?? $item['number'] ?? '');
+                    return (string) ($item['promulgation_number'] ?? $item['ruling_number'] ?? $item['opinion_number'] ?? '0');
                 };
 
                 if ($sortOrder === 'asc') {
-                    $results = $results->sortBy($sortCallback, SORT_NATURAL)->values();
+                    $sortableResults = $sortableResults->sortBy($sortCallback, SORT_NATURAL)->values();
                 } else {
-                    $results = $results->sortByDesc($sortCallback, SORT_NATURAL)->values();
+                    $sortableResults = $sortableResults->sortByDesc($sortCallback, SORT_NATURAL)->values();
                 }
             } elseif ($sortBy === 'date') {
                 $sortCallback = function ($item) {
-                    return (string) ($item['approval_date'] ?? $item['ruling_date'] ?? $item['opinion_date'] ?? '');
+                    return (string) ($item['approval_date'] ?? $item['ruling_date'] ?? $item['opinion_date'] ?? '1970-01-01');
                 };
 
                 if ($sortOrder === 'asc') {
-                    $results = $results->sortBy($sortCallback)->values();
+                    $sortableResults = $sortableResults->sortBy($sortCallback)->values();
                 } else {
-                    $results = $results->sortByDesc($sortCallback)->values();
+                    $sortableResults = $sortableResults->sortByDesc($sortCallback)->values();
                 }
             }
         }
 
-        // --- Execute Manual Pagination on the Sorted Collection ---
+        // Combine Results: Arts and Structures ALWAYS pin to the top!
+        $results = $staticResults->concat($sortableResults);
+
+        // --- Execute Manual Pagination on the final Collection ---
         $total = $results->count();
         $paginatedItems = $results->forPage($page, $perPage)->values();
 
@@ -187,7 +193,7 @@ class GlobalSearchApiController extends Controller
             'total'        => $paginator->total(),
         ];
 
-        return $this->success($responseData, 'Success', 'Global Search Results Fetched');
+        return $this->success($responseData, '', '');
     }
 
     /**
@@ -203,72 +209,11 @@ class GlobalSearchApiController extends Controller
 
         if (!empty($searchTerm)) {
 
-            // 1. Bylaw Documents
+            // 1. Search Arts FIRST
-            $bylawQuery = BylawDocument::with(['category', 'approvalAuthority', 'promulgationAuthority'])
+            $artQuery = Art::with(['law', 'law.category', 'chapter', 'part', 'volum', 'book', 'section', 'gate'])
-                ->where('is_published', true)
                 ->where(function ($q) use ($searchTerm) {
                     $q->where('title', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('description', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('text', 'LIKE', "%{$searchTerm}%");
-                        ->orWhere('promulgation_number', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('official_gazette_number', 'LIKE', "%{$searchTerm}%")
-                        ->orWhereIn('id', function($subQuery) use ($searchTerm) {
-                            $subQuery->select('document_id')
-                                ->from('bylaw_nodes')
-                                ->where('node_content', 'LIKE', "%{$searchTerm}%");
-                        });
-                });
-            $totalCount += $bylawQuery->count();
-            $bylaws = $bylawQuery->take($limitPerTable)->get()->map(function($item) {
-                $formatted = $this->formatBylaw($item);
-                $formatted['type'] = 'bylaw';
-                return $formatted;
-            });
-            $results = $results->merge($bylaws);
-
-            // 2. Judicial Precedents
-            $judicialQuery = JudicialPrecedent::with('category')
-                ->where(function ($q) use ($searchTerm) {
-                    $q->where('ruling_number', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('subject', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('full_text', 'LIKE', "%{$searchTerm}%");
-                });
-            $totalCount += $judicialQuery->count();
-            $judicials = $judicialQuery->take($limitPerTable)->get()->map(function($item) {
-                $formatted = $this->formatJudicial($item);
-                $formatted['type'] = 'judicial_precedent';
-                return $formatted;
-            });
-            $results = $results->merge($judicials);
-
-            // 3. Legal Opinions
-            $opinionQuery = LegalOpinion::with('category')
-                ->where(function ($q) use ($searchTerm) {
-                    $q->where('opinion_number', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('subject', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('full_text', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('issuing_authority', 'LIKE', "%{$searchTerm}%");
-                });
-            $totalCount += $opinionQuery->count();
-            $opinions = $opinionQuery->take($limitPerTable)->get()->map(function($item) {
-                $formatted = $this->formatOpinion($item);
-                $formatted['type'] = 'legal_opinion';
-                return $formatted;
-            });
-            $results = $results->merge($opinions);
-
-            // 4. Search Arts (DEEP RELATIONAL SEARCH INCLUDED)
-            $artQuery = Art::with(['law.category', 'chapter', 'part', 'volum', 'book', 'section', 'gate'])
-                ->where(function ($q) use ($searchTerm) {
-                    $q->where('title', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('text', 'LIKE', "%{$searchTerm}%")
-                        ->orWhere('number', 'LIKE', "%{$searchTerm}%")
-                        ->orWhereHas('law', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('volum', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('book', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('section', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('chapter', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('part', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); })
-                        ->orWhereHas('gate', function ($sub) use ($searchTerm) { $sub->where('title', 'LIKE', "%{$searchTerm}%"); });
                 });
             $totalCount += $artQuery->count();
             $arts = $artQuery->take($limitPerTable)->get()->map(function($item) use ($searchTerm) {
@@ -276,9 +221,9 @@ class GlobalSearchApiController extends Controller
                 $formatted['type'] = 'art';
                 return $formatted;
             });
-            $results = $results->merge($arts);
+            $results = $results->concat($arts);
 
-            // 5. Structural Models
+            // 2. Structural Models
             $structuralModels = [
                 Law::class, Volum::class, Book::class, Division::class,
                 Section::class, Chapter::class, Part::class, Gate::class,
@@ -300,15 +245,68 @@ class GlobalSearchApiController extends Controller
                         $formatted['type'] = ($table === 'section') ? 'sections' : $table;
                         return $formatted;
                     });
-                    $results = $results->merge($items);
+                    $results = $results->concat($items);
                 }
             }
+
+            // 3. Bylaw Documents
+            $bylawQuery = BylawDocument::with(['category', 'approvalAuthority', 'promulgationAuthority'])
+                ->where('is_published', true)
+                ->where(function ($q) use ($searchTerm) {
+                    $q->where('title', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('description', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('promulgation_number', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('official_gazette_number', 'LIKE', "%{$searchTerm}%")
+                        ->orWhereIn('id', function($subQuery) use ($searchTerm) {
+                            $subQuery->select('document_id')
+                                ->from('bylaw_nodes')
+                                ->where('node_content', 'LIKE', "%{$searchTerm}%");
+                        });
+                });
+            $totalCount += $bylawQuery->count();
+            $bylaws = $bylawQuery->take($limitPerTable)->get()->map(function($item) {
+                $formatted = $this->formatBylaw($item);
+                $formatted['type'] = 'bylaw';
+                return $formatted;
+            });
+            $results = $results->concat($bylaws);
+
+            // 4. Judicial Precedents
+            $judicialQuery = JudicialPrecedent::with('category')
+                ->where(function ($q) use ($searchTerm) {
+                    $q->where('ruling_number', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('subject', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('full_text', 'LIKE', "%{$searchTerm}%");
+                });
+            $totalCount += $judicialQuery->count();
+            $judicials = $judicialQuery->take($limitPerTable)->get()->map(function($item) {
+                $formatted = $this->formatJudicial($item);
+                $formatted['type'] = 'judicial_precedent';
+                return $formatted;
+            });
+            $results = $results->concat($judicials);
+
+            // 5. Legal Opinions
+            $opinionQuery = LegalOpinion::with('category')
+                ->where(function ($q) use ($searchTerm) {
+                    $q->where('opinion_number', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('subject', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('full_text', 'LIKE', "%{$searchTerm}%")
+                        ->orWhere('issuing_authority', 'LIKE', "%{$searchTerm}%");
+                });
+            $totalCount += $opinionQuery->count();
+            $opinions = $opinionQuery->take($limitPerTable)->get()->map(function($item) {
+                $formatted = $this->formatOpinion($item);
+                $formatted['type'] = 'legal_opinion';
+                return $formatted;
+            });
+            $results = $results->concat($opinions);
         }
 
         return $this->success([
             'items' => $results->take(15)->values(),
             'count' => $totalCount
-        ], 'Success', 'Global Fast Search Results');
+        ], '', '');
     }
 
     /* -----------------------------------------------------------------
@@ -381,86 +379,44 @@ class GlobalSearchApiController extends Controller
         ];
     }
 
-    private function formatArt($art, $searchTerm, $modelClass)
+    /**
+     * Exact 1:1 match with ArtController::searchModel mapping
+     * All relations (law, chapter, etc) are strictly Strings to prevent App Crashes!
+     */
+    private function formatArt($item, $searchTerm, $modelClass)
     {
-        $law = $art->law ?? Law::find($art->law_id);
+        $law = $item->law ?? Law::find($item->law_id);
-        $isLocked = ! auth()->user()->isSubscriber() && (bool) $law?->is_locked;
+        $isLocked = ! auth()->user()->isSubscriber() && (bool) ($law?->is_locked ?? false);
 
-        $text = $art->text ?? '';
+        $text = $item->text ?? '';
-
+        $searchQuoted = preg_quote($searchTerm, '/');
-        // Multi-byte safe Context Snippet Extraction for Persian/Arabic
+        if (!empty($searchTerm) && preg_match('/\b(.{0,50})\b('.$searchQuoted.')\b(.{0,50})\b/i', $text, $matches)) {
-        if (!empty($searchTerm) && !empty($text)) {
+            $context = trim($matches[1] ?? '').' '.($matches[2] ?? '').' '.trim($matches[3] ?? '');
-            $pos = mb_stripos($text, $searchTerm);
-            if ($pos !== false) {
-                $start = max(0, $pos - 50);
-                $length = mb_strlen($searchTerm) + 100;
-                $context = mb_substr($text, $start, $length);
-                if ($start > 0) $context = '...' . $context;
-                if ($start + $length < mb_strlen($text)) $context = $context . '...';
-            } else {
-                $context = $text; // Fallback if match was in title/relations
-            }
         } else {
             $context = $text;
         }
 
         return [
-            'id'          => $art->id,
+            'id'           => $item->id,
-            'title'       => $art->title,
+            'title'        => $item->title,
-            'number'      => $art->number,
+            'number'       => $item->number,
             'text'         => $isLocked ? null : $context,
             'is_locked'    => $isLocked,
-            'route'       => array_values($this->route($modelClass, $art)),
+            'route'        => array_values($this->route($modelClass, $item)),
             'category'     => optional($law?->category)->name,
-
+            'law'          => $law?->title ?? '', // STRICLY STRING! Prevents Android App Crash!
             'count_art'    => $law?->arts?->count() ?? 0,
-            'count_volums'=> $law?->volums?->count() ?? 0,
+            'count_volums' => $law?->volums?->count() ?? 0, // WITH S
+            'count_volum'  => $law?->volums?->count() ?? 0, // WITHOUT S (Fallback)
             'price'        => $law?->price,
             'image'        => $law?->image,
-
-            // Relational Objects
-            'law' => $art->law != null ? [
-                'id' => $art->law->id,
-                'title' => $art->law->title,
-                'number' => $art->law->number,
-            ] : null,
-            'chapter' => $art->chapter != null ? [
-                'id' => $art->chapter->id,
-                'title' => $art->chapter->title,
-                'number' => $art->chapter->number,
-            ] : null,
-            'part' => $art->part != null ? [
-                'id' => $art->part->id,
-                'title' => $art->part->title,
-                'number' => $art->part->number,
-            ] : null,
-            'volum' => $art->volum != null ? [
-                'id' => $art->volum->id,
-                'title' => $art->volum->title,
-                'number' => $art->volum->number,
-            ] : null,
-            'book' => $art->book != null ? [
-                'id' => $art->book->id,
-                'title' => $art->book->title,
-                'number' => $art->book->number,
-            ] : null,
-            'section' => $art->section != null ? [
-                'id' => $art->section->id,
-                'title' => $art->section->title,
-                'number' => $art->section->number,
-            ] : null,
-            'gate' => $art->gate != null ? [
-                'id' => $art->gate->id,
-                'title' => $art->gate->title,
-                'number' => $art->gate->number,
-            ] : null,
         ];
     }
 
     private function formatStructure($item, $modelClass)
     {
         $law = Law::find($item?->law_id ?? $item->id);
-        $isLocked = ! auth()->user()->isSubscriber() && (bool) $law?->is_locked;
+        $isLocked = ! auth()->user()->isSubscriber() && (bool) ($law?->is_locked ?? false);
 
         $payload = [
             'id'        => $item->id,
@@ -468,11 +424,7 @@ class GlobalSearchApiController extends Controller
             'number'    => $item->number,
             'is_locked' => $isLocked,
             'route'     => array_values($this->route($modelClass, $item)),
-            'law'       => $law ? [
+            'law'       => $law?->title ?? '', // STRICTLY STRING
-                'id' => $law->id,
-                'title' => $law->title,
-                'number' => $law->number,
-            ] : null,
         ];
 
         if ((new $modelClass)->getTable() == 'laws') {