#
5cd2630a |
| 24-Dec-2018 |
Peter Kokot |
Add dependency injection container This patch introduces a dependency injection container for the PHP bug tracker application. Container deals with the creation of all service classe
Add dependency injection container This patch introduces a dependency injection container for the PHP bug tracker application. Container deals with the creation of all service classes and additionally provides retrieving of configuration parameters. Service classes are used everywhere in the app - from accessing database to uploading files. Configuration parameters include infrastructure configuration (database credentials...) and application level configuration (directories locations...). Container is compatible with the PSR-11 container interface defined so it is simple to quickly understand its usage. Advanced features such as autowiring are not included in this phase.
show more ...
|
#
e3c4b0ac |
| 23-Jan-2019 |
Peter Kokot |
Replace deprecated Text_Diff PEAR package with Horde Text_Diff This patch replaces obsolete Text_Diff PEAR package with newer and still maintained Horde Text_Diff version 2.2.0 as sugges
Replace deprecated Text_Diff PEAR package with Horde Text_Diff This patch replaces obsolete Text_Diff PEAR package with newer and still maintained Horde Text_Diff version 2.2.0 as suggested in the packages descriptions. Since bugs.php.net production is not ready for Composer installations yet, the Text_Diff package is bundled in the Git repository directly. Its classes are not compliant with PSR-4 yet so the classmap has been used until future PSR-4 migration and refactorings.
show more ...
|
#
068d8514 |
| 17-Dec-2018 |
Peter Kokot |
Add template engine This patch adds an initial simplistic template engine to separate logic from the presentation. Basic initial features: - escaping via Context::noHtml() a
Add template engine This patch adds an initial simplistic template engine to separate logic from the presentation. Basic initial features: - escaping via Context::noHtml() and Context::e() methods - blocks - nesting options using includes and extending layouts - PHP syntax - variable scopes dedicated to template scope only - Appending blocks (when JS files are in need to be appended) - initial unit and functional tests - Main index page refactored as an example of usage - Very short intro docs how to use the template layer - Thanks to @nhlm for the code review and numerous suggestions to improve the usability and code stability, - Thanks to @KalleZ and for the code review and numerous common sense suggestions about templates themselves. - Thanks to @Maikuolan for the code review and numerous suggestions about the usability. - Moved hash ids redirection to aseparate JavaScript file - Use location instead of window.location in the JavaScript redirection Discussions: - http://news.php.net/php.webmaster/27603 - https://github.com/php/web-bugs/pull/66
show more ...
|
#
14f8c07a |
| 14-Dec-2018 |
Peter Kokot |
Refactor PDO wrapper Some considerations were raised on the mailing list that this PHP application doesn't need a PDO wrapper at all. Changes: - ::fetchRow() method removed
Refactor PDO wrapper Some considerations were raised on the mailing list that this PHP application doesn't need a PDO wrapper at all. Changes: - ::fetchRow() method removed in favor of the vanilla PDOStatement::fetch() - ::fetchAll() override removed in favor of the vanilla PDOStatement::fetchAll() - ::fetchCol() removed since it is not used and is only a wrapper for the PDOStatement::fetchColumn() - PDO fetch_style synced accross the app. When no fetch style is passed the default PDO::FETCH_ASSOC is used as set when connecting to db. - Remove Database::escape() method The custom ::escape() method is a wrapper around PDO::quote() which additionally trims leading and trailing quotes from the string. All this should ideally be done via prepared statements only, except where we can and need to use PDO::quote() this step can be done on the given string or variable at hand directly. - Remove escapeSQL() function The escapeSQL function is a wrapper around the PDO::quote() and is using $dbh from the global scope which is not testable nor good practice further on. Removed and refactored into only PDO::quote() usages on required places. - Remove ::fetchOne() method The fetchOne() method is a simple wrapper around the PDOStatement::fetch() method with very minor tweaks so the usage can be simplified even more. The PDOStatement::fetch(\PDO::FETCH_NUM)[0] will always return either a result from the database column or when row is empty a null. - Probably this should be refactored to the database tables respected repositories further on. - Remove PDO wrapper The app's current goal is to lean on a vanilla PDO wrapper only. Current set of features also don't require additional functionality and extending PDO to a wrapper or create a database abstraction layer yet.
show more ...
|
#
73062503 |
| 05-Dec-2018 |
Peter Kokot |
Refactor database classes
|
#
3f343755 |
| 05-Dec-2018 |
Peter Kokot |
Fix typo
|
#
d337e731 |
| 05-Dec-2018 |
Peter Kokot |
Add dual PSR-4 compatible classes autoloader This patch is a workaround until Composer installation step can be used in production bugs.php.net. Once Composer can be added to the deploym
Add dual PSR-4 compatible classes autoloader This patch is a workaround until Composer installation step can be used in production bugs.php.net. Once Composer can be added to the deployment step together with rsync this can be simplified and only Composer's autoload.php will be used.
show more ...
|
#
b4891d7b |
| 05-Dec-2018 |
Peter Kokot |
Enable displaying errors for local development environments
|
#
83d6860a |
| 03-Dec-2018 |
Niklas Keller |
Add Composer This patch adds initial Composer files and prepares the project for using Composer in development environment. Patch has been initially started via pull request at
Add Composer This patch adds initial Composer files and prepares the project for using Composer in development environment. Patch has been initially started via pull request at https://github.com/php/web-bugs/pull/27
show more ...
|
#
a06f85b0 |
| 24-Oct-2018 |
Peter Kokot |
Refactor long array() syntax to short [] Since site is using PHP 5.4+ already, the longer `array()` syntax can be refactored to shorter `[]`. Also code is already using short array s
Refactor long array() syntax to short [] Since site is using PHP 5.4+ already, the longer `array()` syntax can be refactored to shorter `[]`. Also code is already using short array syntax on some places.
show more ...
|
#
8249fa53 |
| 24-Oct-2018 |
Peter Kokot |
Remove unused configuration db_extension The `db_extension` was once used with PEAR's MDB2. With migration to vanilla PDO, this is not used anymore.
|
#
21e0f760 |
| 23-Oct-2018 |
Peter Kokot |
Change dirname(__FILE__) to shorter __DIR__
|
#
23298a12 |
| 17-Jul-2018 |
Rasmus Lerdorf |
New bugs.php.net code
|
#
f7cc97c2 |
| 18-May-2014 |
Sobak |
PHP 5.4 is minimal required version
|
#
c3110c33 |
| 27-Jan-2012 |
Rasmus Lerdorf |
- The end of an era - switch "Bogus" to "Not a bug" by popular demand - Switch back to master - Add site_method to make it easier to install on non-ssl test sites
|
#
1258b9d5 |
| 25-Mar-2011 |
Felipe Pena |
- Trying to fix the mail sending yet..
|
#
5bec6592 |
| 19-Nov-2010 |
Felipe Pena |
- Added "Security" bug type to control Security bug instead of a package name
|
#
d625bbf4 |
| 02-May-2010 |
Derick Rethans |
- Commit this local change.
|
#
df6ee6d3 |
| 18-Apr-2010 |
Daniel Egeberg |
Fixed PHP bug #51586 (Wrong mailing list for "Documentation Problem" type bugs)
|
#
7b7c024d |
| 16-Mar-2010 |
Jani Taskinen |
- Enable compression
|
#
945beb38 |
| 10-Mar-2010 |
Philip Olson |
Handle PEAR errors, in a moderately graceful manner
|
Revision tags: BEFORE_NEW_BUG_TRACKER |
|
#
911112b7 |
| 05-Mar-2010 |
Jani Taskinen |
- Moved default patch dir to somewhere where they might be stored over reboots too.
|
#
8452f1d0 |
| 03-Mar-2010 |
Jani Taskinen |
- Get rid of local_config for the "main" site
|
#
495c8569 |
| 22-Feb-2010 |
Jani Taskinen |
- Allow using different db ext
|
#
bab47912 |
| 25-Aug-2009 |
Jani Taskinen |
- More totally unnecessary whitespace changes.
|