Merge pull request #157 from symfony-cmf/fix-sonata

remove bogus by_reference => false from children form

Author: lsmith77

Admin/AbstractMenuNodeAdmin.php

@@ -40,23 +40,14 @@ protected function configureListFields(ListMapper $listMapper)
             ;
     }
 
-    protected function isSubjectNotNew()
-    {
-        return $this->hasSubject() && null !== $this->getSubject()->getId();
-    }
-
     /**
      * {@inheritDoc}
      */
     protected function configureFormFields(FormMapper $formMapper)
     {
         $formMapper
             ->with('form.group_general')
-                ->add('name', 'text',
-                    $this->isSubjectNotNew() ? array(
-                        'attr' => array('readonly' => 'readonly')
-                    ) : array()
-                )
+                ->add('name', 'text')
                 ->add('label', 'text')
             ->end()
         ;

Admin/Extension/MenuNodeReferrersExtension.php

@@ -31,9 +31,7 @@ public function configureFormFields(FormMapper $formMapper)
             ->add(
                 'menuNodes',
                 'sonata_type_collection',
-                array(
-                    'by_reference' => false,
-                ),
+                array(),
                 array(
                     'edit' => 'inline',
                     'inline' => 'table',

CHANGELOG.md

@@ -1,6 +1,22 @@
 Changelog
 =========
 
+1.0.1
+-----
+
+* **2013-10-29**: Fix MenuNodeReferrersExtension to also work when adding an
+  additional menu item if there already exists one. Allow to edit the menu item
+  name even after creation as in all other admins - this prevented adding
+  another menu item in the menu tab created by the admin extension. If you want
+  the old behaviour back, extend the admin and change the definition of the
+  `name` field, adding back what is removed in the pull request
+  https://github.com/symfony-cmf/MenuBundle/pull/157
+
+1.0.0
+-----
+
+First stable release.
+
 1.0.0-RC4
 ---------