Revisions for framework/Template/Template.php

MFH 1.75

Tags: HORDE_3_3, HORDE_3_3_RC1, HORDE_3_2_2, HORDE_3_2_1, HORDE_3_2, HORDE_3_2_RC4

Restore case-loop feature (cloop) for BC

Happy New Year

Tags: HORDE_3_2_RC3, HORDE_3_2_RC2

Happy New Year

Merge from HEAD.

Happy New Year

Tags: HORDE_3_1_9, HORDE_3_1_8, HORDE_3_1_7, HORDE_3_1_6, HORDE_3_1_5, HORDE_3_1_4, HORDE_3_1_4_RC1

Happy New Year

Tags: BEFORE_MERGE_3_2, HORDE_3_2_RC1, HORDE_3_2_ALPHA

Assign copyright to The Horde Project for those copyright holders that already agreed to it.

Restore previous Horde_Template class until Horde 4.0.

ack! again!

Fix divider output.  Add divider test.

whoops - keep debug code on for now.

Remove nested tags test for now.  Convert to am more complete associative
array test instead.  Horde_Template:: now passes all tests.

After heavily testing for several days, all templates I can find in Horde
applications are working.  So go ahead and commit the new Template code.
Drastically reduces template display time (approx 4x - 7x), with additional
improvements once we turn on caching code (which is currently disabled).
Refactor some tests to account for some tiny differences in new output (mostly
dealing with whitespace output, which makes no difference when it comes to
actual display, and also with the fact we no longer output empty tags to HTML -
they are silently discarded instead).  Only one test is currently failing - the
nested tags - but nested tags don't seem to currently be used anywhere in
Horde at the present.

Revert revision 1.64. This broke the ability to pass an iterator in to
the template, and using iterators (which let us only fetch and loop
through data _once_, ever) should be a much more significant memory
savings than foreach vs. each. If this really made a large difference,
I can move it into a PHP 5 version for now, but I'd like some evidence
of that first.

Use each() to access array loops in the hopes of lowering memory usage.

Add some notes on design and dev philosophy

- block parsing has been split to Giapeto
- remove resetVars option

remove unused resetVars() function

remove cloops feature

Remove numeric array key constraint (Request #4413).

MFH:
1.58      +9 -9      framework/Template/Template.php
1.586     +12 -11    framework/Horde/Horde.php

Tags: HORDE_3_1_3, HORDE_3_1_2

Allow objects where we allow arrays. This way we can pass an
Iteratable object into a template without having to loop through it
twice.

- don't silence file* calls when they really shouldn't fail (ie,
failing is a programmatic error, not a user/runtime error)

- rework factories to only include class files when the class doesn't
already exist (always allow a custom class to be included _before_ the
factory is called).

MFH: 1.56

Strict check if _getStatement() returns false, it might return an empty string
which evaluates to false.

Bug: 3828

MFH 1.55

Tags: HORDE_3_1_1, HORDE_3_1, HORDE_3_1_RC3, HORDE_3_1_RC2

avoid undefined index errors

MFH 1.54

This should allow if statements in nested loops deeper than 1 level.

Happy new year

Tags: HORDE_3_1_RC1

Happy new year\!

Merged from HEAD.

MFH: Use double-quotes.

Tags: HORDE_3_0_12, HORDE_3_0_11, HORDE_3_0_10, HORDE_3_0_9, HORDE_3_0_8, HORDE_3_0_7, HORDE_3_0_6, HORDE_3_0_6_RC1

Use double-quotes.

Tags: BEFORE_MERGE_3_1

Allow calling listBlocks() when using a non-file template.

MFH: Clean up phpdoc.

Tags: HORDE_3_0_5, HORDE_3_0_5_RC2, HORDE_3_0_5_RC1

Clean up phpdoc.

Remove @version.

apparently I never committed this... idea was to avoid setting
$this->_templateFile if the load failed. This should do that and also
work.

Don't know if this is the right fix or not, but without this line all templates
are horribly broken.

phpdoc, @access tags, use file_get_contents()

- get rid of @version
- phpdoc tweaks
- _getSource() -> _getTemplate()

* Template::getCSSApps() -> listBlocks()

* Avoid loading template source more than once.

* Add 'blocks' template option and don't process blocks by default.

<b>/<i> to <strong>/<em>, and remove bold tags when already in header, clean up some bad html parsing

* Mechanism to retrieve list of block applications from template.
* In Giapeto, bring in CSS for blocks in template.

* Allow Horde templates to include application blocks.

Happy New Year

Happy New Year

Tags: HORDE_3_0_4, HORDE_3_0_4_RC2, HORDE_3_0_4_RC1, HORDE_3_0_3, HORDE_3_0_3_RC1, HORDE_3_0_2, HORDE_3_0_1

Require PEAR.php.

Tags: FRAMEWORK_3, HORDE_3_0, HORDE_3_0_RC3

Inside loops replace <tag:loopvar /> with the loop's keys.

Add get().

Doc fixes.

Prefix private methods with underscores and put public methods first.

doc fixes.

We used String:: methods here to avoid inifinite loops, but instead they seem
to cause them. Regular string functions should be save here because we only
extract tags that are ascii anyway, so we shouldn't have problems with
string boundaries inside multibyte characters. That's the theory.

Bug: 189

Make sure $i isn't undefined here.

Tags: HORDE_3_0_ALPHA

spacing

Happy new year everyone.

Add loop dividers to Horde_Template, and use them to make some sets of links
a bit nicer and more useable in Agora.

get some test files, and rearrange a few things. What package some of these classes
fall under is entirely debatable; I just want to get everything into a structure that can be worked with.

package name

From Nuno Loureiro <nuno@co.sapo.pt>:

- elses were not working in arrays
- this is kind of hard do explain, but it was only parsing ifs in arrays
once, so the first array element ruled for the whole loop. It's probably
easier to look to the code to understand..

Take out explicit casting - we already store the variable in either $this->_arrays
or $this->_scalars, and PHP's reference counting means that if we don't modify
it, assigning it to $this->_ifs as well won't use any more memory at all.

style was still wrong here.

allow array if conditions (Nuno Loureiro <nuno@co.sapo.pt>)

Replace all tags in a single call to str_replace.

typo

better error message

[cjh] Horde_Template:: is now capable of translating text inside
<gettext></gettext> tags.

Submitted by: Nuno Loureiro <nuno@co.sapo.pt>

return a better error when we can't open a template

- Use String::length() and String::pos() to avoid potential infinite loops.
- Store $_ifs as straight booleans for clarity.

Use String::substr()

Bump year.

Put the while in the correct place fixing the parse error.

Put this back, my bad. It is now a slightly documented kludge. :)

reverting this. this is a kludge; we shouldn't be doing things with
side effects like this. if no loops are set, we don't process loops.

Make sure the loop runs at least once so array values in <tag>s get parse if there are no loops in the template.

Loop variables can now be re-used in different contexts - you don't
have to make each use of a loop be exactly identical to others in the
same template.

fix bug with not finding loops at the very start (first character) of
a file. classic strpos error checking problem.

Minor style bits.

Change the way ifs are stored so that you can use an array as a conditional. Allows things like this:

$template = &new Horde_Template();
$template->set('search_results', $search_results, true);
echo $template->parse('
    <if:search_results>
        <loop:search_results>
            <tag:search_results />
        </loop:search_results>
    <else:search_results>
        No Records Found
    </else:search_results>
    </if:search_results>
');

Better way of removing the else block.

Remove Not If's from last commit. Implement else statements in the form:
<if:conditional>
  Output this if true
<else:conditional>
Output this if false
</else:conditional>
</if:conditional>

Add suport for not ifs. You can now do this:
<if:!conditional>
  Stuff to output if conditional isn't true.
</if:!conditional>

Change the basepath to ''.

Add a simple template engine, derived from bTemplate, for Horde
applications to use.