Skip to content
This repository was archived by the owner on Sep 16, 2021. It is now read-only.

Various fixes for tutorial #564

Closed
wants to merge 1 commit into from
Closed

Conversation

seydu
Copy link

@seydu seydu commented Sep 5, 2014

Symfony\Cmf\Bundle\RoutingBundle\Doctrine\Phpcr\Route::setRouteContent() not defined, use setContent() instead
For Symfony\Cmf\Bundle\ContentBundle\Doctrine\Phpcr\StaticContent, title and body are not nullable

showing, however it has no children. You need to map the children on the
``Site`` document, modify it as follows:

.. code-block:: php
Copy link
Member

Choose a reason for hiding this comment

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

i think php is the default type of blocks, so you should just do ... modify it as follows:: and then start the php code below. no need for the opening <?php

@dbu
Copy link
Member

dbu commented Sep 8, 2014

seems good to me. @dantleech can you please have a look?

@dantleech
Copy link
Member

This PR has the same commit as http://symfony.com/doc/master/cmf/index.html#tutorial, except that I updated the other PR yesterday evening.

Also the description of the comment does not seem to relate to the tutorial (i.e. we do not use StaticContent). So closing this - @seydu maybe you could open an issue or make a new PR? Thanks.

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.

4 participants