merged branch adrienbrault/patch-1 (PR #751)

Commits
-------

80a2a7e [Doc] Fix some typos

Discussion
----------

[Doc] Fix some typos

Hi!

[Doc] Fix some typos

fixed chmod mode to apply the umask correctly (closes #749)

fixed a phpdoc (closes #748)

added  a note in the CHANGELOG (refs #745)

merged branch catchamonkey/compiler_outdent (PR #747)

Commits
-------

459a076 The Compiler outdent method will now only reduce the indentation if it's OK  to do so, It moves and modifies the check up above the indentation modification.

Discussion
----------

Compiler outdent step validation moved up and modified

The Compiler outdent method will now only reduce the indentation if it's OK
to do so, It moves and modifies the check up above the indentation modification.
It's a small improvement but saves a wasted decrement of a variable, when you know an exception will be thrown immediately following it.

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

by nikic at 2012-06-07T23:18:00Z

Compared to throwing an exception decrementing a variable has practically no cost ;)

PS: Not saying that the change is bad or something like that, only saying that performance is the wrong reason to do it :)

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

by catchamonkey at 2012-06-07T23:20:23Z

Yeah, there is no performance gain, but the code only executes that which is necessary.
It's like there being no point preparing a response before you know if you are going to send it. (slightly OTT example, but you get what I mean) :)

The Compiler outdent method will now only reduce the indentation if it's OK
 to do so, It moves and modifies the check up above the indentation modification.

added Komodo to the list of supported IDEs

added Coda 2 in the list of supported IDEs

bumped version to 1.8.3

prepared the 1.8.2 release

fixed some CS issues

merged branch blaugueux/master (PR #736)

Commits
-------

c53a02d Fixed DateTimeZone support in date filter.

Discussion
----------

Fixed DateTimeZone support in date filter.

Now $date converted to a DateTime has a DateTimeZone in every case.

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

by fabpot at 2012-05-28T19:20:47Z

Can you add some unit tests to avoid regression to happen later? Thanks.

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

by blaugueux at 2012-05-29T07:52:58Z

@fabpot Here they are.

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

by fabpot at 2012-05-29T10:29:28Z

Great! Can you add a note in the CHANGELOG file and then squash your commit before I merge? Thanks.

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

by blaugueux at 2012-05-29T12:52:22Z

Changelog added. First squash of my life, i cannot say if it's good :(

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

by stof at 2012-05-29T12:55:26Z

it is not. you merged your remote branch instead of forcing the push

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

by blaugueux at 2012-05-29T17:12:05Z

Ok @stof, thanks for your help. I think it's good now.

@fabpot Is it ok for you ?

Fixed DateTimeZone support in date filter.

Prevents the DateTime object to be updated when changing the DateTimeZone.

Unit tests added.

Updated changelog.

updated CHANGELOG

merged branch maartendekeizer/master (PR #738)

Commits
-------

f0a0036 Add abs filter including unit test and documentation

Discussion
----------

Add abs filter to Twig

With abs it is possible to make negative values positive and leave positive values positive.

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

by maartendekeizer at 2012-05-26T19:02:34Z

Example when using abs

{% if days_left < 0 %}
    {{ days_left|abs }} days to late!
{% else %}
    {{ days_left }} days left
{% endif %}

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

by stof at 2012-05-26T19:12:43Z

I'm not sure about adding this filter in the core. I think it would be better in the Twig-extensions repo. @fabpot thoughts ?

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

by maartendekeizer at 2012-05-26T21:18:47Z

@stof abs is a really basic/natural function it belongs to the base mathematic functions. If this was an implementation of for example the ABC-formula or Pythagoras it must be in the extension repo., but abs is more core related I think.

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

by fabpot at 2012-05-27T07:11:33Z

Can you add some unit tests and squash your commits? Thanks.

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

by maartendekeizer at 2012-05-27T14:40:56Z

First time squashing commits; learned something new :-). I will add a unit test tomorrow.

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

by maartendekeizer at 2012-05-27T17:36:58Z

Unit test and documentation added.

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

by fabpot at 2012-05-27T18:41:25Z

After fixing the markup issue, you will have to squash again ;)

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

by maartendekeizer at 2012-05-27T20:44:29Z

Done

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

by fabpot at 2012-05-28T19:19:55Z

Looks like you have not squash your commits.

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

by maartendekeizer at 2012-05-28T20:05:34Z

@fabot you want one commit for the complete pr?

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

by fabpot at 2012-05-28T20:06:53Z

For such a PR, yes.

Add abs filter
including unit test and documentation

merged branch hhamon/doc_fixes (PR #737)

Commits
-------

953a0ce [doc] fixed Composer section of the introduction. Added a side note to the Composer official documentation for getting more information.

Discussion
----------

[doc] fixed twig version in installation documentation.

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

by lyrixx at 2012-05-25T23:30:01Z

why not use something like "1.*" ?

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

by fabpot at 2012-05-26T05:55:16Z

This is just an example and we are not going to update it with each new release. So, we can either use `1.*` as suggested by @lyrixx or/and add a note referring to the Composer docs.

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

by hhamon at 2012-05-28T05:34:54Z

That's better indeed!

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

by hhamon at 2012-05-28T05:40:46Z

@fabpot you can merge this PR

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

by fabpot at 2012-05-28T06:07:05Z

Can you squash?

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

by hhamon at 2012-05-28T06:31:42Z

@fabpot squashed!

[doc] fixed Composer section of the introduction. Added a side note to the Composer official documentation for getting more information.

fixed an obsolete comment in phpdoc

fixed a regression when using a number in template attributes (closes #733)

added an example for the loop variable (closes #732)

fixed compiler when mbstring.func_overload is set to 2 (closes #731)

fixed CS

bumped version to 1.8.2-DEV

prepared the 1.8.1 release

merged branch Tobion/patch-5 (PR #729)

Commits
-------

b692268 removed leftover of 93c4406f8 and made some language improvements

Discussion
----------

removed leftover of 93c4406f8 and made some language improvements

removed leftover of 93c4406f8 and made some language improvements

fixed tests for PHP 5.2

fixed typos in the doc (closes #726)

updated CHANGELOG

fixed CHANGELOG

fixed a regression introduced by aa6b8358161e4ef35299be06a5ff61bfcba46f6a

removed obsolete code

switched to use mbstring whenever possible instead of iconv (because of issues with iconv and Japanese -- thanks @ganchiku for the tip)

merged branch Fustrate/master (PR #725)

Commits
-------

6c06a32 Coding style

Discussion
----------

Minor typos and coding style

- Unnecessary return in Lexer
- Typo in TokenParserBrokerInterface

Coding style

- Unnecessary return in Lexer
- Typo in TokenParserBrokerInterface

merged branch literal/v1.8.0 (PR #723)

Commits
-------

d13f742 fix example in doc for embed tag
b11412e improve documentation for embed tag

Discussion
----------

improve documentation for embed tag

https://github.com/fabpot/Twig/issues/722

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

by fabpot at 2012-05-10T05:10:37Z

I think this is a great improvement over the original version! The notion of "micro-layouts" certainly helps understand the concept behind the `embed` tag.

Thanks.

fix example in doc for embed tag

improve documentation for embed tag

fixed dump function is_safe value when html_errors is not defined in php.ini (closes #720)

bumped version to 1.9.0-DEV

prepared the 1.8.0 release

added documentation for the bitwise operators (closes #707)

added tests for bitwise operators

clarified usage of the dump function (closes #718)

enforced interface when adding tests, filters, functions, and node visitors from extensions

This avoids some hard to find bugs where you use a Twig_Function_Method
for instance for a filter. This is hard to find because the interface is
almost the same but the getSafe() contract is not the same.

updated CHANGELOG

merged branch stof/mutable_datetime (PR #717)

Commits
-------

a23d4fb Fixed the handling of the timezone for DateTime objects
a164469 Added a failing test demonstrating the bug with DateTime and timezones

Discussion
----------

Mutable datetime

PHP DateTime objects are mutable so changing the timezone when displaying
it affects the following code whereas it does not occur for other
supported formats.

Fixed the handling of the timezone for DateTime objects

Added a failing test demonstrating the bug with DateTime and timezones

PHP DateTime objects are mutable so changing the timezone when displaying
it affects the following code whereas it does not occur for other
supported formats.

added a missing entry in the CHANGELOG

fixed markup in the doc

merged two doc files

renamed a doc file

refactored the hacking twig doc chapter

tweaked doc

added a new recipe

fixed embed compilation when used more than once in a template (closes #714)

made the autoescape tag argument optional (defaults to 'html')

added a note about auto-escaping and the embed tag

We cannot use the parent template to guess the auto-escaping strategy
either as it can be any valid expression, for which we have the real
value only at runtime (auto-escaping happening at compilation time).

ensured that each template (Twig_Node_Module) is only visited once

simplified code

added a way to change the default escaping strategy after creating the Twig environment

tweaked doc for the escape filter

simplified usage of the autoescape tag; the only argument is now the escaping strategy or false

tweaked an error message

added a way to dynamically change the auto-escaping strategy according to the template filename

changed the autoescape option to also accept a supported escaping strategy (for BC, true is equivalent to html)

fixed CS

fixed markup in the doc

merged branch fabpot/inline (PR #709)

Commits
-------

07f13df fixed typo
0f623a1 added the parent as an argument to the embed tag (removes the need to use extends)
fab21d8 renamed the inline tag to embed
1b54780 tweaked examples
955726a changed the inline examples to not use even/odd to avoid confustion
fb07f3a renamed indice to index
fafde80 added more info in the docs
2fa8224 added the inline tag

Discussion
----------

added the inline tag

At Sensio, we manage websites with many mostly-static pages that share almost the same structure (with different content of course), but with some subtle differences. After trying many different possibilities (external templates, inheritance, macros, and some ugly ones that I won't even tell you about ;)) to structure our templates, we came to the conclusion that we needed a new concept.

This new concept is implemented via a new tag, `inline`, which I want to include in Twig 1.8 (not for the upcoming 1.7 which is due this week). But before merging this code into master, I want to gather some feedback from the community. One of my questions being: Will you use this new possibility?

I won't tell you more here as the tag is fully documented in the diff.

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

by oojacoboo at 2012-04-23T17:58:59Z

odd/even?  What other potential uses do you see?  I think I can see a couple, but generally speaking, unless it's a content site (ie. blog), the odd/even isn't very useful.  Reusing blocks even with minor customizations on a per page level is highly needed for our development processes, but an odd/even scenario is certainly not.

Maybe I'm missing something here.  What about passing attributes to a block or something?

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

by fabpot at 2012-04-23T18:02:51Z

@oojacoboo odd/even is just an example. Of course, this is not a real use case, this was just an attempt to find an example.

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

by stof at 2012-04-23T18:03:53Z

@oojacoboo there is no reference to odd/even in the code, only in the doc

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

by simensen at 2012-04-23T18:20:10Z

I think that it would be useful in some cases. I can't think of any places I could use it *right now* but I know I've had to jump through hoops on more than a few occasions to get things to work correctly. Being able to keep functionality like this in mind at the design stage might have made things easier in a few of those cases.

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

by lyrixx at 2012-04-23T18:21:34Z

I think it's a good idea.

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

by tyomo4ka at 2012-04-23T18:25:07Z

I think it will be great. Each time when I need to include templat that simple extends another one and replace one small block I think: "Why I should to create new template for it?". Thanks, @fabpot

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

by bmatzner at 2012-04-23T18:28:23Z

Right on. +1

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

by cranberyxl at 2012-04-23T18:40:17Z

I believe the concept is a good one, but the example using even and odd will confuse frontend developers because it's usually used in the context of loops and table striping. even changing it to styleA.twig and styleB.twig would be more understandable.

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

by lyrixx at 2012-04-23T19:01:32Z

@cranberyxl  +1

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

by fabpot at 2012-04-23T19:24:21Z

ok, I've just changed the examples to not used even/odd.

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

by jorgelbg at 2012-04-23T19:27:53Z

I think that the inline tag is a valid concept, but still I can't get it landed on a real case!

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

by sirowl at 2012-04-23T19:35:07Z

As a webdesigner with a steady foundation in CSS I find the term inline very confusing, I had to consult a developer to actually understand what was the idea of the tag as I so heavily associated it with CSS's terminology of inline.
The CSS terminololigy got me on the wrong track in terms of what the tag was supposed to be used for.
I have no good suggestion for a name for it, but inline could probably get very confusing for a lot of webdesigners making use of twig.

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

by fabpot at 2012-04-23T19:41:48Z

@sirowl `inline` was the best term I could come up with but I'm open to any other suggestion.

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

by sirowl at 2012-04-23T19:47:30Z

The best I could come up with is embed, as it is an embedded document of sorts.. or something along those lines.

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

by bschussek at 2012-04-23T19:48:54Z

I like `embed`.

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

by bmatzner at 2012-04-23T19:55:37Z

Hi Fabien, inline makes sense to me, although I understand @sirowl's comment, but I even find the concept quite comparable to the CSS display:block/display:inline logic. @sirowl's suggestion of "embed" would be just as fine, as it's just a matter of learning what this particular tag refers to. The ambiguation against the import tag is probably the most crucial, explaining the advantages of "inline" (or whatever) over doing an import

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

by Baachi at 2012-04-23T20:03:26Z

I lke ```embed```, too.

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

by Nami-Doc at 2012-04-23T20:11:42Z

I like `embed`, but `inline` is fine too.

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

by tijuan at 2012-04-23T20:36:39Z

I like the feature
I like embed

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

by pulse00 at 2012-04-23T20:56:41Z

+1 for `embed`

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

by mickaelandrieu at 2012-04-24T00:35:01Z

I like embed, inline for me is an CSS behavior

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

by matheo at 2012-04-24T04:29:05Z

+1 for `embed`

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

by fabpot at 2012-04-24T05:35:06Z

I've just renamed the `inline` tag to `embed`.

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

by sirowl at 2012-04-24T06:55:17Z

@fabpot agreed, much more clear to me now. Look forward to using it.

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

by pvanliefland at 2012-04-24T07:20:35Z

I would like to use this feature as well

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

by tristanbes at 2012-04-24T07:38:57Z

So if I take another example, to help me understand better

I do have a common structure stated as follows :
   ```
 <body>
    <div class="row">
        <div class="main">
            <div id="homepage">
               {% block content %}
            </div>
       </div>
    </div>
    </body>
```
Let's say for the other pages, the `id="homepage"` is replaced by `id="innerPage"` `id="specialLayout"`

So far I created 3 base template with those changes, so with the new `embed` tag, does this concept fits here or I misunderstood the function of `embed` feature ?

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

by ruian at 2012-04-24T07:47:38Z

@tristanbes with `embed`, you can "include" another twig template and modify directly these differents `block`.
If you want to understand you can read `embed` tests https://github.com/fabpot/Twig/blob/0f623a11129ce28a897dbd5b812241d8b3ea9730/test/Twig/Tests/Fixtures/tags/embed/

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

by samuelmichaud at 2012-04-24T08:02:42Z

Yes ! I will definitely use this ! Thanks

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

by michelsalib at 2012-04-24T08:05:47Z

As @samuelmichaud said we have some uses cases (such as menu generation, breadcrumbs or quick action embedding). So this feature makes total sense.

merged branch hason/fix_test5.2 (PR #710)

Commits
-------

28bd307 Fixed guess template info for PHP 5.2

Discussion
----------

Fixed guess template info for PHP 5.2

Fixed guess template info for PHP 5.2

fixed typo

added the parent as an argument to the embed tag (removes the need to use extends)

bumped version to 1.8.0-DEV

prepared the 1.7.0 release

renamed the inline tag to embed

merged branch hason/travis (PR #633)

Commits
-------

e109233 set travis-ci icon to master
623fac0 Fixed test for "dump" function if xdebug>2.2 is loaded
bd89ec8 Added travis build file

Discussion
----------

Added travis build file

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

by stealth35 at 2012-02-14T14:23:44Z

you could add the twig extension install script

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

by stof at 2012-02-14T14:35:30Z

yeah, there should be 2 builds in the matrix, one with the C extension and one without

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

by hason at 2012-02-15T14:46:30Z

I created build file with matrix of builds. One with the C extension and one without. But I don't know why the C extension is not loaded for PHP 5.2.

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

by hason at 2012-02-16T10:29:02Z

@fabpot All tests passed [](http://travis-ci.org/hason/Twig)

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

by stof at 2012-04-01T10:30:01Z

@fabpot any reason not to merge it ?

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

by drak at 2012-04-02T02:01:44Z

+1, this helps with QA

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

by nikic at 2012-04-02T09:20:07Z

+1

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

by fabpot at 2012-04-20T09:41:50Z

Can you inline the .sh scripts into the travis file directly?

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

by hason at 2012-04-23T21:57:26Z

done

set travis-ci icon to master

Fixed test for "dump" function if xdebug>2.2 is loaded

Added travis build file

tweaked examples

changed the inline examples to not use even/odd to avoid confustion

renamed indice to index

merged branch hhamon/parser_cleanup (PR #708)

Commits
-------

81a1049 [Parser] removed useless else clause.

Discussion
----------

[Parser] removed useless else clause.

added more info in the docs

added the inline tag

[Parser] removed useless else clause.

fixed compatibility with PHP 5.2 and simplify code a bit

fixed template line number in some exceptions

added unit tests to prove the issue with the current introspection exception code

tweaked the iterable example in the docs

tweaked the iterable example in the docs

merged branch martinml/master (PR #705)

Commits
-------

3eecc60 Silence chmod() warnings (operation not permitted) when using CIFS mounts and possibly other cases.

Discussion
----------

Silence chmod() warnings (operation not permitted) when using CIFS

When using chmod() in a CIFS mount (NTFS) in Linux, Twig 1.6.5 in PHP 5.3.3 throws a warning:

    Warning: chmod(): Operation not permitted in /smb/.../Twig/Environment.php on line 1052

There may be other combinations that trigger this warning. Related Symfony issue: https://github.com/symfony/symfony/issues/2125

fixed doc

updated CHANGELOG

removed the array test and rename the traversable test to iterable

merged branch Seldaek/isarray (PR #700)

Commits
-------

c88f8e5 Micro-optimizations
70cca66 Simplify code
d78ed66 Add traversable test
5fd2f98 Add docs
b2e1675 Add is array test

Discussion
----------

is array/traversable tests

It's sometimes necessary to display arbitrary data to be able to know if we should recurse or not.

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

by stof at 2012-04-13T12:13:11Z

I'm not sure about the need for ``is array``. All Twig functions related to arrays also support traversable objects

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

by Seldaek at 2012-04-13T12:23:32Z

Well, I like to have both for explicitness and also because I would try is array without looking at the docs, and I don't think it's right to make is array return true for traversables.

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

by fabpot at 2012-04-20T09:38:33Z

One of the main goal of Twig is to abstract the "real" type of the variables. We have already discussed the introduction of many tests like these ones and I have always rejected them.

As many people seems to have a need for them (especially for arrays), I would consider adding the `traversable` test. But as @stof said, Twig tries to blur the difference between traversable objects and arrays, so I'm -1 for adding another test for arrays.

`traversable` is probably not the best name for web designers but I don't know what else we can use here.

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

by stof at 2012-04-20T09:41:05Z

In Jinja, there is a built-in test for this using ``iterable``: http://jinja.pocoo.org/docs/templates/#builtin-tests

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

by Seldaek at 2012-04-20T09:41:32Z

Ok. I can live with dropping the array test. Now for the name,  `is forloopable`? I am kind of at a loss as well apart from traversable. I think if it's clearly mentioned in the docs it's ok.

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

by Seldaek at 2012-04-20T09:52:12Z

iterable sounds ok to me, but just as confusing as traversable I guess to the unknowing. A sidenote, if I update the PR, any objection to adding the "undefined" test as well?

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

by fabpot at 2012-04-20T10:01:27Z

Let's use `iterable` as this is what Jinja already uses. Why would you want an `undefined` test as we already have `defined`? (`if foo is not defined` works well enough)

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

by Seldaek at 2012-04-20T11:16:26Z

I just saw jinja had it on that page, and I thought why not add it too? Still not a huge fan of the "not" for negation.

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

by fabpot at 2012-04-20T12:18:17Z

Well, let's first finish the `iterable` test first and let's discuss `undefined` in another PR or issue.

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

by fabpot at 2012-04-20T17:10:28Z

I can finish the PR is you want.

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

by Seldaek at 2012-04-20T17:13:11Z

If you like sure, it's mostly deleting code anyway. I just have to finish some work stuff now..