Activity

From 2013-04-18 to 2013-05-17

2013-05-17

17:10 Bug #46036: AuthenticationRequired should not be thrown in PolicyEnforcement if resource is available to Everybody
Patch set 7 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/18695
Gerrit Code Review
16:38 Bug #41459: Version information missing
This can be closed now Pankaj Lele
16:27 Feature #47191: Make (property) Validators aware of parent class and the property they belong to
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/19923
Gerrit Code Review

2013-05-16

17:15 Task #48296 (Needs Feedback): Missing method in ExceptionHandlerInterface
While only implementing the ExceptionHandlerInterface the booting script fails because it tries to inject the system ... Marco Falkenberg
17:07 Bug #48290 (Under Review): Browser must not directly handle cookie
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20830
Gerrit Code Review
17:02 Bug #48290 (Resolved): Browser must not directly handle cookie
Currently the Browser class has a property cookies (not used), and use a wrong signature for the method Request::crea... Dominique Feyer
14:27 Task #48254: Add german translations for bundled validation error messages
Patch set 4 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20823
Gerrit Code Review
14:24 Task #48254: Add german translations for bundled validation error messages
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20823
Gerrit Code Review
12:56 Task #48254: Add german translations for bundled validation error messages
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20823
Gerrit Code Review
12:43 Task #48254 (Under Review): Add german translations for bundled validation error messages
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20823
Gerrit Code Review
12:39 Task #48254 (Resolved): Add german translations for bundled validation error messages
Add the german translations for the validation error messages Alexander Berl
14:06 Task #48251 (Under Review): Documentation for translated validation messages is missing
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20826
Gerrit Code Review
11:43 Task #48251 (Resolved): Documentation for translated validation messages is missing
Currently, there is no documentation available other than the comment in http://forge.typo3.org/issues/33970 on how t... Alexander Berl
01:11 Bug #48231 (Under Review): Ignoring of inheritance type in FlowAnnotationDriver no longer works
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20817
Gerrit Code Review
01:09 Bug #48231 (Resolved): Ignoring of inheritance type in FlowAnnotationDriver no longer works
Due to weird circumstances the tests for the FlowAnnotationDriver were skipped for a while, and change Ie6e46efa858a6... Karsten Dambekalns

2013-05-15

22:38 Task #45279 (Resolved): Add new translations for bundled validator error messages
Applied in changeset commit:8666254913c7ee2c63ec844e8e60f174ebbbfe0e. Laurent Cherpit
22:30 Task #45279: Add new translations for bundled validator error messages
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20808
Gerrit Code Review
22:32 Feature #27117: Bind routes to HTTP request methods
Patch set 7 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/19801
Gerrit Code Review
16:39 Bug #48133 (Resolved): Bug in set file permissions script
Applied in changeset commit:361343830e95ed2aeaecffa0c035c778a9a0617c. Berit Hlubek
12:23 Feature #43966 (Under Review): Allow all routes to include sub routes
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20784
Gerrit Code Review
10:48 Feature #47228: Allow doctrine event listeners to be configured
Patch set 4 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/19984
Gerrit Code Review

2013-05-14

21:11 Feature #47228: Allow doctrine event listeners to be configured
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/19984
Gerrit Code Review
17:47 Bug #47975: initializeObject in a Entity is called at a time where no properties are loaded
I also had that; and I realized it's because of this gem here:... Adrian Föder
16:49 Feature #48167: Command line account and role browsing
done here: https://github.com/svparijs/TYPO3.UserManagement/ Adrian Föder
09:14 Feature #48167 (Accepted): Command line account and role browsing
In order to see what accounts and roles are known to the system, a command line tool should be provided. Adrian Föder
12:54 Bug #43587 (Under Review): Arrays in Yaml configuration should be interleaved rather than merge/overruled
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20773
Gerrit Code Review
09:21 Bug #46689: The new ClassLoader swallows Fatal Errors
Since this change was merged we get the folowing PHP Warnings:... Frans Saris
09:01 Feature #42176: Improve resolving of view
Patch set 33 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/16392
Gerrit Code Review

2013-05-13

17:38 Task #48150 (Under Review): removeAll in a doctrine repository should use DQL for speed
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20749
Gerrit Code Review
17:38 Task #48150 (Rejected): removeAll in a doctrine repository should use DQL for speed
Christian Müller
15:13 Feature #42176: Improve resolving of view
Patch set 12 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/16393
Gerrit Code Review
15:13 Feature #42176: Improve resolving of view
Patch set 32 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/16392
Gerrit Code Review
12:56 Bug #48133 (Under Review): Bug in set file permissions script
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20728
Gerrit Code Review
12:53 Bug #48133 (Resolved): Bug in set file permissions script
When I install the TYPO3Neos demo package the set file permission command doesn't work.
On the lines 55-58 the @sear...
Berit Hlubek
12:23 Bug #48093: AbstractCompositeValidators memory consumption continuously grow
Hi,
sorry I've seen that I have reattached the same patch as in the first post.
Here it is the correct one.
So...
Tarcisio Fedrizzi

2013-05-10

16:03 Bug #46076: PersistentQueryRewriteAspect doesn't work with Collections when doing IN / CONTAINS / MATCHES comparisons
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/18761
Gerrit Code Review
13:49 Bug #48093: AbstractCompositeValidators memory consumption continuously grow
Hi,
there is an error with the patch when $this->validatedInstancesContainer is not set. I've attached a new version...
Tarcisio Fedrizzi

2013-05-09

18:55 Bug #47975: initializeObject in a Entity is called at a time where no properties are loaded
I have the same problem but my call stack differs from your call stack.
My two Entities:...
rottenrice no-lastname-given
17:52 Bug #48093 (New): AbstractCompositeValidators memory consumption continuously grow
Hi,
there is a problem in the way AbstractCompositeValidators passes the validatedInstacesContainer to its children....
Tarcisio Fedrizzi

2013-05-08

14:01 Bug #45290: Body arguments should not be merged before property mapping takes place
Ok, I think I get it now. The problem is that you also need the Controller Arguments information to know that a param... Alexander Berl
12:51 Feature #47951: Warn if persistence stack is not empty at the end of a get-request
Hi Alex no-lastname-given, ;)
interesting idea (at least for development context).
The code that is currently r...
Bastian Waidelich
12:33 Bug #47995 (Closed): The method hasRole() in TYPO3/Flow/Security/Account is currently broken
Thanks for the feedback Dominique,
We'll definitely have to add a comprehensive "breaking changes" section to docu...
Bastian Waidelich

2013-05-07

18:41 Feature #39740 (Closed): Allow setting fixed values for route parts
Closing due to lack of feedback (feel free to re-open or comment if this issue is still relevant) Bastian Waidelich
18:39 Feature #42479 (Closed): Context dependant Package states
Too old, too rusty, not important enough(?). Closing this for now Bastian Waidelich
18:36 Bug #47255: Cache is not flushed for changes in sub routes
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20590
Gerrit Code Review
18:34 Bug #47255 (Under Review): Cache is not flushed for changes in sub routes
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20590
Gerrit Code Review
18:35 Bug #46636 (Resolved): Authentication does not work any longer without redirects
Applied in changeset commit:e06e0f2dd6eb565f00ae535c780ab13b74de8f92. Bastian Waidelich
17:48 Bug #46636: Authentication does not work any longer without redirects
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20586
Gerrit Code Review
17:42 Bug #46636: Authentication does not work any longer without redirects
Patch set 6 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20346
Gerrit Code Review
16:52 Bug #46636: Authentication does not work any longer without redirects
Patch set 5 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20346
Gerrit Code Review
14:08 Bug #46636: Authentication does not work any longer without redirects
Patch set 4 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20346
Gerrit Code Review
12:30 Bug #46636: Authentication does not work any longer without redirects
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20346
Gerrit Code Review
16:43 Bug #47995: The method hasRole() in TYPO3/Flow/Security/Account is currently broken
Thanks for all those information, I update my application based on it, and everything work fine.
I need to spend m...
Dominique Feyer
12:19 Bug #47995: The method hasRole() in TYPO3/Flow/Security/Account is currently broken
Bastian Waidelich wrote:
> Unfortunately the API for this is not very usable yet (at least some convenience method w...
Karsten Dambekalns
11:30 Bug #47995: The method hasRole() in TYPO3/Flow/Security/Account is currently broken
Karsten Dambekalns wrote:
> if you want to check for access to something, you should rather check if permission to...
Bastian Waidelich
11:26 Bug #47995: The method hasRole() in TYPO3/Flow/Security/Account is currently broken
One thing we could discuss is what methods should (alternatively) be able to take a string (role identifier). But the... Karsten Dambekalns
11:10 Bug #47995: The method hasRole() in TYPO3/Flow/Security/Account is currently broken
We could try to use VO here. That should solve the mentioned problem. At least for doctrine our implementation should... Christian Müller
10:33 Bug #47995 (Needs Feedback): The method hasRole() in TYPO3/Flow/Security/Account is currently broken
The actual "issue" is that roles are entities since #28862
Account::hasRole() won't be the only place where this bre...
Bastian Waidelich
09:48 Bug #47995: The method hasRole() in TYPO3/Flow/Security/Account is currently broken
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20555
Gerrit Code Review

2013-05-06

21:04 Bug #47995: The method hasRole() in TYPO3/Flow/Security/Account is currently broken
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20555
Gerrit Code Review
21:00 Bug #47995 (Under Review): The method hasRole() in TYPO3/Flow/Security/Account is currently broken
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20555
Gerrit Code Review
20:51 Bug #47995 (Closed): The method hasRole() in TYPO3/Flow/Security/Account is currently broken
Currently this code always return FALSE:... Dominique Feyer
17:29 Bug #47975: initializeObject in a Entity is called at a time where no properties are loaded
The real problem is, that the Doctrine Proxy __load() calls __wakeup() before the properties are actually loaded.
...
Alexander Berl
11:52 Bug #47975 (Resolved): initializeObject in a Entity is called at a time where no properties are loaded
I have an Entity which is loaded during validation of a Action Argument. (The Action Argument is another Entity with ... Benno Weinzierl
12:09 Bug #45290: Body arguments should not be merged before property mapping takes place
Alexander Berl wrote:
Hi Alexander,
thanks for your feedback, it's appreciated!
> * Why does the *HttpReques...
Bastian Waidelich

2013-05-05

16:44 Feature #47951 (New): Warn if persistence stack is not empty at the end of a get-request
Because the flow framework follows a rfc-standard, persistanceManager::persistAll() is not executed automatically for... Alex no-lastname-given
16:36 Bug #47950 (New): import of remote resources
ResourceManager::importResource($uri) allows the import of resources from a supplied url. The import fails, if an url... Alex no-lastname-given
16:28 Feature #47948 (Closed): support for resource references
It is not always suitable to download and import external resources, which are for example accessible over http: Larg... Alex no-lastname-given

2013-05-03

20:37 Bug #47858: Remove .htaccess from Composer Installer Essentials
I have to edit the .htaccess file to make Flow work on my shared host. So, I second the motion of not keeping .htacce... Jacob Floyd
09:48 Bug #47858 (Needs Feedback): Remove .htaccess from Composer Installer Essentials
It's very hard to tweak @mod_rewrite@ rules if the @.htaccess@ file is not under control of the user. It's a little b... Christopher Hlubek
11:38 Task #47859 (Accepted): Logging: Do not log all decisions in \TYPO3\Flow\Security\Aspect\LoggingAspect
Logging all decisions for @decideOnJoinPoint()@ and @decideOnResource()@ is way to much if these decisions are used c... Christopher Hlubek

2013-05-02

17:37 Bug #45936 (Resolved): Respect open basedir during uploading of files
Applied in changeset commit:6fb381bc6665e65bd0b68dec8ee027c644188d86. Aske Ertmann
17:13 Bug #45936: Respect open basedir during uploading of files
Patch set 2 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20424
Gerrit Code Review
17:13 Bug #45936: Respect open basedir during uploading of files
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20424
Gerrit Code Review
17:37 Bug #45609 (Resolved): If authenticated initializeObject() in every session-scope object is called twice
Applied in changeset commit:298ee692ee2381d0b525e4da071ac9a3307e05b5. Benno Weinzierl
10:36 Bug #47809 (Under Review): Access to undefined index when an inheritance chain member is not an Entity
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20402
Gerrit Code Review
10:24 Bug #47809 (Accepted): Access to undefined index when an inheritance chain member is not an Entity
Adrian Föder
10:24 Bug #47809 (Resolved): Access to undefined index when an inheritance chain member is not an Entity
It comes to an exception like this,... Adrian Föder
10:08 Feature #42479 (New): Context dependant Package states
Karsten Dambekalns

2013-05-01

22:03 Feature #43082 (On Hold): Add CLI support for scaffolding models, views, controller
Alexander Schnitzler
12:38 Bug #47776 (Closed): Configuration is no longer correctly Merged from the various Packages
closed on behalf of Benno. Adrian Föder
12:33 Bug #47776: Configuration is no longer correctly Merged from the various Packages
Turns out that this behaiviour is intended.
The problem in my installation was that the PackageStates.php was comp...
Benno Weinzierl
11:08 Bug #47776: Configuration is no longer correctly Merged from the various Packages
Argh... sorry... just noticed that my last comment is a "missing feature" or "trap" that is around for a long time an... Benno Weinzierl
11:03 Bug #47776: Configuration is no longer correctly Merged from the various Packages
Now i additionally noticed that my main subcontext Settings.yaml is only merged partially.
It seemes to be dependa...
Benno Weinzierl
08:29 Bug #47776 (Closed): Configuration is no longer correctly Merged from the various Packages
I Noticed that since some recent change my policy yaml is completely empty. Some other Configuration Parts are missin... Benno Weinzierl
12:37 Bug #47675 (Resolved): PSR-0 compatible classes using underscore cannot be loaded
Applied in changeset commit:24e15e460535f5002d9b2fdb6a700618b0e9a85a. Karsten Dambekalns
11:44 Bug #47675: PSR-0 compatible classes using underscore cannot be loaded
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20371
Gerrit Code Review
11:38 Bug #45609: If authenticated initializeObject() in every session-scope object is called twice
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20370
Gerrit Code Review

2013-04-30

23:00 Bug #45936: Respect open basedir during uploading of files
Patch set 10 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/18548
Gerrit Code Review
17:36 Bug #47054 (Resolved): IntegerConverter not found for properties annotated with @var int
Applied in changeset commit:f6d8d4fbc3a4cc5ce86c44089d146c37bc312fb7. Philipp Maier
16:46 Bug #47054: IntegerConverter not found for properties annotated with @var int
Patch set 1 for branch *FLOW3-1.1* has been pushed to the review server.
It is available at https://review.typo3.org/...
Gerrit Code Review
16:43 Bug #47054: IntegerConverter not found for properties annotated with @var int
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20358
Gerrit Code Review
17:22 Bug #47114: @var annotations with included variable throw exception
Patch set 4 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/19860
Gerrit Code Review
16:38 Bug #46636: Authentication does not work any longer without redirects
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20346
Gerrit Code Review
11:18 Bug #46636: Authentication does not work any longer without redirects
Bastian Waidelich wrote:
> So maybe Christophers suggestion would be the way to go
I just gave this a quick try...
Bastian Waidelich
11:00 Bug #46636: Authentication does not work any longer without redirects
Bastian Waidelich wrote:
> Karsten Dambekalns wrote:
>
> > IIRC we ignored security in the aspect if the SecurityCon...
Bastian Waidelich
10:57 Bug #46636: Authentication does not work any longer without redirects
Karsten Dambekalns wrote:
> IIRC we ignored security in the aspect if the SecurityContext cannot be initialized - ...
Bastian Waidelich
10:18 Bug #46636: Authentication does not work any longer without redirects
IIRC we ignored security in the aspect if the SecurityContext cannot be initialized - that used to work fine in the p... Karsten Dambekalns
10:02 Bug #46636: Authentication does not work any longer without redirects
What about using some context to disable the @PersistenceQueryRewritingAspect@ in the AccountRepository? This is a fe... Christopher Hlubek
09:42 Bug #46636 (Under Review): Authentication does not work any longer without redirects
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20346
Gerrit Code Review
08:47 Bug #46636 (Accepted): Authentication does not work any longer without redirects
Bastian Waidelich
12:38 Task #47346: Configuration schema for doctrine cacheImplementation still exists
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20027
Gerrit Code Review
12:38 Task #47346: Configuration schema for doctrine cacheImplementation still exists
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20027
Gerrit Code Review
11:27 Bug #47725 (Under Review): BCrypt hashing should support migration of older costs
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20349
Gerrit Code Review
11:24 Bug #47725 (Resolved): BCrypt hashing should support migration of older costs
In the current implementation of the @BCryptHashingStrategy@ a password is hashed with @crypt@ and the hash contains ... Christopher Hlubek

2013-04-29

18:59 Bug #46636: Authentication does not work any longer without redirects
Marco, this one cost me some hours too..
The issue is not only true for HTTP authentication but for most authenticat...
Bastian Waidelich
03:09 Bug #45290: Body arguments should not be merged before property mapping takes place
Hi Bastian,
I like the general idea and here is some input from my side:
* Why does the *HttpRequest* need to k...
Alexander Berl

2013-04-28

21:51 Bug #47675 (Under Review): PSR-0 compatible classes using underscore cannot be loaded
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20274
Gerrit Code Review
21:41 Bug #47675 (Resolved): PSR-0 compatible classes using underscore cannot be loaded
PSR-0 mandates that backslashes *and* underscores be translated to directory separators when loading classes. This is... Karsten Dambekalns
15:55 Bug #45936: Respect open basedir during uploading of files
Patch set 9 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/18548
Gerrit Code Review
13:37 Bug #45936: Respect open basedir during uploading of files
Patch set 8 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/18548
Gerrit Code Review

2013-04-26

22:56 Bug #45062: TranslateViewHelper fails if an array with a single number is passed as arguments
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20130
Gerrit Code Review
12:35 Bug #47534 (Resolved): Algorithms::generateUUID() creates uppercase characters in UUIDs
Applied in changeset commit:60ea1313d0206c8528830872eec2c7a712424b6e. Karsten Dambekalns
11:50 Bug #47534 (Under Review): Algorithms::generateUUID() creates uppercase characters in UUIDs
Patch set 2 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20127
Gerrit Code Review
12:35 Bug #46424 (Resolved): Infinite recursive call in DependencyProxy
Applied in changeset commit:be43db2a12c63d1da71272f5310186d56dceaa7b. Anonymous
12:17 Bug #46424 (Under Review): Infinite recursive call in DependencyProxy
Patch set 3 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20136
Gerrit Code Review
11:33 Bug #46424 (Resolved): Infinite recursive call in DependencyProxy
Adrian Föder
11:37 Bug #47500 (Resolved): createRole: Role is not passed as object to repository
Applied in changeset commit:2399c8ab33f0f9c08799ba7df87d0ce4c18630a1. Christopher Hlubek
11:30 Bug #47500 (Under Review): createRole: Role is not passed as object to repository
Patch set 2 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20110
Gerrit Code Review

2013-04-25

15:52 Bug #46008 (Under Review): Reflection should resolve relative namespaces
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20158
Gerrit Code Review
12:10 Bug #46008: Reflection should resolve relative namespaces
Sebastian Kurfuerst wrote:
> I have a patch for this in the works; almost ready.
Great! Looking forward to this...
Bastian Waidelich
11:43 Bug #46008 (Accepted): Reflection should resolve relative namespaces
I have a patch for this in the works; almost ready. Sebastian Kurfuerst
11:20 Feature #47551: Allow usage of links for destructive actions again
Bastian Waidelich wrote:
> doh, I just realized that I replaced my long comment from yesterday by this addition.. ...
Bastian Waidelich
10:37 Feature #47551: Allow usage of links for destructive actions again
doh, I just realized that I replaced my long comment from yesterday by this addition.. does anyone has the original t... Bastian Waidelich
10:36 Bug #46636: Authentication does not work any longer without redirects
A solution would be to write a new slot which unsets the locked roles and connect it with the authenticatedToken sign... Marco Falkenberg
10:32 Bug #46636: Authentication does not work any longer without redirects
After some debugging I could locate the problem. The thing is that the Security\Context never will be informed about ... Marco Falkenberg
09:38 Bug #46424 (Under Review): Infinite recursive call in DependencyProxy
Patch set 2 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20136
Gerrit Code Review
09:37 Bug #47371 (Resolved): If matched AOP even wraps static methods which will fail on runtime
Applied in changeset commit:86620ec8cead8ba3773a554aab7763d436de9c8a. Steffen Ritter
09:33 Bug #47371: If matched AOP even wraps static methods which will fail on runtime
Patch set 3 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20031
Gerrit Code Review
09:36 Bug #47309 (Closed): Calling this in static context in cached classes
Solved in #47371 Robert Lemke
00:05 Bug #43587: Arrays in Yaml configuration should be interleaved rather than merge/overruled
Hum, didn't know that PHP converts all numeric strings in array keys to their integer equivalent... i.e. there's no d... Alexander Berl

2013-04-24

18:22 Bug #46424 (Resolved): Infinite recursive call in DependencyProxy
Adrian Föder
18:02 Bug #46424: Infinite recursive call in DependencyProxy
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20136
Gerrit Code Review
18:00 Bug #46424: Infinite recursive call in DependencyProxy
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20113
Gerrit Code Review
17:34 Feature #47551: Allow usage of links for destructive actions again
That's pretty much exactly what I thought of. A JavaScript helper would be cool too, just not sure how that could be ... Alexander Berl
16:02 Feature #47551: Allow usage of links for destructive actions again
Even though it is done a lot (we have caught ourselves here) it is really bad practice to have "delete" links in an a... Bastian Waidelich
15:33 Feature #47551 (Rejected): Allow usage of links for destructive actions again
With the new changes that prevent unsafe methods to be destructive (no persistence), it is currently impossible to cr... Alexander Berl
17:17 Bug #47371: If matched AOP even wraps static methods which will fail on runtime
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20032
Gerrit Code Review
17:03 Bug #43587: Arrays in Yaml configuration should be interleaved rather than merge/overruled
The general problem stays - we cannot currently append tags to the ignoreTags configuration, which majorly limits con... Alexander Berl
11:08 Bug #45062: TranslateViewHelper fails if an array with a single number is passed as arguments
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20130
Gerrit Code Review
11:03 Bug #45062 (Under Review): TranslateViewHelper fails if an array with a single number is passed as arguments
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20130
Gerrit Code Review
10:55 Bug #45290: Body arguments should not be merged before property mapping takes place
This one is a bit tricky, especially if we want to resolve it in a API-backwards-compatible (aka BC) way.
I wrote sev...
Bastian Waidelich
10:37 Bug #47534 (Resolved): Algorithms::generateUUID() creates uppercase characters in UUIDs
Applied in changeset commit:654e0f4bb2c557b066538bc53588461c1c3aad28. Karsten Dambekalns
10:07 Bug #47534: Algorithms::generateUUID() creates uppercase characters in UUIDs
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20127
Gerrit Code Review
09:42 Bug #47534: Algorithms::generateUUID() creates uppercase characters in UUIDs
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20124
Gerrit Code Review
09:38 Bug #47534 (Under Review): Algorithms::generateUUID() creates uppercase characters in UUIDs
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20124
Gerrit Code Review
09:36 Bug #47534 (Resolved): Algorithms::generateUUID() creates uppercase characters in UUIDs
The UUIDs generated by that method were always lowercase, now they are uppercase. As soon as you do a case-sensitive ... Karsten Dambekalns
10:22 Bug #47114: @var annotations with included variable throw exception
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/19860
Gerrit Code Review
09:31 Feature #43569: Exception Handler should respect format
I'm still on this one, but I won't make it within the timeframe of #45088 (which is already over).
If anyone comes u...
Bastian Waidelich

2013-04-23

21:46 Bug #47500 (Resolved): createRole: Role is not passed as object to repository
Applied in changeset commit:c1774f1b389c42a928b61ef11f9d1ad934402a52. Christopher Hlubek
17:03 Bug #47500: createRole: Role is not passed as object to repository
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20110
Gerrit Code Review
17:01 Bug #47500 (Under Review): createRole: Role is not passed as object to repository
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20109
Gerrit Code Review
17:00 Bug #47500 (Resolved): createRole: Role is not passed as object to repository
This causes an error when adding roles through the PolicyService createRole method:... Christopher Hlubek
21:38 Feature #27117: Bind routes to HTTP request methods
Patch set 6 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/19801
Gerrit Code Review
12:50 Feature #27117: Bind routes to HTTP request methods
Patch set 5 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/19801
Gerrit Code Review
12:21 Feature #27117: Bind routes to HTTP request methods
Patch set 4 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/19801
Gerrit Code Review
18:51 Bug #46424 (Under Review): Infinite recursive call in DependencyProxy
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20113
Gerrit Code Review
14:08 Bug #47309: Calling this in static context in cached classes
Is just stubled across the same problem. Be aware that also Policies are enforced through AOP wich make rules like th... Benno Weinzierl
12:59 Bug #45609: If authenticated initializeObject() in every session-scope object is called twice
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20094
Gerrit Code Review
11:42 Bug #45609: If authenticated initializeObject() in every session-scope object is called twice
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20094
Gerrit Code Review
11:21 Bug #45609: If authenticated initializeObject() in every session-scope object is called twice
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20104
Gerrit Code Review
11:59 Bug #47487 (New): Functional test classes in package without classes are not compiled
Additionally there is an error if only a Exception class is present in @Classes@. Christopher Hlubek
11:36 Bug #47473 (Resolved): Role->setParentRoles() fails if array of roles contains duplicates
Applied in changeset commit:ec284d851ef79af9440dd94bb73168a56edbe886. Anonymous
11:10 Bug #47473: Role->setParentRoles() fails if array of roles contains duplicates
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20102
Gerrit Code Review
11:06 Bug #47473 (Under Review): Role->setParentRoles() fails if array of roles contains duplicates
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20101
Gerrit Code Review
10:59 Bug #47473 (Resolved): Role->setParentRoles() fails if array of roles contains duplicates
If the array of roles contains duplicates, Doctrine may fail while trying to insert the same role multiple times:
...
Robert Lemke
11:36 Bug #46862 (Resolved): generateUUID can be broken by seeding mt_rand
Applied in changeset commit:e8f8e08e0885419651c494aff6ae804e8bf643b3. Christian Jul Jensen
11:12 Bug #46862: generateUUID can be broken by seeding mt_rand
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20103
Gerrit Code Review
10:45 Feature #45675 (Resolved): Improve Error-Message when @var annotation in entity object is omitted
resolved with https://review.typo3.org/#/c/19079/ Adrian Föder
10:36 Bug #47469 (Resolved): ActionController issues PHP warning if it contains methods with short names
Applied in changeset commit:084d60756f361186676ee96042f38aaedf6eb4cd. Anonymous
10:13 Bug #47469: ActionController issues PHP warning if it contains methods with short names
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20100
Gerrit Code Review
09:59 Bug #47469 (Under Review): ActionController issues PHP warning if it contains methods with short names
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20099
Gerrit Code Review
09:46 Bug #47469 (Resolved): ActionController issues PHP warning if it contains methods with short names
If an ActionController implementation contains a method which has a name shorter than 7 characters, the new static me... Robert Lemke

2013-04-22

23:08 Bug #45609 (Under Review): If authenticated initializeObject() in every session-scope object is called twice
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20094
Gerrit Code Review
22:36 Bug #45609: If authenticated initializeObject() in every session-scope object is called twice
After some more testing i noticed that this happens not only after authentication.
It happens with any session scope...
Benno Weinzierl
22:34 Feature #47456 (New): ManyToOne and OneToOne Relations of Objects passed as Action Argument are loaded automatically
All ManyToOne and OneToOne Relations of Objects passed as Action Argument are loaded automatically by the Validator (... Benno Weinzierl
13:21 Bug #47429: Global policy files no longer allowed
FYI: I'm not entirely sure what should be allowed in global policy files but I wonder why I am supposed to change the... Thomas Hempel
13:13 Bug #47429 (New): Global policy files no longer allowed
Since one of the last changes, global policy files are no longer allowed. After a discussion on the IRC channel, this... Thomas Hempel
13:05 Bug #46689: The new ClassLoader swallows Fatal Errors
ok, that's simple, however not nice, see
https://github.com/doctrine/annotations/blob/master/lib/Doctrine/Common/A...
Adrian Föder
12:28 Bug #46689: The new ClassLoader swallows Fatal Errors
I began to dig a bit, and most errors come from the annotation corner anyways, that is, for example, occurrences of @... Adrian Föder
10:35 Bug #47416 (Resolved): PolicyService needs hasRole()
Applied in changeset commit:899b0dc8d0f4f26d3123bb2eda041f99c01717e4. Karsten Dambekalns
10:29 Bug #47416: PolicyService needs hasRole()
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20066
Gerrit Code Review
09:48 Bug #47416: PolicyService needs hasRole()
Patch set 2 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20065
Gerrit Code Review
09:45 Bug #47416 (Under Review): PolicyService needs hasRole()
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20065
Gerrit Code Review

2013-04-21

23:18 Bug #47416 (Resolved): PolicyService needs hasRole()
Christian Müller

2013-04-20

22:01 Bug #46862: generateUUID can be broken by seeding mt_rand
Patch set 3 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/19515
Gerrit Code Review
17:27 Feature #47404 (New): Add getters and setters methods for introduced properties
Currently you must introduce methods over interface introduction and then wrap introduced methods with around advice.... Rafael Kähm

2013-04-19

10:09 Bug #47371: If matched AOP even wraps static methods which will fail on runtime
Patch set 2 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20031
Gerrit Code Review
10:09 Bug #47371: If matched AOP even wraps static methods which will fail on runtime
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20032
Gerrit Code Review
10:08 Bug #47371 (Under Review): If matched AOP even wraps static methods which will fail on runtime
Patch set 1 for branch *2.0* has been pushed to the review server.
It is available at https://review.typo3.org/20031
Gerrit Code Review
09:50 Bug #47371 (Resolved): If matched AOP even wraps static methods which will fail on runtime
Runtime execution fails if a static method is wrapped with AOP code because it tries to access the instance variable ... Steffen Ritter

2013-04-18

18:42 Task #47346: Configuration schema for doctrine cacheImplementation still exists
I pushed a change that not only solves the two inconsistencies, but at the same time opens up the persistence configu... Alexander Berl
18:27 Task #47346 (Under Review): Configuration schema for doctrine cacheImplementation still exists
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20027
Gerrit Code Review
13:57 Task #47346: Configuration schema for doctrine cacheImplementation still exists
There's also a missing schema update for the log settings, following #46053.
Maybe this issue should be changed to a...
Alexander Berl
13:36 Task #47346 (Resolved): Configuration schema for doctrine cacheImplementation still exists
In http://forge.typo3.org/projects/package-typo3-flow3/repository/revisions/5e5ebf31970c572b49fc73f87832f351248f899f ... Alexander Berl
18:38 Feature #47339 (Under Review): Allow RequestHandlers to get the current Request injected
Patch set 1 for branch *master* has been pushed to the review server.
It is available at https://review.typo3.org/20028
Gerrit Code Review
10:51 Feature #47339 (Needs Feedback): Allow RequestHandlers to get the current Request injected
As a proxy service, I want to provide my own eager RequestHandler which might modify the request headers depending on... Alexander Berl
07:17 Bug #47309: Calling this in static context in cached classes
Hey Wouter,
had that too - and this is because you use AOP with an advice catching all methods in a class.
College ...
Steffen Ritter
 

Also available in: Atom