Activity

From 2012-10-23 to 2012-11-21

2012-11-21

22:04 Bug #42723: Reflection\ObjectAccess consumes to much processing time
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16234
Gerrit Code Review
22:02 Bug #42723: Reflection\ObjectAccess consumes to much processing time
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16234
Gerrit Code Review
17:56 Bug #42955: QueryResult::count() returns 0 for queries with offset
Bastian Waidelich wrote:
> Probably the right way to do this would be to really count the results as soon as an offs...
Bastian Waidelich
17:18 Bug #42955: QueryResult::count() returns 0 for queries with offset
Karsten Dambekalns wrote:
> Didn't try it, but it sounds reasonable. Whatever happens inside Doctrine there
> res...
Bastian Waidelich
16:57 Bug #42955: QueryResult::count() returns 0 for queries with offset
Bastian Waidelich wrote:
> @Karsten can you verify this?
Didn't try it, but it sounds reasonable. Whatever happen...
Karsten Dambekalns
16:50 Bug #42955: QueryResult::count() returns 0 for queries with offset
Ok, the actual issue with this is that the @Doctrine/QueryResult::count()@ throws an exception if the underlying quer... Bastian Waidelich
16:50 Bug #42955: QueryResult::count() returns 0 for queries with offset
Ok, the actual issue with this is that the @Doctrine/QueryResult::count()@ throws an exception if the underlying quer... Bastian Waidelich
17:35 Bug #43195 (Resolved): getLocalizedFilename() chokes on paths with dots.
Applied in changeset commit:fefb00eadefe768c1e8415fc6faac6b1f84c81bc. Karsten Dambekalns
14:31 Bug #43195: getLocalizedFilename() chokes on paths with dots.
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16664
Gerrit Code Review
13:29 Bug #43195 (Under Review): getLocalizedFilename() chokes on paths with dots.
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16664
Gerrit Code Review
13:22 Bug #43195 (Resolved): getLocalizedFilename() chokes on paths with dots.
When a path like @resource://TYPO3.Neos/Public/JavaScript/@ is handed to @getLocalizedFilename()@, it tries to check ... Karsten Dambekalns
16:17 Feature #31777: SubRoutes should allow routes starting or ending with /
Bastian Waidelich wrote:
> I'll look into this asap.
as I wrote, as spät as possible ;) sorry for the delay
Bastian Waidelich
16:16 Feature #31777: SubRoutes should allow routes starting or ending with /
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16671
Gerrit Code Review
16:15 Feature #31777 (Under Review): SubRoutes should allow routes starting or ending with /
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16671
Gerrit Code Review
15:14 Feature #37765 (Resolved): Display detailed informations within the "Could not resolve a route" exception why FLOW3 URI building has failed.
This has been (mainly) resolved with #42135
The exception now looks like...
Bastian Waidelich
14:30 Bug #28862: Inconsistent handling of roles as objects and strings in Account
Patch set 11 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/13868
Gerrit Code Review
12:04 Bug #28862: Inconsistent handling of roles as objects and strings in Account
Patch set 10 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/13868
Gerrit Code Review
14:10 Bug #28672 (Under Review): Inconsistent configuration overrides for packages
Patch set 7 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16328
Gerrit Code Review
14:04 Task #37451: Rework usage of Roles in the security framework
What those changes are missing as of today - to fully replace the idea of the @AuthorizationGroup@ - is an @addRole()... Karsten Dambekalns
14:02 Task #37451 (Under Review): Rework usage of Roles in the security framework
See https://review.typo3.org/#/q/topic:roles,n,z Karsten Dambekalns
12:27 Feature #43193: Optimize Classloader
Christopher Hlubek wrote:
> Not exactly (there is no frozen code cache), but I also didn't notice, that for code cac...
Robert Lemke
12:23 Feature #43193: Optimize Classloader
Not exactly (there is no frozen code cache), but I also didn't notice, that for code caches the @SimpleFileBackend@ i... Christopher Hlubek
12:18 Feature #43193 (Rejected): Optimize Classloader
That's what freezable cache backends are for. In Production Flow uses a freezable cache backend for code caches and f... Robert Lemke
12:14 Feature #43193 (Rejected): Optimize Classloader
The default FileBackend does an expiration check on any call to require, which costs quite some syscalls. For the cla... Christopher Hlubek
11:55 Bug #43192 (Accepted): findByIdentifier() for non-persisted objects not working for custom identifier properties
If an object does not get it's identifier via AOP, it will not be returned "early" from @findByIdentifier()@.
Idea...
Karsten Dambekalns
10:42 Bug #43190 (Accepted): Misleading exception message for incompatible database structure
Karsten Dambekalns
10:38 Bug #43190 (Accepted): Misleading exception message for incompatible database structure
If the database structure doesn't match the current schema (for example, if a table has not been created yet), a user... Robert Lemke

2012-11-20

16:35 Bug #43110 (Resolved): FlowSession: renewId() looses data of existing session
Applied in changeset commit:1b6267f73f9ab74ba4a6ba719e078ce530bf1b6f. Robert Lemke

2012-11-19

18:03 Bug #43110 (Under Review): FlowSession: renewId() looses data of existing session
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16575
Gerrit Code Review
17:59 Bug #43110 (Resolved): FlowSession: renewId() looses data of existing session
Using FlowSession, on renewing the session identifier (for example after an authenticate() call), some session data s... Robert Lemke
17:55 Feature #42366 (Rejected): Make session handling compatible to any PHP session handler
This feature request has been dropped in favor of a more capable FlowSession implementation. Robert Lemke
17:49 Feature #43091 (Under Review): Add logging of vote results on resources
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16573
Gerrit Code Review
11:29 Feature #43091 (Resolved): Add logging of vote results on resources
For access decision voting on joinpoints the results are logged nicely. For votes on resources this is missing. Karsten Dambekalns
17:46 Feature #43089 (Under Review): Allow configuration processing in code migrations
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16571
Gerrit Code Review
10:36 Feature #43089 (Resolved): Allow configuration processing in code migrations
Code migrations sometimes need to check or even modify YAML configuration. Karsten Dambekalns
16:44 Feature #43104: Add Confirm- Radio- CheckBoxDialog to CommandController(CLI)
Rafael, thanks for your submission.
The functionality generally is nice (i.e. the idea); I would say it could be an ...
Adrian Föder
16:25 Feature #43104 (Rejected): Add Confirm- Radio- CheckBoxDialog to CommandController(CLI)
For interactive confirmation or multiple choice by Commands (CLI).
for example: the same code is on https://gist.g...
Rafael Kähm
16:34 Feature #40418 (Rejected): Add an option to flow3:cache:flush thats keeps user sessions active
Closing according to comments on https://review.typo3.org/#/c/15846/
Martin Brüggemann, Nov 8:
> [...] Active S...
Bastian Waidelich
15:47 Feature #37854: Refactor ConfigurationManager to Support Custom Configuration Files
Patch set 8 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/11982
Gerrit Code Review
15:47 Feature #42176: Improve resolving of view
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16392
Gerrit Code Review

2012-11-18

21:39 Feature #43082 (Needs Feedback): Add CLI support for scaffolding models, views, controller
Add a cli mechanism to quickly generate code scaffolds on the command line.
Some examples on how it could be used:...
Ingo Pfennigstorf

2012-11-17

01:52 Feature #42176: Improve resolving of view
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16392
Gerrit Code Review

2012-11-16

14:29 Feature #37188 (Closed): Global verbosity flags for CLI Commands
Duplicate of #42452 Bastian Waidelich
14:28 Task #42452: Command handling should support global flags
Bastian Waidelich wrote:
> If "--context" / "-c" is set, the command is transparently executed in that context [.....
Bastian Waidelich

2012-11-14

12:28 Feature #42961 (Resolved): Support non PSR-0 packages
Composers autoloader supports non PSR-0 packages if they follow the so called "PEAR-style" (see http://getcomposer.or... Bastian Waidelich
12:25 Bug #42955: QueryResult::count() returns 0 for queries with offset
Okay, this variant seems to be working... but why? ^^
now I need to rewrite the paginate template ;)
thanks for...
René Pflamm
12:08 Bug #42955: QueryResult::count() returns 0 for queries with offset
Hi,
thanks for the details.
I need to test this, but could you try the following for now:...
Bastian Waidelich
12:08 Bug #42955: QueryResult::count() returns 0 for queries with offset
I found out, an Exception is logged:
@Uncaught exception in line 106 of /myPath/Packages/Framework/Doctrine.ORM/C...
René Pflamm
11:58 Bug #42955: QueryResult::count() returns 0 for queries with offset
_The controller:_... René Pflamm
11:37 Bug #42955 (Needs Feedback): QueryResult::count() returns 0 for queries with offset
Hi René,
> On my Project I notice if I have more then itemsPerPage and switched to another Page this one ist Empty.
...
Bastian Waidelich
11:27 Bug #42955 (Resolved): QueryResult::count() returns 0 for queries with offset
*NOTE* See comments below
On my Project I notice if I have more then itemsPerPage and switched to another Page thi...
René Pflamm

2012-11-13

11:35 Feature #32607: Export localized strings for JS consumption
If this can be of any use, here's a quick and dirty solution I threw together for a project: https://gist.github.com/... Francois Suter
10:27 Feature #32607: Export localized strings for JS consumption
I think that should go hand-in-hand with #42407 which I just added as a relation to this issue. Adrian Föder
10:23 Feature #32607: Export localized strings for JS consumption
I agree that this would be very useful. Filtering the messages would be a nice option.
A reference could be the JS...
Francois Suter
02:18 Feature #42176: Improve resolving of view
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16393
Gerrit Code Review
00:58 Feature #42176: Improve resolving of view
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16392
Gerrit Code Review
00:05 Feature #42176: Improve resolving of view
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16393
Gerrit Code Review

2012-11-12

23:42 Feature #37854: Refactor ConfigurationManager to Support Custom Configuration Files
Patch set 7 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/11982
Gerrit Code Review
23:42 Feature #42176: Improve resolving of view
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16392
Gerrit Code Review
11:30 Feature #42176: Improve resolving of view
I've created a basic WIP implementation and pushed it to gerrit.
So far it only works for ActionController, not Widg...
Marc Neuhaus
23:34 Bug #28862: Inconsistent handling of roles as objects and strings in Account
Patch set 9 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/13868
Gerrit Code Review
23:04 Bug #28862: Inconsistent handling of roles as objects and strings in Account
Patch set 8 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/13868
Gerrit Code Review
18:02 Bug #42888: ResourceManager chokes on non existing files
Adrian Föder wrote:
> An @ is a bit hacky IMO, do you have another Idea?
Right, but we already use it in other ...
Bastian Waidelich
17:46 Bug #42888: ResourceManager chokes on non existing files
ah, aand.. in most cases the $uri would be a string <ins>pointing</ins> to a filesystem file <ins>location</ins>; how... Adrian Föder
17:45 Bug #42888: ResourceManager chokes on non existing files
ah ok, good; because I wondered and hesitated to ask why a non existing file should result into a memory flood ;)
...
Adrian Föder
17:39 Bug #42888: ResourceManager chokes on non existing files
The description was wrong before, I only ran out of memory cause the exception tried to render the affected domain ob... Bastian Waidelich
17:03 Bug #42888 (Needs Feedback): ResourceManager chokes on non existing files
When importing a non existing file via @ResourceManager::importResource()@ the ResourceManager -kills the PHP process... Bastian Waidelich
15:35 Bug #42801 (Resolved): New FlowSession session class is broken
Applied in changeset commit:beac8272f684c69cc08df4b5f5e90736279aa654. Robert Lemke
09:54 Feature #42135 (Resolved): Include technical details to exceptions in dev context
Solved with https://review.typo3.org/15787 Karsten Dambekalns
09:51 Feature #29489 (Closed): Simple remote packages support
Karsten Dambekalns

2012-11-11

02:49 Feature #42176: Improve resolving of view
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16393
Gerrit Code Review
02:29 Feature #42176 (Under Review): Improve resolving of view
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16392
Gerrit Code Review
02:27 Feature #37854: Refactor ConfigurationManager to Support Custom Configuration Files
Patch set 6 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/11982
Gerrit Code Review

2012-11-10

19:24 Bug #42833 (Closed): Migration fails if InnoDB is not default engine
Christian Müller
01:14 Bug #42833: Migration fails if InnoDB is not default engine
My fault. Please close...
Can't reproduce my migration Problem again :(
Jan Roth
01:07 Bug #42833 (Closed): Migration fails if InnoDB is not default engine
MySQL Migration-Statements should have an Engine in Create Table Statements.
Migration fails if not.
Jan Roth
15:22 Bug #41571: UriBuilder doesn't properly handle arguments of SubRequests
http://forge.typo3.org/issues/35790 Is it possible that these two bugs are connected? Christoph Gross

2012-11-09

21:06 Bug #42801: New FlowSession session class is broken
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16365
Gerrit Code Review
20:39 Bug #42801 (Under Review): New FlowSession session class is broken
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16365
Gerrit Code Review
16:51 Bug #42801 (Accepted): New FlowSession session class is broken
Robert Lemke
08:29 Bug #42801: New FlowSession session class is broken
Martin Brüggemann wrote:
> @Karsten: Did you activate FlowSession handling
Nah, of course I obviously did not thi...
Karsten Dambekalns
06:55 Bug #42801: New FlowSession session class is broken
@Karsten: Did you activate FlowSession handling in your Caches.yaml like described here? https://review.typo3.org/#/c... Martin Brüggemann
19:35 Bug #42807 (Resolved): Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
Applied in changeset commit:bc8e604853889861e23cd78da170a32e4c7731a1. Karsten Dambekalns
18:25 Bug #42807 (Under Review): Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16364
Gerrit Code Review
18:11 Bug #42807: Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
This work:
$jsonPathsAndFilenames = Files::readDirectoryRecursively($packagePath, '.json');
asort($jsonPathsAndFil...
Dominique Feyer
18:11 Bug #42807: Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
This work:
$jsonPathsAndFilenames = Files::readDirectoryRecursively($packagePath, '.json');
asort($jsonPathsAndFil...
Dominique Feyer
18:02 Bug #42807: Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
It's a production server so no xdebug, here is a raw var dump:... Dominique Feyer
17:54 Bug #42807: Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
I tried again, after removing Data/Temporary I can use Flow fine, with PHP 5.3 and 5.4. The versions you have (Domini... Karsten Dambekalns
17:29 Bug #42807: Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
I can confirm this one.. ran into it yesterday. Cedric Ziel
11:13 Bug #42807: Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
I don't need to revert everything, just, the line 605 - 608 in Classes/TYPO3/Flow/Package/PackageManager.php and ever... Dominique Feyer
11:08 Bug #42807: Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
If I revert your commit:
git revert a148ca1ed1ded30ef8ecba4cdab2a0a575bd2c67
Everything work nicely ... so it's...
Dominique Feyer
10:59 Bug #42807: Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
After the surf deployment, my last commit are:
TYPO3.Flow 450778452d666fc1af2f6899806debcc49f6df94
doctrine/dbal ...
Dominique Feyer
09:16 Bug #42807 (Needs Feedback): Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
I updated all packages, removed all caches and PackageStates.php, before this:... Karsten Dambekalns
09:03 Bug #42807 (Accepted): Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
Yesterday I merged https://review.typo3.org/16081 after checking locally. It removes some code for that very problem ... Karsten Dambekalns
01:05 Bug #42807: Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
The annoying thing is after renamming the composer.json file to the original name everything work :( Dominique Feyer
00:36 Bug #42807 (Resolved): Libraries/doctrine/dbal/docs/examples/sharding/composer.json broken Flow
With recent Flow version (master), after surf deploy, I was unable to use flow (web or cli), after som debug it look ... Dominique Feyer

2012-11-08

22:24 Bug #42801 (Needs Feedback): New FlowSession session class is broken
For me Neos works fine with the change in place. Karsten Dambekalns
20:56 Bug #42801 (Resolved): New FlowSession session class is broken
Unfortunately the new FlowSession patch has been merged today some minutes before I could save my comment at review.t... Martin Brüggemann
22:01 Task #27283: Handle @InheritanceType in a correct manner
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/10893
Gerrit Code Review
15:18 Task #27283: Handle @InheritanceType in a correct manner
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/10893
Gerrit Code Review
21:36 Feature #37820 (Resolved): Check validator options and make them self-documenting
Applied in changeset commit:d374b12f2e7d13142ffc3ea9078ff3ba50ca7758. Karsten Dambekalns
16:13 Feature #37820: Check validator options and make them self-documenting
Patch set 7 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/11865
Gerrit Code Review
21:36 Feature #42539 (Resolved): Allow Object.yaml property value to be also an array
Applied in changeset commit:fd2178cfa3cf880cd3add52189e517470b30eb4c. Adrian Föder
20:41 Feature #42539: Allow Object.yaml property value to be also an array
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16068
Gerrit Code Review
16:08 Feature #42539: Allow Object.yaml property value to be also an array
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16068
Gerrit Code Review
17:35 Feature #42403 (Resolved): Session handler based on HTTP foundation with multi-backend support
Applied in changeset commit:5b56ade837f86ecb80678392b229cffd691232f3. Robert Lemke
14:23 Feature #42403: Session handler based on HTTP foundation with multi-backend support
Patch set 6 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15942
Gerrit Code Review
11:29 Feature #42403 (Under Review): Session handler based on HTTP foundation with multi-backend support
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15942
Gerrit Code Review
15:35 Feature #41358 (Resolved): Allow class/interface constants in configuration directives
Applied in changeset commit:50832abe17af1b8c37eb4fa84d0d87d8039c7a3f. Adrian Föder
11:03 Feature #41358: Allow class/interface constants in configuration directives
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15074
Gerrit Code Review
13:35 Task #42785 (Resolved): Make Mvc\Controller\Exception extend Mvc\Exception
Applied in changeset commit:2c27eb19fc8c6ba777c2a542bebebc3c6fffa8c7. Karsten Dambekalns
12:33 Task #42785 (Under Review): Make Mvc\Controller\Exception extend Mvc\Exception
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/14576
Gerrit Code Review
12:30 Task #42785 (Resolved): Make Mvc\Controller\Exception extend Mvc\Exception
The exception subclasses FLOW3\Exception directly. Karsten Dambekalns
12:35 Feature #42011 (Resolved): Display package keys in migration status
Applied in changeset commit:4b04b7dcfec6b54e60632056869dfe3eb4a9b831. Adrian Föder
12:14 Feature #42011: Display package keys in migration status
Patch set 11 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15629
Gerrit Code Review
11:14 Feature #42025: Uri type converter
Patch set 7 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15522
Gerrit Code Review
10:58 Bug #42342: Fatal error / no possibility to flush caches after package has been removed
Patch set 7 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15931
Gerrit Code Review
10:31 Bug #42342: Fatal error / no possibility to flush caches after package has been removed
Patch set 6 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15931
Gerrit Code Review
10:55 Feature #42784 (Under Review): Allow exposing the class name of an object in JsonView
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16050
Gerrit Code Review
10:54 Feature #42784 (Resolved): Allow exposing the class name of an object in JsonView
Besides telling the identifier of a persisted object, it should additionally be possible to expose the class name of ... Karsten Dambekalns
09:39 Bug #13559 (Accepted): ObjectSerializer failes with persistent objects within arrays
Possible without too much hassle. Karsten Dambekalns
09:34 Bug #42715: ObjectSerializer not capable of Collections
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16231
Gerrit Code Review
08:41 Feature #29972: Configurable Redirects
Bastian Waidelich wrote:
> ObjectPathMappings can be marked *obsolet* (triggered when an affected object is change...
Bastian Waidelich
08:41 Task #42781 (Rejected): Introduce global function flow_var_dump()
Robert Lemke
08:22 Task #42781 (Rejected): Introduce global function flow_var_dump()
Currently there's a namespaced function \TYPO3\Flow\var_dump() which helps debugging nested object structures.
It ...
Robert Lemke

2012-11-07

21:06 Bug #42770 (Resolved): Routing includes parameters from the current request
I believe that after https://review.typo3.org/#/c/15261/ some routing arguments from the current request is passed al... Aske Ertmann
16:00 Feature #42761 (Resolved): Let getByTag() return the identifiers, not only the values
Robert Lemke
15:08 Feature #42761 (Under Review): Let getByTag() return the identifiers, not only the values
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16253
Gerrit Code Review
15:00 Feature #42761 (Resolved): Let getByTag() return the identifiers, not only the values
Currently getByTag() only returns a numeric array of values – an array indexed by entry identifier would be more usef... Robert Lemke
15:35 Bug #42758 (Resolved): Unit test for PersistenceQueryRewritingAspect broken
Applied in changeset commit:3d946860c611b601492acd59a031d654d3503daa. Karsten Dambekalns
14:42 Bug #42758 (Under Review): Unit test for PersistenceQueryRewritingAspect broken
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16251
Gerrit Code Review
14:40 Bug #42758 (Resolved): Unit test for PersistenceQueryRewritingAspect broken
The solution for #42601 broke one test in the PersistenceQueryRewritingAspectTest. Karsten Dambekalns
14:58 Feature #28683 (Closed): CachingFramework should accept cacheIdentifiers with special characters
My suggestion: use md5() or an escaping function to sanitize the identifier (if possible). I'd rather not put it into... Robert Lemke
14:35 Bug #42601 (Resolved): Content Security: QOM rewriting is omitted if used in certain cases in an Action Controller
Applied in changeset commit:9af3204b3ceb08b488370d6d85802cac87821154. Robert Lemke
14:35 Bug #42756 (Resolved): Functional BrowserTest fails in Neos setup
Applied in changeset commit:9736f94532c3776154e701354559fa3dba67705e. Karsten Dambekalns
14:20 Bug #42756: Functional BrowserTest fails in Neos setup
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16250
Gerrit Code Review
14:02 Bug #42756: Functional BrowserTest fails in Neos setup
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16250
Gerrit Code Review
13:57 Bug #42756: Functional BrowserTest fails in Neos setup
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16250
Gerrit Code Review
13:44 Bug #42756 (Under Review): Functional BrowserTest fails in Neos setup
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16250
Gerrit Code Review
13:36 Bug #42756 (Resolved): Functional BrowserTest fails in Neos setup
The functional @BrowserTest@ in Flow (added as part of the solution to #42049) fails when run in a Neos context. This... Karsten Dambekalns
11:41 Feature #90 (Closed): Add PHP dependency support
Composer can do this, just put the requirements in composer.json. Karsten Dambekalns
11:40 Task #3374 (Rejected): Support dedicated package versions per context
This would clash with the way composer manages packages. And in the last years, noone really needed it, or it would h... Karsten Dambekalns
11:39 Task #41414 (Needs Feedback): Check packageKey naming / file structure below Packages/Vendor
Karsten Dambekalns
11:38 Bug #28672: Inconsistent configuration overrides for packages
Julle, this is related to the current issue(s)… Karsten Dambekalns
09:35 Task #40863 (Resolved): Update to Doctrine ORM version 2.3.x
Applied in changeset commit:b807ceb06695ae742da2be7e2d9b3d51f21bfd97. Karsten Dambekalns
09:22 Task #40863: Update to Doctrine ORM version 2.3.x
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/14568
Gerrit Code Review

2012-11-06

21:14 Feature #37854: Refactor ConfigurationManager to Support Custom Configuration Files
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/11982
Gerrit Code Review
17:16 Feature #37854: Refactor ConfigurationManager to Support Custom Configuration Files
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/11982
Gerrit Code Review
14:06 Task #42726: Document new package structure and management
Documentation coverage wish:
> I want to install an arbitrary package from "out there", for example TYPO3.SwiftMai...
Adrian Föder
13:59 Task #42726 (Resolved): Document new package structure and management
Topics
- Composer
- Package structure
- Upgrading
- Installing
- Maintaining
help me make the docs better, ...
Christian Jul Jensen
13:13 Bug #42723: Reflection\ObjectAccess consumes to much processing time
btw: this reduces the execution time within ObjectAccess from > 40% to below 10% in the given example Steffen Ritter
13:07 Bug #42723 (Under Review): Reflection\ObjectAccess consumes to much processing time
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16234
Gerrit Code Review
12:48 Bug #42723 (Resolved): Reflection\ObjectAccess consumes to much processing time
JsonView, Validation, Serialisation and many other parts of Flow use ObjectAccess to retrieve information about how t... Steffen Ritter
11:42 Feature #41297 (Under Review): Add a way to extends the Account security model
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16232
Gerrit Code Review
11:25 Feature #41297: Add a way to extends the Account security model
I think this is a bug, not a feature - and the solution is quite simple - it's a onliner annotation.
Anyhow it is ...
Steffen Ritter
11:27 Bug #42715 (Under Review): ObjectSerializer not capable of Collections
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16231
Gerrit Code Review
10:34 Bug #42715 (Resolved): ObjectSerializer not capable of Collections
Using the serializer on Objects containing instances of \Doctrine\Common\Collections\Collection will fail with an exc... Steffen Ritter
10:35 Bug #42511 (Resolved): "Uri" constructor silently accepts unparsable Uri strings
Applied in changeset commit:1b03b89c9a61c5a8066544929a9be590eb3521b3. Adrian Föder

2012-11-04

20:06 Bug #33055: AccessDeniedException instead of WebRedirect
I can confirm this problem on latest Flow master. I think that either throwing an "AccessDeniedException" is wrong he... Andreas Wolf

2012-11-01

22:29 Bug #42606: Content Security with nested objects
Wrong issue infos!
It doesnt work also with only 1 nested loop!
Julian Kleinhans
22:15 Bug #42606 (New): Content Security with nested objects
I think there is a strange bug in the content security
Initial situation:
3 Models
- Product
-- Category
--...
Julian Kleinhans
19:59 Bug #42601 (Under Review): Content Security: QOM rewriting is omitted if used in certain cases in an Action Controller
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16106
Gerrit Code Review
19:10 Bug #42601 (Under Review): Content Security: QOM rewriting is omitted if used in certain cases in an Action Controller
The QOM Query Rewriting Aspect checks if the security context is initialized. If it is not yet initialized, it will s... Robert Lemke
19:05 Bug #40241 (Rejected): Encryption key sometimes readable for anybody
We generally don't set umasks or owners / groups while creating files because server setups may vary a lot. If I'm no... Robert Lemke
11:16 Feature #41900 (Accepted): Check for duplicate PSR-0 autoload namespaces
Christian Jul Jensen

2012-10-31

19:30 Bug #42342: Fatal error / no possibility to flush caches after package has been removed
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15931
Gerrit Code Review
19:02 Feature #33539: Append query arguments from RoutePart handlers
I'd a missing feature for creating links inside modules at the moment. But since we've fixed recursive namespaced arg... Aske Ertmann
18:22 Task #42452: Command handling should support global flags
Christopher Hlubek wrote:
> [...] this is much more convenient than going back to the beginning and inserting @hel...
Bastian Waidelich
18:10 Task #42452: Command handling should support global flags
Not exactly, Git also supports @git rebase --help@. For long commands this is much more convenient than going back to... Christopher Hlubek
11:37 Task #42452: Command handling should support global flags
Right now it follow the git model ;) Karsten Dambekalns
17:35 Feature #42169 (Resolved): Use named indexes for placeholders in translation labels
Applied in changeset commit:71318ee5819bc78a04d2c632774ac0abb13b946f. Thomas Hempel
14:21 Task #42550: Add top-level .htaccess to block everything but Web
> Although we ask to use @Web@ as document root, that is often not done and/or possible (think about the practice of ... Robert Lemke
14:13 Task #42550 (Under Review): Add top-level .htaccess to block everything but Web
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16078
Gerrit Code Review
14:07 Task #42550: Add top-level .htaccess to block everything but Web
Agreed. Also, the complex variant doesn't even work - it blocks as soon as @Web@ is used as doc root as intended, bec... Karsten Dambekalns
13:46 Task #42550: Add top-level .htaccess to block everything but Web
yep, agree the simple variant too, and since @Web/@ has an htaccess already, two files are *affected*, but only one n... Adrian Föder
13:42 Task #42550: Add top-level .htaccess to block everything but Web
Karsten Dambekalns wrote:
> Two ways to do this so far.
I'd prefer the simple solution with two files and w/o e...
Bastian Waidelich
12:41 Task #42550: Add top-level .htaccess to block everything but Web
Two ways to do this so far.
_Simple, but affecting two files_
@.htaccess@...
Karsten Dambekalns
11:54 Task #42550 (Under Review): Add top-level .htaccess to block everything but Web
Although we ask to use @Web@ as document root, that is often not done and/or possible (think about the practice of de... Karsten Dambekalns
13:52 Bug #34879 (Accepted): Proxied object is not update()able
Karsten Dambekalns
13:37 Bug #34879: Proxied object is not update()able
I managed to experience the error again; not in the exact same situation like initially described, but maybe comparab... Adrian Föder
10:09 Feature #42539 (Under Review): Allow Object.yaml property value to be also an array
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16068
Gerrit Code Review
10:01 Feature #42539 (Resolved): Allow Object.yaml property value to be also an array
It would be nice to be allowed setting also arrays as values for Object properties.
Use case: consider the followi...
Adrian Föder

2012-10-30

15:14 Bug #42520 (New): Cache must be flushed globally for package state changes
Currently after (de)activating a package the cache is only flushed for the *current context*. Instead it must be flus... Bastian Waidelich
15:10 Feature #42479 (Under Review): Context dependant Package states
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16047
Gerrit Code Review
12:13 Feature #42025: Uri type converter
Patch set 6 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15522
Gerrit Code Review
12:10 Feature #42025: Uri type converter
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15522
Gerrit Code Review
12:04 Feature #42025: Uri type converter
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15522
Gerrit Code Review
11:57 Bug #42511 (Under Review): "Uri" constructor silently accepts unparsable Uri strings
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/16035
Gerrit Code Review
11:37 Bug #42511 (Resolved): "Uri" constructor silently accepts unparsable Uri strings
The Uri::__construct() method basically relies on the parse_url() method. As of PHP documentation, http://de3.php.net... Adrian Föder

2012-10-29

20:35 Feature #42355 (Resolved): Use public key fingerprint for UUID in RsaWalletService
Applied in changeset commit:8a706f20a7bd3cd34f08778a52a40acf35f17313. Christopher Hlubek
19:34 Task #42452: Command handling should support global flags
Good point. Some more ideas for global flags (extract from irc):... Bastian Waidelich
17:25 Bug #42342: Fatal error / no possibility to flush caches after package has been removed
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15931
Gerrit Code Review
15:02 Feature #42479: Context dependant Package states
As discussed the PackageStates.php syntax should be rather like:... Bastian Waidelich
14:18 Feature #42479 (Closed): Context dependant Package states
It should be possible to (de)activate packages for certain contexts only.
Suggested PackageStates.php syntax:...
Bastian Waidelich
10:36 Bug #42434 (Resolved): Resource handler not available when session is initialized
Applied in changeset commit:84ef743d1530b5344b2599efea211bd311f9e16c. Ferdinand Kuhl
09:40 Bug #42434: Resource handler not available when session is initialized
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15964
Gerrit Code Review
09:59 Feature #42467 (Resolved): NotEmpty Validator does not work with arrays
Forard-port this patch: https://review.typo3.org/#/c/14707/ Alexander Schnitzler
09:42 Task #42465 (New): Document i18n settings
The documentation does not explain the available settings for i18n. Karsten Dambekalns

2012-10-28

09:57 Task #42452 (Closed): Command handling should support global flags
The current help handling in Flow is not intuitive. It would be much easier if every command would understand a "--he... Christopher Hlubek
09:38 Bug #36720: Settings from package cannot override default value with empty array
Patch set 6 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/10896
Gerrit Code Review

2012-10-26

22:37 Bug #42434: Resource handler not available when session is initialized
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15964
Gerrit Code Review
22:30 Bug #42434 (Under Review): Resource handler not available when session is initialized
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15964
Gerrit Code Review
22:20 Bug #42434 (Resolved): Resource handler not available when session is initialized
In current master (and all earlier releases) the bootstrap initializes the session before the resource wrappers. If y... Ferdinand Kuhl
11:23 Feature #42355: Use public key fingerprint for UUID in RsaWalletService
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15925
Gerrit Code Review
11:23 Feature #42355: Use public key fingerprint for UUID in RsaWalletService
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15925
Gerrit Code Review

2012-10-25

21:14 Feature #42403 (Resolved): Session handler based on HTTP foundation with multi-backend support
I'd like a new session implementation which
* simplifies testing by using the HTTP foundation
* allows for using ...
Robert Lemke
15:56 Bug #40410: Exception when using Apc, Memcached of Redis cache backend for reflection status and object configuration
Kobbe van Daatselaar wrote:
> In general it would be nice if the caching segment of the documentation was updated wi...
Thomas Plessis
15:36 Task #42319 (Resolved): Make PATTERN_MATCH_PACKAGEKEY more strict (again)
Applied in changeset commit:9762f6271c1bedbffe7295dfc9281e30a6d029a1. Karsten Dambekalns
13:56 Feature #33539 (Needs Feedback): Append query arguments from RoutePart handlers
Do we still need this for Neos?
I'm still a bit unsure about this approach..
Bastian Waidelich
13:39 Feature #42169: Use named indexes for placeholders in translation labels
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15809
Gerrit Code Review
10:40 Bug #42342: Fatal error / no possibility to flush caches after package has been removed
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15931
Gerrit Code Review
10:33 Bug #42342: Fatal error / no possibility to flush caches after package has been removed
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15931
Gerrit Code Review
10:33 Bug #42342 (Under Review): Fatal error / no possibility to flush caches after package has been removed
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15931
Gerrit Code Review
10:33 Bug #42269: Unittests fail with PHPunit 3.7(.8)
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15905
Gerrit Code Review
10:29 Bug #37473: Subsequent Exceptions related to Doctrine Entity Manager makes it snap shut
Apparently this is the intended behavior of Doctrine:
> In other words, when an exception occurs during flush/comm...
Matthieu Napoli
10:19 Feature #42366 (Under Review): Make session handling compatible to any PHP session handler
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15930
Gerrit Code Review
08:40 Feature #42366: Make session handling compatible to any PHP session handler
Sounds good, do you provide a change, Martin? Christian Müller
07:55 Feature #42366 (Rejected): Make session handling compatible to any PHP session handler
Today there's no possibility to get Flow working with a custom php session save handler like redis or memcached. You ... Martin Brüggemann
09:35 Bug #42368 (Resolved): Typo in setfilepermissions Script
Applied in changeset commit:c3371afade3336c3ded27b99898ba05a5474f6fa. Ingo Pfennigstorf
08:39 Bug #42368: Typo in setfilepermissions Script
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15929
Gerrit Code Review
08:21 Bug #42368 (Under Review): Typo in setfilepermissions Script
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15929
Gerrit Code Review
08:17 Bug #42368 (Resolved): Typo in setfilepermissions Script
The output of the filepermissions script currently is:
"TYPo3 Flow File Permission Script"
Maybe the "o" should...
Ingo Pfennigstorf

2012-10-24

17:12 Feature #42355 (Under Review): Use public key fingerprint for UUID in RsaWalletService
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15925
Gerrit Code Review
17:11 Feature #42355 (Resolved): Use public key fingerprint for UUID in RsaWalletService
Christopher Hlubek
16:12 Bug #41048 (Rejected): AbstractAuthenticationController namespace does not match folder structure
Just bad rebaisng. Christian Jul Jensen
16:11 Bug #42114 (Needs Feedback): PHPUnit functional tests cannot be run when only installed through composer
Christian Jul Jensen
12:55 Bug #42342: Fatal error / no possibility to flush caches after package has been removed
Robert Lemke wrote:
> Steps to reproduce:
>
> - create Acme.Demo package
> - warm up caches
> - remove the pack...
Robert Lemke
12:55 Bug #42342 (Resolved): Fatal error / no possibility to flush caches after package has been removed
Steps to reproduce (*production context!*):
- create Acme.Demo package
- warm up caches
- remove the package man...
Robert Lemke

2012-10-23

17:38 Task #42319: Make PATTERN_MATCH_PACKAGEKEY more strict (again)
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15910
Gerrit Code Review
17:15 Task #42319 (Under Review): Make PATTERN_MATCH_PACKAGEKEY more strict (again)
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15910
Gerrit Code Review
17:14 Task #42319 (Resolved): Make PATTERN_MATCH_PACKAGEKEY more strict (again)
The pattern used to check for a valid package key was made less strict during composer integration. Although composer... Karsten Dambekalns
15:50 Bug #42269 (Under Review): Unittests fail with PHPunit 3.7(.8)
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15905
Gerrit Code Review
15:41 Bug #42269: Unittests fail with PHPunit 3.7(.8)
The issue here is that PHPUnit-Mock-Objects in has added an internal identifier '__phpunit_id' to mock objects, this ... Christian Jul Jensen
12:12 Feature #42011: Display package keys in migration status
Patch set 10 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15629
Gerrit Code Review
11:14 Feature #42011: Display package keys in migration status
Patch set 9 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15629
Gerrit Code Review
11:00 Feature #42025: Uri type converter
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/15522
Gerrit Code Review
 

Also available in: Atom