Edit Poll

src/Http/Controllers/PollManagerController.php

@@ -66,10 +66,14 @@ public function edit(Poll $poll)
      *
      * @param Poll $poll
      * @param Request $request
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function update(Poll $poll, Request $request)
     {
         PollHandler::modify($poll, $request->all());
+
+        return redirect(route('poll.index'))
+            ->with('success', 'Your poll has been updated successfully');
     }
 
     /**

src/Views/dashboard/edit.blade.php

@@ -1,6 +1,55 @@
-<div class="container">
-    {{ $poll->question }}
-    @foreach($poll->options as $option)
-        <h3>{{ $option->name }}</h3>
-    @endforeach
-</div>
+@extends('larapoll::layouts.app')
+@section('title')
+    Polls- Edit
+@endsection
+@section('content')
+    <div class="container">
+        <ol class="breadcrumb">
+            <li><a href="#">Home</a></li>
+            <li><a href="#">Polls</a></li>
+            <li class="active">Edit Poll</li>
+        </ol>
+        <div class="well col-md-8 col-md-offset-2">
+            @if($errors->any())
+                <ul class="alert alert-danger">
+                    @foreach($errors->all() as $error)
+                        <li>{{ $error }}</li>
+                    @endforeach
+                </ul>
+            @endif
+            <form method="POST" action=" {{ route('poll.update', $poll->id) }}">
+                {{ csrf_field() }}
+                <!-- Question Input -->
+                <div class="form-group">
+                    <label>{{ $poll->question }}</label>
+                </div>
+                <ul class="options">
+                    @foreach($poll->options as $option)
+                        <li>{{ $option->name }}</li>
+                    @endforeach
+                </ul>
+
+                @php
+                    $maxCheck = $poll->maxCheck;
+                    $count_options = $poll->optionsNumber()
+                @endphp
+                <select name="count_check" class="form-control">
+                    @for($i =1; $i<= $count_options; $i++)
+                        <option  {{ $i==$maxCheck? 'selected':'' }} >{{ $i }}</option>
+                    @endfor
+                </select>
+
+                <div class="radio">
+                    <label>
+                        <input type="checkbox" name="close" {{ $poll->isLocked()? 'checked':'' }}> Close
+                    </label>
+                </div>
+
+                <!-- Create Form Submit -->
+                <div class="form-group">
+                    <input name="update" type="submit" value="Update" class="btn btn-primary form-control"/>
+                </div>
+            </form>
+        </div>
+    </div>
+@endsection

src/helpers/PollHandler.php

@@ -35,11 +35,13 @@ public static function modify(Poll $poll, $data)
         }
 
         if(array_key_exists('close', $data)){
-            if($data['close']){
+            if(isset($data['close']) && $data['close']){
                 $poll->lock();
             }else{
                 $poll->unLock();
             }
+        }else{
+            $poll->unLock();
         }
     }
 }

tests/ManipulatesPollTest.php

@@ -10,12 +10,6 @@ class ManipulatesPollTest extends \TestCase
 {
     use DatabaseTransactions;
 
-    /** @test */
-    public function admin_can_see_poll_list()
-    {
-        $this->assertTrue(true);
-    }
-
     /** @test */
     public function admin_can_add_new_options()
     {

tests/PollDashboardTest.php

@@ -191,7 +191,7 @@ public function an_admin_can_modify_poll_type()
             'count_check' => 2,
         ];
         $this->post(route('poll.update', $poll->id), $options)
-            ->assertResponseStatus(200)
+            ->assertResponseStatus(302)
             ->assertEquals(2, Poll::findOrFail($poll->id)->maxCheck);
     }
 
@@ -213,7 +213,7 @@ public function an_admin_can_close_a_poll()
         ];
 
         $this->post(route('poll.update', $poll->id), $options)
-            ->assertResponseStatus(200)
+            ->assertResponseStatus(302)
             ->assertTrue(Poll::findOrFail($poll->id)->isLocked());
     }
 
@@ -232,11 +232,10 @@ public function an_admin_can_reopen_a_closed_poll()
 
         $this->assertTrue($poll->lock());
         $options = [
-            'close' => 0,
         ];
 
         $this->post(route('poll.update', $poll->id), $options)
-            ->assertResponseStatus(200)
+            ->assertResponseStatus(302)
             ->assertFalse(Poll::findOrFail($poll->id)->isLocked());
     }