Refactor SchoolController manage for master

changeweb · Jul 24, 2019 · c2a5d14 · c2a5d14
1 parent 1ed36c4
commit c2a5d14
Show file tree

Hide file tree

Showing 15 changed files with 509 additions and 306 deletions.
diff --git a/app/Http/Controllers/SchoolController.php b/app/Http/Controllers/SchoolController.php
@@ -2,13 +2,13 @@
 
 namespace App\Http\Controllers;
 
+use App\User;
 use App\School;
 use App\Myclass;
 use App\Section;
-use App\User;
 use App\Department;
-//use App\Http\Resources\SchoolResource;
 use Illuminate\Http\Request;
+use App\Http\Requests\SchoolRequest;
 
 class SchoolController extends Controller
 {
@@ -17,28 +17,19 @@ class SchoolController extends Controller
      *
      * @return \Illuminate\Http\Response
      */
-    public function index()
-    {
-      $schools = School::all();
-      $classes = Myclass::all();
-      $sections = Section::all();
-      $teachers = User::join('departments', 'departments.id', '=', 'users.department_id')
-                            ->where('role', 'teacher')
-                            ->orderBy('name','ASC')
-                            ->where('active', 1)
-                            ->get();
-      $departments = Department::bySchool(\Auth::user()->school_id)->get();
-      return view('school.create-school', compact('schools', 'classes', 'sections', 'teachers', 'departments'));
-    }
+    public function index() {
+      $schools = School::orderBy('created_at', 'desc')->paginate();
 
-    /**
-     * Show the form for creating a new resource.
-     *
-     * @return \Illuminate\Http\Response
-     */
-    public function create()
-    {
-        //
+      return view('schools.index', compact('schools'));
+      //$classes = Myclass::all();
+      //$sections = Section::all();
+      //$teachers = User::join('departments', 'departments.id', '=', 'users.department_id')
+                            //->where('role', 'teacher')
+                            //->orderBy('name','ASC')
+                            //->where('active', 1)
+                            //->get();
+      //$departments = Department::bySchool(\Auth::user()->school_id)->get();
+      //return view('school.create-school', compact('schools', 'classes', 'sections', 'teachers', 'departments'));
     }
 
     /**
@@ -47,23 +38,18 @@ public function create()
      * @param  \Illuminate\Http\Request  $request
      * @return \Illuminate\Http\Response
      */
-    public function store(Request $request)
+    public function store(SchoolRequest $request)
     {
-      $request->validate([
-        'school_name' => 'required|string|max:255',
-        'school_medium' => 'required',
-        'school_about' => 'required',
-        'school_established' => 'required',
-      ]);
-      $tb = new School;
-      $tb->name = $request->school_name;
-      $tb->established = $request->school_established;
-      $tb->about = $request->school_about;
-      $tb->medium = $request->school_medium;
-      $tb->code = date("y").substr(number_format(time() * mt_rand(),0,'',''),0,6);
-      $tb->theme = 'flatly';
-      $tb->save();
-      return back()->with('status', 'Created');
+        School::create([
+            'name'        => $request->name,
+            'established' => $request->established,
+            'about'       => $request->about,
+            'medium'      => $request->medium,
+            'code'        => date("y").substr(number_format(time() * mt_rand(), 0, '', ''), 0, 6),
+            'theme'       => 'flatly'
+        ]);
+
+        return redirect()->route('schools.index');
     }
 
     /**
@@ -78,25 +64,16 @@ public function show($school_id)
       return view('school.admin-list',compact('admins'));
     }
 
-    /**
-     * Show the form for editing the specified resource.
-     *
-     * @param  int  $id
-     * @return \Illuminate\Http\Response
-     */
-    public function edit(Request $request, $school_id)
-    {
-      $school = School::find($school_id);
-      if (!$school) {
-        abort(404);
-      }
-      if ($request->isMethod('post')) {
+    public function edit(School $school) {
+        return view('schools.edit', compact('school'));
+    }
+
+    public function update(Request $request, School $school) {
         $school->name = $request->name;
         $school->about = $request->about;
-        $school->theme = $request->school_theme;
         $school->save();
-      }
-      return view('school.edit-school',compact('school'));
+
+        return redirect()->route('schools.index');
     }
 
     public function addDepartment(Request $request){
@@ -116,25 +93,6 @@ public function changeTheme(Request $request){
       $tb->save();
       return back();
     }
-    /**
-     * Update the specified resource in storage.
-     *
-     * @param  \Illuminate\Http\Request  $request
-     * @param  int  $id
-     * @return \Illuminate\Http\Response
-     */
-    public function update(Request $request, $id)
-    {
-      $tb = School::find($id);
-      $tb->name = $request->name;
-      $tb->about = $request->about;
-      //$tb->code = $request->code;
-      return ($tb->save())?response()->json([
-        'status' => 'success'
-      ]):response()->json([
-        'status' => 'error'
-      ]);
-    }
 
     /**
      * Remove the specified resource from storage.

diff --git a/app/Http/Requests/SchoolRequest.php b/app/Http/Requests/SchoolRequest.php
@@ -0,0 +1,33 @@
+<?php
+
+namespace App\Http\Requests;
+
+use Illuminate\Foundation\Http\FormRequest;
+
+class SchoolRequest extends FormRequest
+{
+    /**
+     * Determine if the user is authorized to make this request.
+     *
+     * @return bool
+     */
+    public function authorize()
+    {
+        return true;
+    }
+
+    /**
+     * Get the validation rules that apply to the request.
+     *
+     * @return array
+     */
+    public function rules()
+    {
+        return [
+            'name'        => 'required|string|max:255',
+            'medium'      => 'required',
+            'about'       => 'required',
+            'established' => 'required',
+        ];
+    }
+}
diff --git a/database/factories/SchoolFactory.php b/database/factories/SchoolFactory.php
@@ -5,10 +5,11 @@
 
 $factory->define(School::class, function (Faker $faker) {
     return [
-        'name'   => $faker->name,
-        'about'  => $faker->sentences(3, true),
-        'medium' => $faker->randomElement(['bangla', 'english']),
-        'code'   => date("y").substr(number_format(time() * mt_rand(),0,'',''),0,6),
-        'theme'  => 'flatly',
+        'name'        => $faker->name,
+        'about'       => $faker->sentences(3, true),
+        'medium'      => $faker->randomElement(['bangla', 'english']),
+        'code'        => date("y").substr(number_format(time() * mt_rand(),0,'',''),0,6),
+        'established' => $faker->name,
+        'theme'       => 'flatly',
     ];
 });
diff --git a/resources/views/layouts/master/create-school-form.blade.php b/resources/views/layouts/master/create-school-form.blade.php
diff --git a/resources/views/masters/index.blade.php b/resources/views/masters/index.blade.php
@@ -8,7 +8,9 @@
                 <div class="page-panel-title">@lang('Dashboard')</div>
 
                 <div class="panel-body">
-                    <a class="btn btn-danger btn-lg btn-block" href="{{url('create-school')}}" role="button">@lang('Manage Schools')</a>
+                    <a class="btn btn-danger btn-lg btn-block" href="{{ route('schools.index') }}" role="button">
+                        @lang('Manage Schools')
+                    </a>
                 </div>
             </div>
         </div>

diff --git a/resources/views/school/edit-school.blade.php b/resources/views/school/edit-school.blade.php