minor symfony#75 Add the app_ prefix to form field types to avoid collisions (mantulo)

javiereguiluz · javiereguiluz · commit 2accbbad5c2b · 2015-06-30T15:53:13.000+02:00
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
diff --git a/src/AppBundle/Form/CommentType.php b/src/AppBundle/Form/CommentType.php
@@ -52,6 +52,6 @@ public function configureOptions(OptionsResolver $resolver)
      */
     public function getName()
     {
-        return 'comment';
+        return 'app_comment';
     }
 }
diff --git a/src/AppBundle/Form/PostType.php b/src/AppBundle/Form/PostType.php
@@ -59,6 +59,6 @@ public function configureOptions(OptionsResolver $resolver)
      */
     public function getName()
     {
-        return 'post';
+        return 'app_post';
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -52,6 +52,6 @@ public function configureOptions(OptionsResolver $resolver)`
`52`	`52`	`*/`
`53`	`53`	`public function getName()`
`54`	`54`	`{`
`55`		`- return 'comment';`
	`55`	`+ return 'app_comment';`
`56`	`56`	`}`
`57`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -59,6 +59,6 @@ public function configureOptions(OptionsResolver $resolver)`
`59`	`59`	`*/`
`60`	`60`	`public function getName()`
`61`	`61`	`{`
`62`		`- return 'post';`
	`62`	`+ return 'app_post';`
`63`	`63`	`}`
`64`	`64`	`}`