Skip to content

Add tree to content-mapper #23

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.

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

Merged
merged 27 commits into from
Dec 16, 2013
Merged

Add tree to content-mapper #23

merged 27 commits into from
Dec 16, 2013

Conversation

wachterjohannes
Copy link
Member

Add tree to content-mapper:

  • test coverage
  • finish the code
  • gather feedback for my changes
  • submit changes to the documentation
Q A
Bug fix? no
New feature? yes
Tests pass? yes
Fixed tickets none
Doc none

@wachterjohannes
Copy link
Member Author

@danrot @chirimoya merge after this #20

@wachterjohannes
Copy link
Member Author

@danrot added a few functions to this branch to generate url form controller!

/**
* @return boolean
*/
public function getHasChildren();
Copy link
Contributor

Choose a reason for hiding this comment

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

@wachterjohannes Doesn't hasChildrensound better to you?

Copy link
Member

Choose a reason for hiding this comment

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

+1

Copy link
Member Author

Choose a reason for hiding this comment

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

@danrot @chirimoya this is for twig! when designer types {{ structure.hasChildren }} -> structure->getHasChildren will be called!

@danrot
Copy link
Contributor

danrot commented Dec 16, 2013

@wachterjohannes The url generation does not work correctly for Ä, Ö, Ü at the beginning, please fix that

@wachterjohannes
Copy link
Member Author

@danrot strtolower ignores Ä,Ö,Ü ! i have added special rules for that!

@danrot danrot merged commit 1759bb1 into develop Dec 16, 2013
@danrot danrot deleted the feature/content-tree branch December 16, 2013 14:36
wachterjohannes added a commit that referenced this pull request Aug 14, 2014
wachterjohannes added a commit that referenced this pull request Aug 14, 2014
wachterjohannes added a commit that referenced this pull request Aug 14, 2014
chirimoya added a commit that referenced this pull request Aug 14, 2014
mmsbrggr pushed a commit that referenced this pull request Aug 14, 2014
[FEATURE] generate right url for internal and external nodes
wachterjohannes added a commit that referenced this pull request Sep 30, 2014
wachterjohannes added a commit that referenced this pull request Oct 31, 2014
wachterjohannes added a commit that referenced this pull request Oct 31, 2014
wachterjohannes added a commit that referenced this pull request Oct 31, 2014
wachterjohannes added a commit that referenced this pull request Oct 31, 2014
chirimoya added a commit that referenced this pull request Oct 31, 2014
mmsbrggr pushed a commit that referenced this pull request Oct 31, 2014
[FEATURE] generate right url for internal and external nodes
wachterjohannes pushed a commit to wachterjohannes/sulu that referenced this pull request May 8, 2017
GlennVL91 pushed a commit to Tactics/sulu that referenced this pull request Jun 20, 2019
alexander-schranz added a commit to alexander-schranz/sulu that referenced this pull request Feb 22, 2024
* Update routing and twig recipes

* Upgrade doctrine bundle configuration

* Update symfony/routing config

* Fixerror routes

* Upgrade debug and security dependency

* Add AppFixtures from DoctrineFixtureBundle recipe
alexander-schranz pushed a commit to alexander-schranz/sulu that referenced this pull request Nov 25, 2024
…ception

Refactor model-not-found exception
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants