Skip to content

Commit f55c9d0

Browse files
committed
Fix example var names testing for defined common includes
Signed-off-by: Matt Friedman <[email protected]>
1 parent bcfd898 commit f55c9d0

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

development/extensions/tutorial_key_concepts.rst

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -297,21 +297,21 @@ is already defined, and if not, then include your script and define the script.
297297

298298
.. code-block:: twig
299299
300-
{% if not $INCLUDED_JQUERYUIJS %}
300+
{% if not definition.INCLUDED_JQUERYUIJS %}
301301
{% INCLUDEJS '@vendor_extname/jquery-ui.js' %}
302-
{% DEFINE $INCLUDED_JQUERYUIJS = true %}
302+
{% DEFINE INCLUDED_JQUERYUIJS = true %}
303303
{% endif %}
304304
305305
Some example template variable definitions to use with common libraries (the common practice should be to name
306306
the variable definition after the library filename, e.g. highslide.js becomes HIGHSLIDEJS):
307307

308-
* HighSlide JS: ``$INCLUDED_HIGHSLIDEJS``
309-
* Font Awesome CSS: ``$INCLUDED_FONTAWESOMECSS``
310-
* ColorBox JS: ``$INCLUDED_COLORBOXJS``
311-
* ColPick JS: ``$INCLUDED_COLPICKJS``
312-
* MoTools JS: ``$INCLUDED_MOTOOLSJS``
313-
* Dojo JS: ``$INCLUDED_DOJOJS``
314-
* Angular JS: ``$INCLUDED_ANGULARJS``
308+
* HighSlide JS: ``INCLUDED_HIGHSLIDEJS``
309+
* Font Awesome CSS: ``INCLUDED_FONTAWESOMECSS``
310+
* ColorBox JS: ``INCLUDED_COLORBOXJS``
311+
* ColPick JS: ``INCLUDED_COLPICKJS``
312+
* MoTools JS: ``INCLUDED_MOTOOLSJS``
313+
* Dojo JS: ``INCLUDED_DOJOJS``
314+
* Angular JS: ``INCLUDED_ANGULARJS``
315315

316316
.. seealso::
317317

0 commit comments

Comments
 (0)