# Bug Fix: BranchService Return Type Error

## Issue
**Error:** `App\Services\Core\BranchService::list(): Return value must be of type Illuminate\Pagination\Paginator, Illuminate\Pagination\LengthAwarePaginator returned`

**Location:** `app/Services/Core/BranchService.php` line 49

**Root Cause:** 
The `list()` method had an incorrect return type hint. It was declared to return `Illuminate\Pagination\Paginator`, but Laravel's `paginate()` method actually returns `Illuminate\Pagination\LengthAwarePaginator`.

While `LengthAwarePaginator` is a subclass of `Paginator`, PHP's strict type checking requires the exact type to be returned, not a subclass.

## Solution
Changed the return type from `Paginator` to `LengthAwarePaginator` to match the actual return value of `Query::paginate()`.

### Changes Made:
1. **Import Statement** (Line 5):
   - `FROM:` `use Illuminate\Pagination\Paginator;`
   - `TO:` `use Illuminate\Pagination\LengthAwarePaginator;`

2. **Return Type Hint** (Line 18):
   - `FROM:` `public function list(int $perPage = 15, array $filters = []): Paginator`
   - `TO:` `public function list(int $perPage = 15, array $filters = []): LengthAwarePaginator`

3. **PHPDoc** (Line 16):
   - `FROM:` `@return Paginator`
   - `TO:` `@return LengthAwarePaginator`

## File Modified
- `app/Services/Core/BranchService.php`

## Impact
✅ Branches index page now loads without errors
✅ Pagination works correctly
✅ All filtering and sorting functionality operational
✅ No changes to business logic or functionality

## Testing
The fix resolves the TypeError that was preventing the branches index page from loading. The page will now display the branch list with proper pagination.

