@@ -50,7 +50,7 @@ |
||
| 50 | 50 | * |
| 51 | 51 | * @param \Illuminate\Http\Request $request |
| 52 | 52 | * |
| 53 | - * @return \Illuminate\Http\Response |
|
| 53 | + * @return \Illuminate\Http\RedirectResponse |
|
| 54 | 54 | */ |
| 55 | 55 | public function register(Request $request, Users $users) |
| 56 | 56 | { |
@@ -16,7 +16,7 @@ |
||
| 16 | 16 | /** |
| 17 | 17 | * Custom authorization method. Use only if you are managing a model with "user_id" field |
| 18 | 18 | * |
| 19 | - * @param $ability |
|
| 19 | + * @param string $ability |
|
| 20 | 20 | * @param array $record |
| 21 | 21 | */ |
| 22 | 22 | public function bCAuthorize($ability, $record = []) |
@@ -27,7 +27,7 @@ |
||
| 27 | 27 | * Get the notification's delivery channels. |
| 28 | 28 | * |
| 29 | 29 | * @param mixed $notifiable |
| 30 | - * @return array |
|
| 30 | + * @return string[] |
|
| 31 | 31 | */ |
| 32 | 32 | public function via($notifiable) |
| 33 | 33 | { |
@@ -96,6 +96,9 @@ |
||
| 96 | 96 | return $this->model->findOrCreate($data); |
| 97 | 97 | } |
| 98 | 98 | |
| 99 | + /** |
|
| 100 | + * @param string $attribute |
|
| 101 | + */ |
|
| 99 | 102 | public function findBy($attribute, $value, $columns = array('*')) |
| 100 | 103 | { |
| 101 | 104 | $this->applyCriteria(); |
@@ -107,8 +107,9 @@ discard block |
||
| 107 | 107 | //$model = $this->app->make($this->model()); |
| 108 | 108 | $model = app()->make($this->model()); |
| 109 | 109 | |
| 110 | - if (!$model instanceof Model) |
|
| 111 | - throw new RepositoryException("Class {$this->model()} must be an instance of Illuminate\\Database\\Eloquent\\Model"); |
|
| 110 | + if (!$model instanceof Model) { |
|
| 111 | + throw new RepositoryException("Class {$this->model()} must be an instance of Illuminate\\Database\\Eloquent\\Model"); |
|
| 112 | + } |
|
| 112 | 113 | |
| 113 | 114 | return $this->model = $model; |
| 114 | 115 | } |
@@ -143,12 +144,14 @@ discard block |
||
| 143 | 144 | } |
| 144 | 145 | |
| 145 | 146 | public function applyCriteria() { |
| 146 | - if($this->skipCriteria === true) |
|
| 147 | - return $this; |
|
| 147 | + if($this->skipCriteria === true) { |
|
| 148 | + return $this; |
|
| 149 | + } |
|
| 148 | 150 | |
| 149 | 151 | foreach($this->getCriteria() as $criteria) { |
| 150 | - if($criteria instanceof Criteria) |
|
| 151 | - $this->model = $criteria->apply($this->model, $this); |
|
| 152 | + if($criteria instanceof Criteria) { |
|
| 153 | + $this->model = $criteria->apply($this->model, $this); |
|
| 154 | + } |
|
| 152 | 155 | } |
| 153 | 156 | |
| 154 | 157 | return $this; |
@@ -26,6 +26,9 @@ |
||
| 26 | 26 | return $this->model->withDepth()->find($id, $columns); |
| 27 | 27 | } |
| 28 | 28 | |
| 29 | + /** |
|
| 30 | + * @param string $name |
|
| 31 | + */ |
|
| 29 | 32 | public function tree($name) |
| 30 | 33 | { |
| 31 | 34 | $root = $this->findBy('name', $name); |
@@ -97,6 +97,9 @@ |
||
| 97 | 97 | } |
| 98 | 98 | } |
| 99 | 99 | |
| 100 | + /** |
|
| 101 | + * @param string $name |
|
| 102 | + */ |
|
| 100 | 103 | public function getUniqueUsername($name) |
| 101 | 104 | { |
| 102 | 105 | $nrRand = rand(0,100); |