fixed previous merge and added some unit tests

merged branch shvchk/patch-2 (PR #522)

Commits
-------

5f30db0 !empty($date[0]) → !empty($date)
0182dbc indentation fix
c7a06cb php warning fix
a4a2d6f Support of negative timestamps

Discussion
----------

Support of negative timestamps

Date could be checked against this condition, as proposed in my patch:

```php
ctype_digit((string) $date)
|| (('-' === $date[0])
    && (ctype_digit(substr($date, 1))))
```
or this:

```php
preg_match('/^\-?\d+/', $date)
```
Second is maybe more readable, but 20% slower.

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

by nikic at 2011/11/20 01:59:16 -0800

Couldn't we do an int validation here? I.e. `if (false !== filter_var($date, FILTER_VALIDATE_INT)) { ... }`. That will allow both positive and negative integers in the allowed integer range. Note though that it will no longer allow ints starting with a zero (apart from zero itself). To support that one would need to pass the ALLOW_OCTAL flag.

Some tests: http://codepad.viper-7.com/w0MDTH

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

by damianb at 2011/11/20 07:50:39 -0800

@nikic why not `ctype_digit`?

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

by nikic at 2011/11/20 07:53:26 -0800

@damianb ctype_digit is what this PR wants to replace, because it does not allow negative numbers. Replacing it with `ctype_digit((string) $date) || (!empty($date[0]) && ('-' === $date[0]) && ctype_digit(substr($date, 1)))` seems like overkill to me though, that's why I proposed to use `false !== filter_var($date, FILTER_VALIDATE_INT)` instead, which apart from being shorter is also semantically more correct.

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

by damianb at 2011/11/20 08:03:09 -0800

@nikic But does filter_var with FILTER_VALIDATE_INT perform better than what's proposed?

Also, I would think that the `empty($date[0])` should just be replaced with an empty check on the `$date` var itself - seems quite pointless to check if the first character is empty, that'd imply that either a) the string itself is empty b) `date[0] == 0`, which could result in a nasty gotcha for someone down the road.

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

by shvchk at 2011/11/20 09:36:17 -0800

@damianb, you are right, I will update the code.
@nikic, on 32-bit systems ```PHP_INT_MAX``` is ```2147483647```, so your code will fail on such systems for 01.01.1900, for example, which is ```-2208988800``` and is not that rarely used.

!empty($date[0]) → !empty($date)

indentation fix

php warning fix

prepared the 1.4.0-RC1 release

adding more structure to the for doc

added a note about using the loop variable in a for condition

Support of negative timestamps

optimized variable access when using PHP 5.4

fixed a warning (PHP 5.4)

merged branch arnaud-lb/test-fix (PR #513)

Commits
-------

9b4e298 fixed failing test

Discussion
----------

fixed failing test

fixed failing test

fixed typo

merged branch henrikbjorn/composer-autoload (PR #509)

Commits
-------

cf66a05 Add Autoloadig instructions for Composer

Discussion
----------

Add Autoloadig instructions for Composer

Add psr-0 autoload directive to composer.json for easier Autoloading
when using Composer as a package management tool

Add Autoloadig instructions for Composer

Add psr-0 autoload directive to composer.json for easier Autoloading
when using Composer as a package management tool

merged branch craue/patch-1 (PR #507)

Commits
-------

98a859b fixed link name for `extends` tag

Discussion
----------

fixed link name for `extends` tag

fixed link name for `extends` tag

removed json_encode call in Twig_Error when the filename is a string (to avoid the escaping of /)

fixed operator precedences in the doc

converted item parameter of Twig_Template::getAttribute() to string

fixed for tag loop variable when using a condition

refactored "for" tag (moved some logic from the token parser to the node)

changed the precedence of the .. operator

fixed a typo in the doc

updated CHANGELOG

merged branch hason/arrayloader (PR #496)

Commits
-------

253f813 added an exception for undefined templates in ArrayLoader::isFresh

Discussion
----------

added an exception for undefined templates in ArrayLoader::isFresh

merged branch trompette/patch-2 (PR #501)

Commits
-------

36020a9 typo

Discussion
----------

typo

typo

fixed for tag when using a condition with strict_variables set to false

added an exception for undefined templates in ArrayLoader::isFresh

refactored code

added Twig_Function_Node to allow more complex functions to have their own Node class

added Twig_Filter_Node to allow more complex tests to have their own Node class (the default filter has been converted to use this new feature)

removed a function that is not used anymore

migrated most tests to proper Node classes

moved the defined test to a Twig_Test_Node class

added Twig_Test_Node to allow more complex tests to have their own Node class (sameas has been converted as an example)

added a better error message when a template is empty but contain a BOM

optimized tokenizer by pre-computing all start tag positions (memory and CPU)

optimized a comparison in the tokenizer

added notes in the doc about how the defined test and the default filter work

merged branch nikic/allowIsDefinedOnMethods (PR #490)

Commits
-------

930330f Allow defined test and default filter on methods

Discussion
----------

Allow defined test and default filter on methods

Patch for #487.

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

by fabpot at 2011/10/30 03:45:49 -0700

Problem is when some of the arguments reference variables that do not exist (`foo.bar(bar.foobar)`). That's why I've added the check on arguments.

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

by nikic at 2011/10/30 03:58:58 -0700

@fabpot I'm not exactly sure what is the expected behavior in that case. When writing `foo.defined(undefined)|default()` I think an exception is the expected behavior (because there is an undefined variable without default, even if it is in the method arguments). With `foo.undefined(undefined)|default()` on the other hand I'm not sure. One could argue that as the method does not exist the arguments never need to be evaluated.

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

by fabpot at 2011/10/30 05:49:46 -0700

The same code is used for the `defined` test: what would be the expected behavior for `foo.defined(underfined) is defined`?

merged branch nikic/patch-1 (PR #489)

Commits
-------

7da9f2f Make long var/block tests faster

Discussion
----------

Make long var/block tests faster

When using `*` every single character is a different token, whereas with `x` there is only a single token. I don't think this hurts the actual test, but it improves the test runtime by something like 2/3 seconds on my machine.

Allow defined test and default filter on methods

Make long var/block tests faster

fixed previous commit

renamed none to null in doc

changed the documentation to advertize null as begin the null value and none as an alias

fixed in operator for empty strings

merged branch nikic/updateExtensionDocs (PR #488)

Commits
-------

2ebc788 Update extension docs and interface

Discussion
----------

Update extension docs and interface

Followup to #484: Twig isn't using globals for functions anymore.

Update extension docs and interface

Twig isn't using globals for functions anymore.

fixed typo in doc

merged branch phmLabs/documentation_return (PR #483)

Commits
-------

389f2c2 Changed mode from 0755 to 0644
c48bd2f Fixed wrong used PHPDoc tags

Discussion
----------

Fixed wrong used PHPDoc tags

@param was used instead of @return

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

by fabpot at 2011/10/25 05:47:59 -0700

Can you revert the mode change (from 0644 to 0755)?

added more unit tests for defined and default

fixed the "defined" test and the "default" filter

added more unit tests (some do not pass... yet)

added missing supported charset for htmlspecialchars

fixed deep nested defined tests

removed unneeded complexity in GetAttr node

added a way to change Twig's options when running integration tests (updated tests accordingly)

Revert "fixed exception"

This reverts commit df47659e9de71a48a24f31ed0cc16a2674a06d43.

fixed example in doc (closes #484)

Changed mode from 0755 to 0644

added some references to the doc

Fixed wrong used PHPDoc tags

reverted mode change from previous commit

merged branch phmLabs/Twig_NodeInterface_Bug (PR #482)

Commits
-------

c91357d Moved interfaces

Discussion
----------

Moved interfaces

The Countable and IteratorAggregate interfaces have to be part of the Twig_NodeInterface as the Twig_NodeInterface is used as a typehint and the function tries to iterate over its elements.

E.g:

Twig_NodeTraverser::traverseForVisitor

foreach ($node as $k => $n)

Revert "added a tip for the block tag"

This reverts commit 1a2175fc4992b74a473845eab0972d464086398f.

Moved interfaces

added a tip for the block tag

merged branch yegeniy/patch-1 (PR #474)

Commits
-------

cf03a65 add another example add another example to constant.rst

Discussion
----------

add another example

An example for using the `constant` function

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

by yegeniy at 2011/10/19 09:48:37 -0700

Should be ready.

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

by nikic at 2011/10/19 10:58:01 -0700

I think you should just add that as a line to the other example, so you got this in the end:

```rst
``constant``
============

``constant`` returns the constant value for a given string:

.. code-block:: jinja

    {{ some_date|date(constant('DATE_W3C')) }}
    {{ constant('Namespace\\Classname::CONSTANT_NAME') }}
```

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

by fabpot at 2011/10/21 07:24:32 -0700

That's indeed a good addition to the doc. Can you squash your commits in to one before I merge this pull request? Thanks.

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

by yegeniy at 2011/10/22 06:28:58 -0700

Thanks!

fixed typos

changed the way extensions are loaded (addFilter/addFunction/addGlobal/addTest/addNodeVisitor/addTokenParser/addExtension can now be called in any order)

add another example
add another example to constant.rst

added Twig_Environment::display()

simplified some code

added more unit tests

added an optimization when the same temporary variable would have been created twice

fixed doc

made the escape filter smarter when the encoding is not supported by PHP

added a convert_encoding filter

added a Twig_Node_Expression_DefaultFilter to replace the current way of dealing with default filters

added public getters on the environment to be able to get various stuff

fixed exception

fixed download page in the docs

merged branch trompette/patch-1 (PR #465)

Commits
-------

0451cfd typo

Discussion
----------

typo

typo

updated CHANGELOG

refactored Twig_Node_Expression_GetAttr for better readability of compiled templates

optimized variable access when possible

refactored 'defined' test and 'default' filter

Node manipulations have been moved outside the compile() method.
That's needed if we want to be able to visit all nodes that will be
compiled.

optimized lexer by pre-computing regexes

made an optimization by removing array_merge usage

bumped version to 1.4.0-DEV

prepared the 1.3.0-RC1 release

updated CHANGELOG

prepared the 1.3.0-RC1 release

fixed a unit test for PHPUnit 3.6

fixed optimizer