web/konrad/twig.git
12 years agoTwig extension: fix case when accessing property of an array casted into object
Charles [Fri, 21 Sep 2012 10:29:41 +0000]
Twig extension: fix case when accessing property of an array casted into object

12 years agoFix double free
Charles [Fri, 21 Sep 2012 09:42:13 +0000]
Fix double free

12 years agoEnhancements for twig extension
Charles [Fri, 21 Sep 2012 07:47:45 +0000]
Enhancements for twig extension

- Fix extension memory leak
- Fix gcc compile warning (cast to const char *)
- Add TWIG_HAS_DYNAMIC_PROPERTY
- Remove the previous translation of 'array_key_exists($item, $object)'
  into TWIG_ARRAY_KEY_EXISTS when checking object property because what
  it really means is property_exists which has already been handled by
  TWIG_HAS_PROPERTY and TWIG_HAS_DYNAMIC_PROPERTY
- Fix native extension spacing

12 years agoFix NativeExtensionTest
Charles [Thu, 20 Sep 2012 11:35:44 +0000]
Fix NativeExtensionTest

12 years agoNative extension: handle dynamic properties defined in the get_properties handler...
Charles [Thu, 20 Sep 2012 11:29:21 +0000]
Native extension: handle dynamic properties defined in the get_properties handler in a per instance fashion.

12 years agoupdated CHANGELOG
Fabien Potencier [Tue, 18 Sep 2012 09:38:15 +0000]
updated CHANGELOG

12 years agotweaked docs
Fabien Potencier [Tue, 18 Sep 2012 09:37:33 +0000]
tweaked docs

12 years agomerged branch drak/reusable_fixtures (PR #805)
Fabien Potencier [Tue, 18 Sep 2012 09:34:35 +0000]
merged branch drak/reusable_fixtures (PR #805)

Commits
-------

36372c6 Made node tests reusable

Discussion
----------

Reusable tests and fixtures

This PR makes Twig integration testing reusable for 3rd party extensions.

I have implemented resuable functional tests.

The [test runner](https://github.com/zikula/core/blob/master/src/Zikula/Bundle/CoreBundle/Tests/Twig/IntegrationTest.php) and the [fixtures](https://github.com/zikula/core/tree/master/src/Zikula/Bundle/CoreBundle/Tests/Twig/Fixtures) are the only part that an extension needs to implement itself.

I've included documentation in the PR.

---------------------------------------------------------------------------

by jusuff at 2012-08-11T20:41:55Z

This PR would make things little easier. Right now tests are organized in confusing way and are hard to implement for Twig extensions

---------------------------------------------------------------------------

by drak at 2012-08-14T09:09:45Z

@fabpot - This is ready for merging now. When it's merged I'll update the documentation as this PR significantly eases our ability to test extensions.

---------------------------------------------------------------------------

by fabpot at 2012-09-16T14:28:16Z

@drak Do you plan to update this PR with the code you have in Zikula?

---------------------------------------------------------------------------

by drak at 2012-09-16T14:37:30Z

@fabpot - Yes, I just need a couple of days (or less).

---------------------------------------------------------------------------

by drak at 2012-09-17T12:00:34Z

@fabpot - I have ported over the code.

---------------------------------------------------------------------------

by drak at 2012-09-18T08:23:18Z

@fabpot - I updated Zikula to us this particular branch so there is a practical test. I've updated the links in the top description part of this PR with the relevant links. The PR also refactors Twig's core tests to use the re-usable test base classes.

---------------------------------------------------------------------------

by fabpot at 2012-09-18T09:01:10Z

@drak Can you squash your commits before I merge? Thanks.

---------------------------------------------------------------------------

by drak at 2012-09-18T09:14:08Z

@fabpot - I've squashed them now. Thanks!

12 years agoMade node tests reusable
Drak [Mon, 17 Sep 2012 11:33:27 +0000]
Made node tests reusable

12 years agomerged branch hason/loader (PR #838)
Fabien Potencier [Tue, 18 Sep 2012 05:04:46 +0000]
merged branch hason/loader (PR #838)

Commits
-------

c503287 Fixed typo in Filesystem loader

Discussion
----------

Fixed typo in Filesystem loader

12 years agoFixed typo in Filesystem loader
Martin Hasoň [Mon, 17 Sep 2012 12:11:40 +0000]
Fixed typo in Filesystem loader

12 years agoadded missing information in the doc
Fabien Potencier [Sun, 16 Sep 2012 14:38:44 +0000]
added missing information in the doc

12 years agomerged branch fabpot/named-paths (PR #772)
Fabien Potencier [Sun, 16 Sep 2012 14:34:28 +0000]
merged branch fabpot/named-paths (PR #772)

Commits
-------

7e5acd1 fixed some possible warnings
f0d0d6d fixed typo
b7076fe added Twig_Loader_Filesystem::getNamespaces()
8f7ccd1 moved an exception
9426072 tweaked documentation
b9afa84 renamed the default filesystem namespace to __main__
c34541d changed notation of namespaced templates to @namespace/template_path
0eb4d01 added namespaced templates support in Twig_Loader_Filesystem
c24ea1b added Twig_Loader_Filesystem::prependPath()

Discussion
----------

added namespaced templates support in Twig_Loader_Filesystem

Everything is explained in the updated documentation.

Basically, it gives more flexibility when it comes to manage many templates that are not necessarily related (think frontent vs backend for instance, or templates from different Symfony bundles, ...). It is useful if, for each namespace, you need to look for templates in different directories. This implementation would probably be enough to replace the current Symfony bundle template paths, with more flexibility and less restrictions.

Feedback is more than welcome. One thing I'm not sure about is the `#` separator between the namespace and the template path. Is it the best choice? We need a character that is not used in filesystem paths, so good candidates are: `@`, `!`, `%`, `$`, `*`, `;`, `?`.

Some immediate benefits: give the same template flexibility that we have in Symfony to other frameworks using Twig like Silex. Another benefit would be the ability to share template between Symfony and other frameworks like Twig (more on a proof of concept later).

---------------------------------------------------------------------------

by alessandro1997 at 2012-07-14T16:25:42Z

This is very nice, and I think # is a great choice for the separator character. We could also use @.

---------------------------------------------------------------------------

by jjbohn at 2012-07-14T16:25:59Z

:+1: for #

---------------------------------------------------------------------------

by Symfomany at 2012-07-14T16:27:45Z

Pipe character isn't candidate?

---------------------------------------------------------------------------

by inspiran at 2012-07-14T16:32:52Z

I remember from the wetter.com talk at sf live that they actually implemented a similar functionality: allow twig templates to be defined different from the default path.   So they had something like:

{% extends "cms://Home/weather/..."  ... %}
If we could make the "cms" part a configurable setting then one could use it to point to a custom folder or even to a content provider (symfony cmf?)

Maybe that would be another approach?

---------------------------------------------------------------------------

by fabpot at 2012-07-14T16:33:21Z

The other syntax possibility is something more like we have in Symfony: `@namespace/template_path`.

---------------------------------------------------------------------------

by fabpot at 2012-07-14T16:34:37Z

@inspiran: that's another possibility: use the PHP stream notation.

---------------------------------------------------------------------------

by mvrhov at 2012-07-14T16:36:18Z

I'd also prefer uri/php streams syntax...

---------------------------------------------------------------------------

by Tobion at 2012-07-14T16:41:16Z

On Windows I can create a file with `#` in it. How does it distinguish that? I don't think we will find a single seperation character that works on all file systems, do we? And `@` (e.g. namespace@file) would be semantically strange (unless you read it from right-to-left).
So I'd also prefer uri/php stream syntax.

---------------------------------------------------------------------------

by Symfomany at 2012-07-14T16:43:17Z

@fabpot  Pointer Syntax? or greater than character?

---------------------------------------------------------------------------

by markstory at 2012-07-14T16:43:48Z

I'm in favour of the `foo://` style syntax.  It is similar to other parts of PHP, and avoids overlap with filesystem characters, and doesn't introduce new unique syntax.

---------------------------------------------------------------------------

by stof at 2012-07-14T16:44:25Z

@fabpot ``@namespace/template_path`` would make it confusing in Symfony. Because locating a resource with the kernel would use ``@AcmeDemoBundle/Resources/views/layout.html.twig`` whereas Twig would find the same file as ```@AcmeDemoBundle/layout.html.twig``.

And none of ``#``, ``@``, ``!``, ``%``, ``$`` or ``;`` is forbidden in filenames on Windows.

So I think the URI/stream notation is fine

---------------------------------------------------------------------------

by maryo at 2012-07-14T16:46:44Z

On Windows @!%$; are valid. Invalid characters are \/:*?"<>|

---------------------------------------------------------------------------

by maryo at 2012-07-14T16:47:55Z

Is > allowed on Linux?

---------------------------------------------------------------------------

by jschreuder at 2012-07-14T16:49:17Z

Some thoughts from a non-Symfony user:

* Not sure about the `#`, on the one hand it is used as the comment character on the other hand I'm also familiar with it as a membership operator (which supports its usage)
* The streams/PHP syntax: I don't really like this, the syntax signifies some type of protocol to be used. Even stretched to the limits of its definition you wouldn't be able to define different namespaces as different protocols (same protocol, different location).
* `@namespace/file/path` - kind of like this one, the `@` signifies that it is followed by a special symbol (being the namespace name) and other than that a normal path.
* Another option we've used in Fuel is the double colon `::` as in `namespace::file/path` (like the `#` it signifies membership, but doesn't have another significance in Twig)

---------------------------------------------------------------------------

by gunnarlium at 2012-07-14T17:10:15Z

Does it matter what symbols are valid in filenames? Won't the separator just be used for exploding the string into namespace and path, and thus never be used to directly access a file?

---------------------------------------------------------------------------

by alessandro1997 at 2012-07-14T17:15:27Z

As @gunnarlium said, what's the problem with filenames?

```php
$path = 'namespace#new#page.html';
$parts = explode('#', $path, 2);
list($namespace, $filename) = $parts;

// "namespace and "new#page.html"
var_dump($namespace, $filename);
```

---------------------------------------------------------------------------

by stof at 2012-07-14T17:16:19Z

@gunnarlium the issue is that if you use this symbols in the filename of your template without namespace, Twig will consider it as a namespaced template for another file name

---------------------------------------------------------------------------

by stof at 2012-07-14T17:16:45Z

@alessandro1997 the issue is that namespaces are optional

---------------------------------------------------------------------------

by alessandro1997 at 2012-07-14T17:17:07Z

@stof Oh, right. I didn't get that :-)

---------------------------------------------------------------------------

by gunnarlium at 2012-07-14T17:20:32Z

@stof Ok, I see. So what we want is a symbol which is not allowed as a filename? Or (suboptimally) just add a requirement that template filenames can't contain certain reserved symbols?

---------------------------------------------------------------------------

by fabpot at 2012-07-14T17:23:37Z

But then, who is using `#` in a filename?

Anyway, I'm going to implement a version based on PHP streams to stick to the PHP way. Using a PHP stream might also allow us to get rid of the array loader.

---------------------------------------------------------------------------

by alessandro1997 at 2012-07-14T17:24:01Z

@gunnarlium But how would we check if the user is using reserved symbols in template names? Twig isn't aware of the available templates' names until the user requests them. And when he/she does it's too late because Twig doesn't know if the separator is used as namespace or if it's part of the filename.

---------------------------------------------------------------------------

by stof at 2012-07-14T17:33:38Z

@fabpot even if no sane guy would probably use ``#`` in a filename, it would mean that Twig assumes that all its userbase is sane :)

---------------------------------------------------------------------------

by lolautruche at 2012-07-14T18:51:53Z

-1 for #
+1 for php stream, though I agree with @jschreuder's arguments against it
+2 for @ syntax like in Symfony. I find it more consistent.

---------------------------------------------------------------------------

by fabpot at 2012-07-14T19:50:31Z

One problem with PHP streams is that it makes using template names quite ugly and verbose:

    {% include 'twig://namespace/index.html' %}

And for non-namespaced templates:

    {% include 'twig:///index.html' %}

for which we can provide a shortcut:

    {% include 'index.html' %}

---------------------------------------------------------------------------

by fabpot at 2012-07-14T19:54:35Z

If we are using `@` like in Symfony, it will be:

    {% include '@namespace/index.html' %}

And for non-namespaced templates:

    {% include 'index.html' %}

And for Symfony users, the notation would be the same as for resources. If we register each bundle path as a namespace, the path would even be exactly the same:

    {% include '@AcmeDemoBundle/Resources/views/layout.html.twig' %}

EDIT: that's not so simple because of the templates stored under `app/`.

---------------------------------------------------------------------------

by mahono at 2012-07-14T19:56:54Z

might not be the best idea, but what about using Backslash as namespace char?

---------------------------------------------------------------------------

by fabpot at 2012-07-14T19:57:50Z

@mahono: I have not dared to propose this alternative ;)

---------------------------------------------------------------------------

by fabpot at 2012-07-14T20:06:17Z

I've just made an additional commit that implements the `@namespace/template_path` notation.

---------------------------------------------------------------------------

by Baachi at 2012-07-14T20:32:41Z

The `@` syntax is the most consistent solution.
But the integration in symfony2 is really hard and a BC break.

The stream syntax is not a good solution.
`twig` might already be already registered as a stream.

I would suggest the `::` syntax.

12 years agofixed some possible warnings
Fabien Potencier [Sun, 16 Sep 2012 14:32:36 +0000]
fixed some possible warnings

12 years agofixed typo
Fabien Potencier [Sun, 16 Sep 2012 13:21:03 +0000]
fixed typo

12 years agomerged branch loalf/patch-1 (PR #835)
Fabien Potencier [Sun, 16 Sep 2012 13:20:46 +0000]
merged branch loalf/patch-1 (PR #835)

Commits
-------

5c805a1 Update doc/intro.rst

Discussion
----------

Update doc/intro.rst

How to load Twig using Composer

12 years agoUpdate doc/intro.rst
Javier Lopez [Sun, 16 Sep 2012 12:08:03 +0000]
Update doc/intro.rst

How to load Twig using Composer

12 years agomerged branch pborreli/patch-1 (PR #834)
Fabien Potencier [Thu, 13 Sep 2012 13:15:32 +0000]
merged branch pborreli/patch-1 (PR #834)

Commits
-------

2c1be8f Fixed typo

Discussion
----------

Fixed typo

http://english.stackexchange.com/questions/3640/is-substract-versus-subtract-a-proper-word

12 years agoFixed typo
Pascal Borreli [Wed, 12 Sep 2012 08:29:08 +0000]
Fixed typo

http://english.stackexchange.com/questions/3640/is-substract-versus-subtract-a-proper-word

12 years agoadded a small clarification in the doc
Fabien Potencier [Fri, 7 Sep 2012 13:42:22 +0000]
added a small clarification in the doc

12 years agoadded more information about single-quoted vs double-quoted strings
Fabien Potencier [Fri, 31 Aug 2012 07:12:18 +0000]
added more information about single-quoted vs double-quoted strings

12 years agomerged branch franmomu/patch-1 (PR #821)
Fabien Potencier [Thu, 30 Aug 2012 11:00:12 +0000]
merged branch franmomu/patch-1 (PR #821)

Commits
-------

93b41c3 [Test] Fix double semicolon

Discussion
----------

[Test] Fix double semicolon

12 years ago[Test] Fix double semicolon
Fran Moreno [Thu, 30 Aug 2012 10:51:43 +0000]
[Test] Fix double semicolon

12 years agomerged branch tkoomzaaskz/master (PR #819)
Fabien Potencier [Wed, 29 Aug 2012 05:44:07 +0000]
merged branch tkoomzaaskz/master (PR #819)

Commits
-------

682b0aa improved twig docs: twig for templates designers, NetBeans PHP IDE supports twig natively from 7.2

Discussion
----------

twig for template designers: NetBeans 7.2 supports twig natively

from NetBeans 7.2 twig support is native and does not have to be installed manually.

http://plugins.netbeans.org/plugin/40565/php-symfony2-framework
quote:
Yes; this extension will be part of NetBeans 7.2, more information here [1].
Tomas
[1] http://blogs.oracle.com/netbeansphp/entry/initial_support_for_symfony2

https://blogs.oracle.com/netbeansphp/entry/initial_support_for_symfony2

I checked it myself - installed NB7.2 and it wonderfully highlights twig.

12 years agoimproved twig docs: twig for templates designers, NetBeans PHP IDE supports twig...
Tomasz Ducin [Tue, 28 Aug 2012 20:41:01 +0000]
improved twig docs: twig for templates designers, NetBeans PHP IDE supports twig natively from 7.2

12 years agomade a better test
Fabien Potencier [Tue, 28 Aug 2012 15:05:50 +0000]
made a better test

12 years agofixed an error when a token parser pass a closure as a test to the subparse() method...
Fabien Potencier [Tue, 28 Aug 2012 12:04:14 +0000]
fixed an error when a token parser pass a closure as a test to the subparse() method (closes #818)

12 years agobumped version to 1.9.3-DEV
Fabien Potencier [Sat, 25 Aug 2012 17:36:54 +0000]
bumped version to 1.9.3-DEV

12 years agoprepared the 1.9.2 release
Fabien Potencier [Sat, 25 Aug 2012 17:32:57 +0000]
prepared the 1.9.2 release

12 years agoupdated CHANGELOG
Fabien Potencier [Fri, 24 Aug 2012 13:50:46 +0000]
updated CHANGELOG

12 years agofixed the in operator for objects that contains circular references (closes #813)
Fabien Potencier [Fri, 24 Aug 2012 13:49:46 +0000]
fixed the in operator for objects that contains circular references (closes #813)

12 years agofixed CS
Fabien Potencier [Fri, 24 Aug 2012 12:10:43 +0000]
fixed CS

12 years agomerged branch Banane9/patch-1 (PR #817)
Fabien Potencier [Fri, 24 Aug 2012 12:10:21 +0000]
merged branch Banane9/patch-1 (PR #817)

Commits
-------

ca3b156 Added Highlighter for Notepad++ to the list

Discussion
----------

Added Highlighter for Notepad++ to the list

Ok I edited it :)

12 years agoupdated CHANGELOG
Fabien Potencier [Fri, 24 Aug 2012 12:10:15 +0000]
updated CHANGELOG

12 years agoAdded Highlighter for Notepad++ to the list
Arne Kiesewetter [Fri, 24 Aug 2012 09:41:34 +0000]
Added Highlighter for Notepad++ to the list

12 years agomerged branch arnaud-lb/792 (PR #814)
Fabien Potencier [Thu, 23 Aug 2012 12:18:58 +0000]
merged branch arnaud-lb/792 (PR #814)

Commits
-------

8fa97bb php5.2 fix
d6fc86a [Tests] Test accessing a public property when \ArrayAccess is implemented
6704227 [ext] made ext consistent with 8ec73cf4759f90b961de1732da1732e60c1dbfff
228d2b0 [ext] use only read_property handler

Discussion
----------

Alternative fix for 792

This fixes #792 as described in https://github.com/fabpot/Twig/issues/792#issuecomment-7482480

- 228d2b0 removes the *object-to-array convertion* trick, since the bug comes from there, and it doesn't appear to be needed
- 6704227 is the equivalent of aa6b8358161e4ef35299be06a5ff61bfcba46f6a + 8ec73cf4759f90b961de1732da1732e60c1dbfff (this were not needed before because the bug was hidden by the object to array trick: numeric strings and integers are treated equally in arrays keys)

---------------------------------------------------------------------------

by stof at 2012-08-23T10:19:17Z

@arnaud-lb you need to fix the PHP 5.2 support

12 years agomerged branch arnaud-lb/fix-zts-5.2 (PR #815)
Fabien Potencier [Thu, 23 Aug 2012 12:18:46 +0000]
merged branch arnaud-lb/fix-zts-5.2 (PR #815)

Commits
-------

46ce339 [ext] fixed php 5.2 ZTS builds

Discussion
----------

[ext] Fixed php 5.2 ZTS builds

In 5.2 zend_hash_apply_with_arguments() and its callback argument do not have a TSRMLS.

12 years ago[ext] fixed php 5.2 ZTS builds
Arnaud Le Blanc [Thu, 23 Aug 2012 11:58:44 +0000]
[ext] fixed php 5.2 ZTS builds

zend_hash_apply_with_arguments and its callback argument do not have a
TSRMLS in php 5.2.

12 years agophp5.2 fix
Arnaud Le Blanc [Thu, 23 Aug 2012 11:32:55 +0000]
php5.2 fix

12 years ago[Tests] Test accessing a public property when \ArrayAccess is implemented
Victor Berchet [Mon, 30 Jul 2012 15:02:03 +0000]
[Tests] Test accessing a public property when \ArrayAccess is implemented

Test case for GH-792

12 years ago[ext] made ext consistent with 8ec73cf4759f90b961de1732da1732e60c1dbfff
Arnaud Le Blanc [Mon, 13 Aug 2012 20:28:14 +0000]
[ext] made ext consistent with 8ec73cf4759f90b961de1732da1732e60c1dbfff

12 years ago[ext] use only read_property handler
Arnaud Le Blanc [Mon, 13 Aug 2012 20:25:20 +0000]
[ext] use only read_property handler

12 years agomerged branch arnaud-lb/807 (PR #812)
Fabien Potencier [Mon, 20 Aug 2012 19:10:44 +0000]
merged branch arnaud-lb/807 (PR #812)

Commits
-------

3ca6654 avoid crash when TWIG_GET_ARRAY_ELEMENT returns NULL
3694d4b allow --EXCEPTION-- tests to have a --DATA-- block

Discussion
----------

Fix for #807 (Segmentation fault when exception thrown inside any of ArrayAccess implementation methods)

This fixes #807

12 years agoavoid crash when TWIG_GET_ARRAY_ELEMENT returns NULL
Arnaud Le Blanc [Mon, 20 Aug 2012 18:28:22 +0000]
avoid crash when TWIG_GET_ARRAY_ELEMENT returns NULL

fixes #807

12 years agoallow --EXCEPTION-- tests to have a --DATA-- block
Arnaud Le Blanc [Mon, 20 Aug 2012 18:27:57 +0000]
allow --EXCEPTION-- tests to have a --DATA-- block

12 years agomerged branch garak/patch-1 (PR #809)
Fabien Potencier [Fri, 17 Aug 2012 17:01:21 +0000]
merged branch garak/patch-1 (PR #809)

Commits
-------

fce92f8 Update doc/templates.rst

Discussion
----------

Update doc/templates.rst

Komodo and Komodo Edit have builtin Twig support since last version (7.1)

12 years agoUpdate doc/templates.rst
Massimiliano Arione [Fri, 17 Aug 2012 15:14:24 +0000]
Update doc/templates.rst

Komodo and Komodo Edit have builtin Twig support since last version (7.1)

12 years agomerged branch sdepablos/master (PR #802)
Fabien Potencier [Tue, 7 Aug 2012 08:38:05 +0000]
merged branch sdepablos/master (PR #802)

Commits
-------

b98fff1 Added missing abs and date_modify functions to the filters toctree

Discussion
----------

Added missing abs and date_modify functions to the filters toctree

Both abs.rst and date_modify.rst were createad, but were not linked on the filters toctree

12 years agoAdded missing abs and date_modify functions to the filters toctree
Sergi de Pablos [Tue, 7 Aug 2012 08:31:19 +0000]
Added missing abs and date_modify functions to the filters toctree

12 years agoadded a test to verify that the merge filter actually behaves like said in the docume...
Fabien Potencier [Sat, 28 Jul 2012 08:32:10 +0000]
added a test to verify that the merge filter actually behaves like said in the documentation (closes #789)

12 years agofixed typo
Fabien Potencier [Tue, 17 Jul 2012 08:29:54 +0000]
fixed typo

12 years agoadded Twig_Loader_Filesystem::getNamespaces()
Fabien Potencier [Tue, 17 Jul 2012 06:28:40 +0000]
added Twig_Loader_Filesystem::getNamespaces()

12 years agomoved an exception
Fabien Potencier [Tue, 17 Jul 2012 06:26:32 +0000]
moved an exception

12 years agotweaked documentation
Fabien Potencier [Tue, 17 Jul 2012 06:23:45 +0000]
tweaked documentation

12 years agorenamed the default filesystem namespace to __main__
Fabien Potencier [Tue, 17 Jul 2012 06:23:39 +0000]
renamed the default filesystem namespace to __main__

12 years agochanged notation of namespaced templates to @namespace/template_path
Fabien Potencier [Sat, 14 Jul 2012 20:05:02 +0000]
changed notation of namespaced templates to @namespace/template_path

12 years agoadded namespaced templates support in Twig_Loader_Filesystem
Fabien Potencier [Sat, 14 Jul 2012 13:26:10 +0000]
added namespaced templates support in Twig_Loader_Filesystem

12 years agoadded Twig_Loader_Filesystem::prependPath()
Fabien Potencier [Sat, 14 Jul 2012 14:15:01 +0000]
added Twig_Loader_Filesystem::prependPath()

12 years agobumped version to 1.9.2-DEV
Fabien Potencier [Sun, 22 Jul 2012 10:32:58 +0000]
bumped version to 1.9.2-DEV

12 years agoprepared the 1.9.1 release
Fabien Potencier [Sun, 22 Jul 2012 10:25:41 +0000]
prepared the 1.9.1 release

12 years agoreverted partially a previous commit
Fabien Potencier [Sat, 21 Jul 2012 22:07:57 +0000]
reverted partially a previous commit

This code was not only useful for macros but also when you call a method on
a template instance -- this is not recommended/supported but used by the Symfony profiler)

12 years agooptimized macro calls when auto-escaping is on (closes #779)
Fabien Potencier [Sat, 21 Jul 2012 21:20:56 +0000]
optimized macro calls when auto-escaping is on (closes #779)

12 years agomoved some macro related logic to the generated code
Fabien Potencier [Sat, 21 Jul 2012 20:47:06 +0000]
moved some macro related logic to the generated code

12 years agomerged branch acasademont/patch-1 (PR #781)
Fabien Potencier [Fri, 20 Jul 2012 12:41:38 +0000]
merged branch acasademont/patch-1 (PR #781)

Commits
-------

ee87848 Little typo
c31dd7b The empty test now also checks for empty arrays

Discussion
----------

The empty test now also checks for empty arrays

Little change in the docs to reflect that, as i had to check the source code to see if it worked

12 years agoLittle typo
Albert Casademont [Thu, 19 Jul 2012 11:03:26 +0000]
Little typo

12 years agoThe empty test now also checks for empty arrays
Albert Casademont [Thu, 19 Jul 2012 11:00:01 +0000]
The empty test now also checks for empty arrays

12 years agoupdated CHANGELOG
Fabien Potencier [Thu, 19 Jul 2012 05:29:29 +0000]
updated CHANGELOG

12 years agomerged branch pborreli/master (PR #771)
Fabien Potencier [Wed, 18 Jul 2012 21:07:28 +0000]
merged branch pborreli/master (PR #771)

Commits
-------

1c98282 Removed duplicate array keys
30a168c Fixing typos

Discussion
----------

Fixed typos

12 years agofixed typo in previous merge
Fabien Potencier [Wed, 18 Jul 2012 21:05:29 +0000]
fixed typo in previous merge

12 years agomerged branch Nami-Doc/patch-1 (PR #773)
Fabien Potencier [Wed, 18 Jul 2012 21:03:43 +0000]
merged branch Nami-Doc/patch-1 (PR #773)

Commits
-------

a6f03bd Reverse argument for number_format

Discussion
----------

Reverse argument for number_format

Just a little fix, the doc isn't following twig/php's argument order

12 years agomerged branch havvg/patch-1 (PR #777)
Fabien Potencier [Wed, 18 Jul 2012 21:00:28 +0000]
merged branch havvg/patch-1 (PR #777)

Commits
-------

0869d15 fix example autoescape for javascript

Discussion
----------

fix example autoescape for javascript

{% autoescape js %} triggers an exception "An escaping strategy must be a string or a Boolean in .."

12 years agoadded a note about accessing variable attributes that contains operators (closes...
Fabien Potencier [Wed, 18 Jul 2012 07:53:29 +0000]
added a note about accessing variable attributes that contains operators (closes #775)

12 years agomade Twig_Loader_Chain more explicit about problems
Fabien Potencier [Wed, 18 Jul 2012 07:48:23 +0000]
made Twig_Loader_Chain more explicit about problems

12 years agofix example autoescape for javascript
Toni Uebernickel [Tue, 17 Jul 2012 14:52:07 +0000]
fix example autoescape for javascript

{% autoescape js %} triggers an exception "An escaping strategy must be a string or a Boolean in .."

12 years agomerged branch Tobion/patch-6 (PR #774)
Fabien Potencier [Mon, 16 Jul 2012 07:42:44 +0000]
merged branch Tobion/patch-6 (PR #774)

Commits
-------

9d1f645 fix Twig_Function_Node extending wrong class

Discussion
----------

fix Twig_Function_Node extending wrong class

12 years agofix Twig_Function_Node extending wrong class
Tobias Schultze [Sun, 15 Jul 2012 21:06:53 +0000]
fix Twig_Function_Node extending wrong class

12 years agoReverse argument for number_format
Nami-Doc [Sun, 15 Jul 2012 11:50:43 +0000]
Reverse argument for number_format

12 years agorefactored the loader documentation
Fabien Potencier [Sat, 14 Jul 2012 14:10:03 +0000]
refactored the loader documentation

12 years agoRemoved duplicate array keys
Pascal Borreli [Fri, 13 Jul 2012 17:01:16 +0000]
Removed duplicate array keys

12 years agoFixing typos
Pascal Borreli [Fri, 13 Jul 2012 17:00:31 +0000]
Fixing typos

12 years agobumped version to 1.9.1-DEV
Fabien Potencier [Fri, 13 Jul 2012 16:31:16 +0000]
bumped version to 1.9.1-DEV

12 years agoprepared the 1.9.0 release
Fabien Potencier [Fri, 13 Jul 2012 16:26:34 +0000]
prepared the 1.9.0 release

12 years agofixed the template escaper guesser example
Fabien Potencier [Fri, 13 Jul 2012 06:40:06 +0000]
fixed the template escaper guesser example

12 years agoclarified usage of the string and array loaders
Fabien Potencier [Fri, 13 Jul 2012 06:31:08 +0000]
clarified usage of the string and array loaders

12 years agomerged branch Adel-E/patch-1 (PR #767)
Fabien Potencier [Thu, 12 Jul 2012 19:30:05 +0000]
merged branch Adel-E/patch-1 (PR #767)

Commits
-------

0cb37ba Fix typehint "int" to "integer"

Discussion
----------

Fix typehint "int" to "integer"

---------------------------------------------------------------------------

by nikic at 2012-07-09T21:13:48Z

Why would you want to do that?

---------------------------------------------------------------------------

by Adel-E at 2012-07-09T21:21:34Z

Because in this file all @param use integer instead of int

12 years agoscaled back on named HTML entities since HTML 5's XML serialisation is actually XML...
Fabien Potencier [Thu, 12 Jul 2012 14:51:31 +0000]
scaled back on named HTML entities since HTML 5's XML serialisation is actually XML (not XHTML)

see https://github.com/zendframework/zf2/commit/d5862587ae7c2ff2da873f1bf30003103134a365

12 years agomade the parsing independent of the template loaders
Fabien Potencier [Tue, 10 Jul 2012 16:30:12 +0000]
made the parsing independent of the template loaders

This basically reverts 57ff88255ef35ad8da325d0cb0bf140e653b132c

12 years agofixed PHP 5.2 issue in unit tests
Fabien Potencier [Tue, 10 Jul 2012 12:56:21 +0000]
fixed PHP 5.2 issue in unit tests

12 years agofixed exception trace when an error occurs when rendering a child template
Fabien Potencier [Tue, 10 Jul 2012 12:40:33 +0000]
fixed exception trace when an error occurs when rendering a child template

12 years agoFix typehint "int" to "integer"
Adel [Mon, 9 Jul 2012 20:28:19 +0000]
Fix typehint "int" to "integer"

12 years agofixed composer version
Fabien Potencier [Wed, 4 Jul 2012 17:33:25 +0000]
fixed composer version

12 years agomerged branch shieldo/patch-2 (PR #763)
Fabien Potencier [Sat, 30 Jun 2012 08:06:47 +0000]
merged branch shieldo/patch-2 (PR #763)

Commits
-------

4e4d41b improved English in example text

Discussion
----------

improved English in example text

12 years agoimproved English in example text
Douglas Greenshields [Fri, 29 Jun 2012 18:16:04 +0000]
improved English in example text

12 years agomerged branch stealth35/sanitize_ext (PR #761)
Fabien Potencier [Thu, 28 Jun 2012 13:39:18 +0000]
merged branch stealth35/sanitize_ext (PR #761)

Commits
-------

9229ef3 Removed useless code

Discussion
----------

[Ext] Removed useless code

All init hooks could be bypassed (just return `SUCESS`)
There is no `ini` entries and no globals
Phpinfo automatically get the version when `MINFO` is null
Twig don't work with PHP 4 (ref to `20010901`)

12 years agoRemoved useless code
stealth35 [Wed, 27 Jun 2012 15:57:06 +0000]
Removed useless code

12 years agofixed PHP 5.2 incompatibilty
Fabien Potencier [Wed, 27 Jun 2012 09:49:34 +0000]
fixed PHP 5.2 incompatibilty

12 years agofixed url escaping strategy for PHP 5.2
Fabien Potencier [Wed, 27 Jun 2012 09:18:21 +0000]
fixed url escaping strategy for PHP 5.2

12 years agomerged branch Seldaek/date_modify (PR #760)
Fabien Potencier [Wed, 27 Jun 2012 09:12:10 +0000]
merged branch Seldaek/date_modify (PR #760)

Commits
-------

2c9e26e Fix date_modify filter for PHP 5.2

Discussion
----------

Fix date_modify filter for PHP 5.2

The return of the object was added in 5.3, this hopefully fixes it.

12 years agoFix date_modify filter for PHP 5.2
Jordi Boggiano [Wed, 27 Jun 2012 08:25:59 +0000]
Fix date_modify filter for PHP 5.2

12 years agoadded support for more escaping strategies (url, css, and html_attr)
Fabien Potencier [Wed, 27 Jun 2012 07:07:27 +0000]
added support for more escaping strategies (url, css, and html_attr)