Skip to content

Commit

Permalink
minor #75 Add the app_ prefix to form field types to avoid collisions…
Browse files Browse the repository at this point in the history
… (mantulo)

This PR was merged into the master branch.

Discussion
----------

Add the app_ prefix to  form field types to avoid collisions

Best Practice
http://symfony.com/doc/current/best_practices/forms.html#custom-form-field-types

Commits
-------

627844f Add the app_ prefix to  form field types to avoid collisions
  • Loading branch information
javiereguiluz committed Jun 30, 2015
2 parents deaceb9 + 627844f commit 2accbba
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/AppBundle/Form/CommentType.php
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,6 @@ public function configureOptions(OptionsResolver $resolver)
*/
public function getName()
{
return 'comment';
return 'app_comment';
}
}
2 changes: 1 addition & 1 deletion src/AppBundle/Form/PostType.php
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,6 @@ public function configureOptions(OptionsResolver $resolver)
*/
public function getName()
{
return 'post';
return 'app_post';
}
}

0 comments on commit 2accbba

Please sign in to comment.