Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | [feature/template-engine] Replaced globals with dependency injection. | Oleg Pudeyev | 2011-05-12 | 4 | -39/+65 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Deleted useless catch. | Oleg Pudeyev | 2011-05-12 | 1 | -4/+0 |
| | | | | | | In tests there is no need to catch unexpected exceptions. PHPBB3-9726 | ||||
* | [feature/template-engine] Delete obsolete comments pertaining to notices. | Oleg Pudeyev | 2011-05-12 | 1 | -7/+0 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Clarify cache directory path for set_custom_template | Oleg Pudeyev | 2011-05-12 | 1 | -1/+4 |
| | | | | | | | Even if the template may be outside of phpBB, phpBB's cache directory is still going to be used for storing compiled template code. PHPBB3-9726 | ||||
* | [feature/template-engine] Try to handle failed template includes. | Oleg Pudeyev | 2011-05-12 | 1 | -2/+3 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Rename $filename to $compiled_path for clarity. | Oleg Pudeyev | 2011-05-12 | 1 | -6/+6 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Move DEBUG_EXTRA check for $recompile up. | Oleg Pudeyev | 2011-05-12 | 1 | -14/+10 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Deleted useless assignment. | Oleg Pudeyev | 2011-05-12 | 1 | -1/+0 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Added docblocks to get_*_ref in context. | Oleg Pudeyev | 2011-05-12 | 1 | -0/+22 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Fixed reference usage. | Oleg Pudeyev | 2011-05-12 | 3 | -3/+7 |
| | | | | | | | Hopefully this is right, I have not checked it against the manual (assuming the manual even covers these things). PHPBB3-9726 | ||||
* | [feature/template-engine] Disposed of underscores in property names. | Oleg Pudeyev | 2011-05-12 | 2 | -32/+32 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Remove commented out error reporting logic. | Oleg Pudeyev | 2011-05-12 | 1 | -10/+0 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Refactor hook logic into a separate function. | Oleg Pudeyev | 2011-05-12 | 1 | -7/+24 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Reinstate phpbb_template#destroy function. | Oleg Pudeyev | 2011-05-12 | 1 | -0/+9 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Add back IN_PHPBB preamble. | Oleg Pudeyev | 2011-05-12 | 3 | -1/+53 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Moved template classes to subdirectory. | Oleg Pudeyev | 2011-05-08 | 5 | -0/+0 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Renamed template executor and friends to renderer. | Oleg Pudeyev | 2011-05-08 | 4 | -33/+33 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Added docblocks and boilerplate to new files. | Oleg Pudeyev | 2011-05-07 | 4 | -0/+76 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Removed more dead code. | Oleg Pudeyev | 2011-05-07 | 1 | -6/+0 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Added phpbb_template_context class. | Oleg Pudeyev | 2011-05-07 | 6 | -246/+367 |
| | | | | | | Objects of this class hold variables assigned to templates. PHPBB3-9726 | ||||
* | [feature/template-engine] Always call ob_end_clean. | Oleg Pudeyev | 2011-05-05 | 1 | -1/+1 |
| | | | | | | We have to stop output buffering even when rendering fails. PHPBB3-9726 | ||||
* | [feature/template-engine] Removed some dead code. | Oleg Pudeyev | 2011-05-04 | 1 | -3/+1 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Split template execution logic into classes. | Oleg Pudeyev | 2011-05-04 | 4 | -63/+123 |
| | | | | | | | | | | | Template executor interface defines a template executor object. It is an object which can execute (i.e. display/render) a template. Currently there are two implementations: * phpbb_template_executor_include includes php code from a file. * phpbb_template_executor_eval eval's php code. PHPBB3-9726 | ||||
* | [feature/template-engine] Removed storedb-related logic. | Oleg Pudeyev | 2011-05-04 | 1 | -124/+2 |
| | | | | | | | | | phpBB 3.1 will not provide the option to store templates in the database. This commit removes code that handles templates stored in database from the template engine. PHPBB3-9726 | ||||
* | [feature/template-engine] Delete template class, use phpbb_template instead. | Oleg Pudeyev | 2011-05-04 | 5 | -13/+4 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Rename template_compile methods for clarity. | Oleg Pudeyev | 2011-05-04 | 3 | -12/+12 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Extracted compile_stream_to_stream. | Oleg Pudeyev | 2011-05-01 | 1 | -4/+19 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Close output stream in compile(). | Oleg Pudeyev | 2011-05-01 | 1 | -1/+4 |
| | | | | | | | There is no need to leave the stream to the garbage collector, and the amount of data stuck in it may be substantial. PHPBB3-9726 | ||||
* | [feature/template-engine] Fixed description of assign_display(). | Oleg Pudeyev | 2011-05-01 | 1 | -1/+1 |
| | | | | | | | This function returns false on failure, which can happen if display() failed. Document the failure return value. PHPBB3-9726 | ||||
* | [feature/template-engine] Only copy files to cache if they exist. | Oleg Pudeyev | 2011-05-01 | 1 | -2/+3 |
| | | | | | | | | When testing eval code path, compiled templates may not be written to the file system, and unconditionally trying to copy them breaks tests. PHPBB3-9726 | ||||
* | [feature/template-engine] Relax missing file exception check. | Oleg Pudeyev | 2011-05-01 | 1 | -1/+1 |
| | | | | | | | As long as the exception message has the correct description and file name we should not care which function raised the exception. PHPBB3-9726 | ||||
* | [feature/template-engine] Check return value of display() in assign_display(). | Oleg Pudeyev | 2011-05-01 | 1 | -1/+5 |
| | | | | | | If display() failed, propagate the failure out of assign_display(). PHPBB3-9726 | ||||
* | [feature/template-engine] Removed $this from new phpbb_template_compile calls. | Oleg Pudeyev | 2011-04-26 | 1 | -3/+3 |
| | | | | | | The compile class no longer takes template as a parameter. PHPBB3-9726 | ||||
* | [feature/template-engine] Fix negative variable expressions | Igor Wiedler | 2011-04-25 | 4 | -13/+9 |
| | | | | | | | | | | | | | compile_tag_if had the flawed approach of adding an isset statement for all variables to the beginning of the if. This fails for negative expressions, and checking those takes a considerable effort. The easier solution is to make the variable expression itself conditional, defaulting to null if it is not set. Thanks to naderman for the solution. PHPBB3-9726 | ||||
* | [feature/template-engine] Added a test for reuse of loop identifiers. | Oleg Pudeyev | 2011-04-25 | 2 | -0/+16 |
| | | | | | | | | | This currently does not pass, thus it is commented out. The reuse appears implausible in the same file, however it may be also done across template files where it is much harder to detect. PHPBB3-9726 | ||||
* | [feature/template-engine] Corrected miscompilation of loop size constructs. | Oleg Pudeyev | 2011-04-25 | 3 | -1/+47 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Created a script to compile templates. | Oleg Pudeyev | 2011-04-24 | 1 | -0/+24 |
| | | | | | | | Script takes path to template as the only argument and outputs the compiled template to standard output. PHPBB3-9726 | ||||
* | [feature/template-engine] Corrected an off-by-one error in nested namespaces. | Oleg Pudeyev | 2011-04-24 | 4 | -4/+29 |
| | | | | | | | | This error resulted in a dot from the namespace being placed into variable reference in compiled template code, thus creating bogus compiled template code. PHPBB3-9726 | ||||
* | [feature/template-engine] Added a test for multilevel references in loops. | Oleg Pudeyev | 2011-04-24 | 2 | -0/+24 |
| | | | | | | | | | This currently fails. This test is a reduced version of permission_mask template in acp, which is not correctly compiled by the current template engine code. PHPBB3-9726 | ||||
* | [feature/template-engine] Deleted silencing of notices. | Oleg Pudeyev | 2011-04-24 | 2 | -3/+5 |
| | | | | | | | The code is now supposed to be notice-free, therefore there is no need to have the notices silenced. PHPBB3-9726 | ||||
* | [feature/template-engine] Deleted $template from phpbb_template_compile class. | Oleg Pudeyev | 2011-04-24 | 3 | -96/+74 |
| | | | | | | | | | | | | | phpbb_template_compile is now much simpler. It takes complete file paths as inputs, either source template path or source template path and output compiled template path. The number of methods also went down to two - compile template and returned compiled text or compile and write to file. phpbb_compile class is responsible for determining source and compiled paths. It already had all the data necessary for this, now the code is in the same place as the data it uses. PHPBB3-9726 | ||||
* | [ticket/9924] Pass template instance into $template->display hook | Igor Wiedler | 2011-04-24 | 1 | -1/+1 |
| | | | | | | | This is a cherry-pick of 053cf790a93e9cfb521f484901d79c72783f868f which appears to have been partially reverted here. PHPBB3-9924 | ||||
* | [feature/template-engine] Removed a useless space. | Oleg Pudeyev | 2011-04-24 | 1 | -1/+1 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Delete class_exists checks, rely on autoloading. | Oleg Pudeyev | 2011-04-24 | 1 | -10/+0 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Allow leading underscores in variable names. | Oleg Pudeyev | 2011-04-24 | 3 | -3/+33 |
| | | | | | | Subsilver uses ._file in overall_header. PHPBB3-9726 | ||||
* | [feature/template-engine] Fix recompilation logic. | Oleg Pudeyev | 2011-04-24 | 2 | -11/+14 |
| | | | | | | | | | Do not change $recompile from true to false - any recompilation condition alone is sufficient to force recompilation. Also uncomment the nonexistent file test which passes with this fix. PHPBB3-9726 | ||||
* | [feature/template-engine] Adjust path in includephp template. | Oleg Pudeyev | 2011-04-24 | 1 | -1/+1 |
| | | | | | | Now that tests are run from top level the '..' is wrong. PHPBB3-9726 | ||||
* | [feature/template-engine] Delete ?>, add newline at EOF. | Oleg Pudeyev | 2011-04-23 | 2 | -3/+1 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Improved template engine. | Marek A. Ruszczynski | 2011-04-23 | 3 | -877/+1204 |
| | | | | PHPBB3-9726 | ||||
* | [feature/template-engine] Update tests. | Marek A. Ruszczynski | 2011-04-23 | 5 | -55/+30 |
| | | | | PHPBB3-9726 |