8000 Add cmf_find_translation twig function documentation by EmmanuelVella · Pull Request #769 · symfony-cmf/symfony-cmf-docs · GitHub
[go: up one dir, main page]

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

Conversation

@EmmanuelVella
Copy link
Contributor


* **cmf_find|find(path)**: Get the document for the provided path or null if not found.
No publish workflow check is done.
* **cmf_find_translation|findTranslation(($path, $locale, $fallback = true)**: Get the document translation
Copy link
Member

Choose a reason for hiding this comment

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

duplicated opening (
and can we call the paramter $pathOrDocument to be more explicit?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed !

@EmmanuelVella EmmanuelVella force-pushed the cmf_find_translation branch from ed74246 to e4d058f Compare July 21, 2016 14:19
@dbu
Copy link
Member
dbu commented Jul 21, 2016

sorry, one last thing: we should have a versionadded block after the list of twig functions that says in which version we added this function. otherwise users of an older version will be confused. (for the exact syntax, look for ..versionadded: in the doc sources, we use that in several places)

@dbu
Copy link
Member
dbu commented Jul 21, 2016

version would be 1.3.1 of the core bundle

@EmmanuelVella EmmanuelVella force-pushed the cmf_find_translation branch from e4d058f to 5598a40 Compare July 21, 2016 15:12
@EmmanuelVella
Copy link
Contributor Author

No problem, is it ok like this ?

@dbu
Copy link
Member
dbu commented Jul 21, 2016

sorry, no the doc should be in the table with the others. the versionadded just explains when this particular function has been added. like "The cmf_find_translation twig function has been added in version 1.3.1 of the CMF CoreBundle."

@EmmanuelVella EmmanuelVella force-pushed the cmf_find_translation branch from 5598a40 to 8ea2576 Compare July 21, 2016 16:08
@EmmanuelVella
Copy link
Contributor Author

Ho sorry, I just fixed it, hope it's ok now !

@dbu dbu merged commit 671e610 into symfony-cmf:1.3 Jul 22, 2016
@dbu
Copy link
Member
dbu commented Jul 22, 2016

yep, that 7270 was the idea. thank you!

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.

2 participants

0