Skip to content
This repository has been archived by the owner on Jan 9, 2019. It is now read-only.

PHP7 Fatal Errors fixed #33 #34

Open
wants to merge 7 commits into
base: 2.0
Choose a base branch
from
Open

PHP7 Fatal Errors fixed #33 #34

wants to merge 7 commits into from

Conversation

amenk
Copy link

@amenk amenk commented Feb 8, 2016

  • Use Contracts in Parameters
  • added the "use" statement to a closure

 * Use Contracts in Parameters
 * added the "use" statement to a closure
@@ -42,7 +43,7 @@ public function findHidden($id)
public function findByAttribute(array $attribute)
{
$collection = [];
$all = Album::where(function($query) {
$all = Album::where(function($query) use ($attribute) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Expected 1 space after FUNCTION keyword; 0 found

amenk added 3 commits February 8, 2016 18:49
I think we should refer directly in the module, not using a global facade (which is actually not defined)
@@ -11,7 +13,7 @@ class GalleryController extends Controller
*/
public function index()
{
$allAlbums = \Gallery::album()->all();
$allAlbums = Gallery::album()->all();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spaces must be used to indent lines; tabs are not allowed

I think we should refer directly in the module, not using a global facade (which is actually not defined)
@@ -14,7 +15,7 @@ class PhotosController extends Controller
*/
public function create()
{
$albumArray = \Gallery::album()->all()->toArray();
$albumArray = Gallery::album()->all()->toArray();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spaces must be used to indent lines; tabs are not allowed

@amenk
Copy link
Author

amenk commented Feb 8, 2016

@Nitpick-CI you are very annoying :-)

@amenk amenk mentioned this pull request Feb 8, 2016
I think we should refer directly in the module, not using a global facade (which is actually not defined)
@@ -38,7 +39,7 @@ public function store(Request $request)
'order' => 0,
]);

\Gallery::album()->add($album);
Gallery::album()->add($album);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spaces must be used to indent lines; tabs are not allowed

@@ -65,7 +66,7 @@ public function store(Request $request)
*/
public function show($albumId, $photoId)
{
$photo = \Gallery::photo()->find($photoId);
$photo = Gallery::photo()->find($photoId);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spaces must be used to indent lines; tabs are not allowed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants