Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Sniff to prevent if/loops within any controller methods? #73

Open
bmitch opened this issue Apr 5, 2017 · 0 comments
Open

Sniff to prevent if/loops within any controller methods? #73

bmitch opened this issue Apr 5, 2017 · 0 comments
Labels
New Sniff

Comments

@bmitch
Copy link
Owner

bmitch commented Apr 5, 2017 •
edited
Loading

Not sure if this is a good idea or not but I guess users can always disable if they don't want to use it.

If the file ends in Controller.php then no methods within should contain any logic. Logic should be delegated to other classes.

  • If file ends in controller.
  • If file contains if, else, or any loop keyword it fails.
bmitch added a commit that referenced this issue Apr 10, 2017
@bmitch
WIP #73
d2353a2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
New Sniff
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bmitch