Activity

From 2012-01-22 to 2012-02-20

2012-02-20

21:36 Bug #33707: It is possible to authenticate with an expired account
Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9129
Gerrit Code Review
21:36 Revision 873bdd53: [BUGFIX] Makes account expiry work as expected
The expiry if an account was checked in a way that left an account
enabled until after it's last day. Now it will exp...
Karsten Dambekalns
21:33 Revision 9c5f208e: Merge "[BUGFIX] Makes account expiry work as expected"
Karsten Dambekalns
16:32 Bug #33932 (Under Review): AOP Proxy Compiler does not handle numeric annotation parameters
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9125
Gerrit Code Review
16:32 Revision 895d48b9: [BUGFIX] Fixes handling of numeric annotation parameters
Annotations like @FooBar(amount=2) lost their parameter value
in renderAnnotation(). This change fixes that.
Change-...
Karsten Dambekalns
12:20 Bug #34110 (Under Review): Proxy Class strips off Class Doc Comments
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9121
Gerrit Code Review
12:17 Bug #34110 (Resolved): Proxy Class strips off Class Doc Comments
The Proxy Class replaces the class doc comment of the original class with ... Bastian Waidelich
10:35 Bug #34050: DatetimeParser code documentation is wrong
This also causes the DateTimeValidator to not work Kira Backes

2012-02-19

21:18 Bug #33719: .Shortcuts symlinks use absolute path
Patch set 8 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868
Gerrit Code Review
21:15 Bug #33719: .Shortcuts symlinks use absolute path
Patch set 7 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868
Gerrit Code Review
00:10 Feature #33371: HTTP 1.1 Support
@robert - I've seen/heard your comments in the scrum protocols/meeetings about this HTTP work you're doing.
I've b...
Jacob Floyd

2012-02-17

18:56 Bug #34050 (Resolved): DatetimeParser code documentation is wrong
The public methods expect a DateTime object, which they then forward as a string to the internal methods, which then ... Kira Backes
18:42 Bug #34048 (Closed): ResolveValidator breaks on DateTime method arguments
When a dateTime argument is set in a method the ResolveValidator breaks, he tries to create a DateTime (instead of a ... Kira Backes

2012-02-16

20:49 Bug #34018: Annotation with a numeric value is not correctly generated
Patch for the above - suggested - solution.
Tried to add a test in CompilerTest.php, but they are all using the Va...
Georg Großberger
20:40 Bug #34018 (Closed): Annotation with a numeric value is not correctly generated
I have a domain model declared abstract with the following annotation:... Georg Großberger
15:46 Bug #33971: Missing id attributes in XLIFF files are not reported to the user
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9050
Gerrit Code Review
15:41 Bug #33971 (Under Review): Missing id attributes in XLIFF files are not reported to the user
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9050
Gerrit Code Review
11:10 Bug #33971 (Resolved): Missing id attributes in XLIFF files are not reported to the user
XLIFF requires id attributes, but if they are missing, FLOW3 does not complain. Instead, _something_ happens in the X... Karsten Dambekalns
14:27 Bug #33621 (Under Review): Filebackend reports error when trying to rename file due to concurrent proccesses
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8931
Gerrit Code Review
14:27 Bug #27989: Wrong check in our atomic writes code
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8931
Gerrit Code Review
09:20 Bug #27989: Wrong check in our atomic writes code
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8787
Gerrit Code Review
12:19 Revision ca35b1a8: [TASK] Implement best practices in XLIFF files
Adds some "required" attributes to the XLIFF files.
Change-Id: Ib30f5653197229c5d2062eba2c0020ca6e191216
Resolves: #...
Karsten Dambekalns
12:08 Feature #28399: Validation message and code should be configurable for bundled validators
Thinking about it - with the ability to use L18N for outputting the validation message, I don't think we need that fe... Bastian Waidelich
12:01 Feature #33970: Add translation files for bundled validator error messages
FYI this is (kind of) how you can use those to output localized validation messages:... Bastian Waidelich
10:51 Feature #33970 (Under Review): Add translation files for bundled validator error messages
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9045
Gerrit Code Review
10:50 Feature #33970 (Resolved): Add translation files for bundled validator error messages
Karsten Dambekalns
11:58 Bug #33973 (Under Review): Duplicate error id used in StringValidator
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9047
Gerrit Code Review
11:56 Bug #33973 (Resolved): Duplicate error id used in StringValidator
StringValidator uses the error id 1238108068, but that is used in StringLengthValidator as well. Karsten Dambekalns
11:57 Revision 30d2c4cc: [BUGFIX] Fixes duplicate error id in StringValidator
StringValidator uses the error id 1238108068, but that is
used in StringLengthValidator. This change makes the former...
Karsten Dambekalns
11:53 Feature #6456: Customization/localization of validator errors
With the current version and #33970 applied this works quite nice. The respective Fluid code looks something like:
<p...
Bastian Waidelich
11:32 Bug #33932 (Accepted): AOP Proxy Compiler does not handle numeric annotation parameters
Karsten Dambekalns
11:27 Bug #33883 (Accepted): Repositories must be singleton, but no checks are done on this
Well, repositories must be singleton, otherwise I don't even want to imagine what happens…
I changed the bug title...
Karsten Dambekalns
11:22 Bug #33968 (Needs Feedback): Missing privileges to create symlinks in Windows cause exception
Rens, could you take care of this? Yo are working on symlinks and Windows anyway, right?
The error is caused by th...
Karsten Dambekalns
09:52 Bug #33968 (Closed): Missing privileges to create symlinks in Windows cause exception
Warning: symlink() [<a href='function.symlink'>function.symlink</a>]: Cannot create symlink, error code(1314) in C:\x... Richard Lederer
11:19 Feature #33937 (Needs Feedback): Convenience method to resolve public "resource://" paths
What is this needed for? Any file functions can use the resource:// URL directly. To fetch the web URL, a method exis... Karsten Dambekalns
11:10 Bug #33882: XliffParser does not allow id-less translation
Kira Backes wrote:
> Okay, but then you should throw an exception in case of a missing id attribute?
Good point, ...
Karsten Dambekalns
10:57 Bug #33882: XliffParser does not allow id-less translation
Okay, but then you should throw an exception in case of a missing id attribute? Kira Backes
10:48 Bug #33882 (Rejected): XliffParser does not allow id-less translation
This will never change, as XLIFF requires id attributes to be present. Plus, id-based translation is recommended as i... Karsten Dambekalns
10:50 Revision b484e6a4: [FEATURE] Add translations for validation error messages
Change-Id: I8c8c4d19b673d02339369d2a16961441ad65bdc5
Resolves: #33970
Releases: 1.1
Karsten Dambekalns
07:56 Major Feature #33258: Implement support for Assetic
Hey Adrian,
Thanks for starting with a concept! Some comments from my side:
* I'd suggest to use "Assets.yaml" ...
Sebastian Kurfuerst
02:46 Bug #33562 (Resolved): FileBackend didnt use defaultLifetime if lifetime is null
Applied in changeset commit:a0ee47d189f0a2141b6c8de2813d984c69a44eba. Karsten Dambekalns
02:46 Task #33836 (Resolved): Add better error handling to XML file parsing
Applied in changeset commit:987a264f0322b1125ed41fcc26dd41b494b6b2ce. Karsten Dambekalns

2012-02-15

23:24 Feature #33959: Make quantity optional when using single numeric arguments in translations
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9040
Gerrit Code Review
23:18 Feature #33959 (Under Review): Make quantity optional when using single numeric arguments in translations
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9040
Gerrit Code Review
23:15 Feature #33959 (Resolved): Make quantity optional when using single numeric arguments in translations
When using plural forms, the quantity to be used for finding the correct plural form has to be given explicitly. This... Karsten Dambekalns
22:47 Bug #33719: .Shortcuts symlinks use absolute path
Patch set 6 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868
Gerrit Code Review
21:38 Bug #33562: FileBackend didnt use defaultLifetime if lifetime is null
Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9037
Gerrit Code Review
21:37 Revision f22d865d: [BUGFIX] Fix default lifetime use in cache backend
This fixes the FileBackend so it actually uses the default lifetime
in cases it should do that.
ArrayAccess detectio...
Karsten Dambekalns
21:32 Revision 7807b255: Merge "[BUGFIX] Fix default lifetime use in cache backend"
Karsten Dambekalns
16:01 Task #33836: Add better error handling to XML file parsing
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8968
Gerrit Code Review
16:00 Revision 987a264f: [TASK] Add better handling of XML parse errors to I18n
When parsing XML files the exception thrown in case of errors did not
differentiate between filesystem and parsing er...
Karsten Dambekalns
12:40 Revision 8259dca1: Merge "[TASK] Remove CGL DocBook XML files from package"
Karsten Dambekalns
12:27 Revision fbc69cc6: Merge "[TASK] Adjust test fixture to Doctrine.ORM 2.2.0"
Karsten Dambekalns
11:25 Feature #33937 (Accepted): Convenience method to resolve public "resource://" paths
It would be helpful to have an easy way to resolve the 'resource://...' path
to static resources.
Currently we end...
Bastian Waidelich
09:51 Major Feature #33258 (Accepted): Implement support for Assetic
Sebastian Kurfuerst
09:48 Major Feature #33258: Implement support for Assetic
Johannes,
we "plan" anything you like :) We're currently in the absolut early stage of collecting some ideas and imp...
Adrian Föder
00:37 Bug #33932 (Resolved): AOP Proxy Compiler does not handle numeric annotation parameters
In the TYPO3\FLOW3\Object\Proxy\Compiler::renderAnnotation function, there is currently no case for numeric annotatio... Alexander Berl

2012-02-14

22:55 Major Feature #33258: Implement support for Assetic
Great!
Do you plan to integrate filters and nested assets, too?

It would be nice to be able to set the debug mode (...
Johannes K
16:00 Major Feature #33258: Implement support for Assetic
So far, I've created two repositories at Github in order to get a clue and to get a possibility to work with and to p... Adrian Föder
10:57 Feature #32106: Support for Object source in PropertyMapper
Hey Sebastian,
sorry for my late Reply.
Sadly i'm still drowning in an ocean of work and can't see an end at th...
Marc Neuhaus
10:17 Bug #33883: Repositories must be singleton, but no checks are done on this
It is "fixable" by setting the scope of the repositories to Singleton Kira Backes
02:43 Task #33142 (Resolved): Ignore Tags / Annotations in external libraries, find some good solution
Applied in changeset commit:cebbf8e2851b28ffaf32abafbb675e47af6ae783. Christian Müller

2012-02-13

15:12 Task #33142: Ignore Tags / Annotations in external libraries, find some good solution
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7747
Gerrit Code Review
15:10 Revision cebbf8e2: [FEATURE] Make tag ignores configurable
Makes the list of ignored tags configurable. This can be used to add
tags to the list if needed by your package by si...
Christian Müller
14:47 Bug #33300 (Resolved): Validators should allow empty values
Applied in changeset commit:9b627954f289d793b808db2ac19edc6a04c25426. Bastian Waidelich
14:17 Bug #33883 (Resolved): Repositories must be singleton, but no checks are done on this
I have models with injected repositories and due to some change in the last 2 weeks in master I can no longer use FLO... Kira Backes
14:10 Bug #33882 (Rejected): XliffParser does not allow id-less translation
If I want to translate completely without IDs I’m still forced to add IDs in the translation .xlf, because the id-les... Kira Backes

2012-02-12

19:44 Bug #31779: Command execution on Windows breaks with special chars in path
Sorry to re-post this bug, but it seems to have reappeared (or wasn't really fixed yet). I did some more testing with... Alexander Berl
18:08 Feature #33862: a shorthand for custom validators
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986
Gerrit Code Review
01:50 Feature #33862: a shorthand for custom validators
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986
Gerrit Code Review
01:46 Feature #33862: a shorthand for custom validators
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986
Gerrit Code Review
01:41 Feature #33862: a shorthand for custom validators
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986
Gerrit Code Review
01:04 Feature #33862 (Under Review): a shorthand for custom validators
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986
Gerrit Code Review
00:44 Feature #33862 (Resolved): a shorthand for custom validators
Instead of using the fully qualified classname (which is long) Extbase has a shorthand of the form <ExtName>:<ValName... Stefan Neufeind

2012-02-10

15:08 Task #33836 (Under Review): Add better error handling to XML file parsing
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8968
Gerrit Code Review
14:21 Task #33836 (Resolved): Add better error handling to XML file parsing
When parsing XML files the exception thrown in case of errors does not differentiate between filesystem and parsing e... Karsten Dambekalns
15:07 Feature #33049: Allow configuration of context without environment variable (needed for IIS)
Just some extra info related to this issue:
On http://www.iis.net/ConfigReference/system.webServer/fastCgi/applica...
Rens Admiraal
14:52 Feature #33838: Add a way to configure Doctrine Mapping Type
Same problem in 1.0.2 Dominique Feyer
14:48 Feature #33838 (Accepted): Add a way to configure Doctrine Mapping Type
Karsten Dambekalns
14:43 Feature #33838 (Resolved): Add a way to configure Doctrine Mapping Type
As I need some JSON object storage in FLOW3, i need to use a custome Doctrine Mapping Type to handle to conversion be... Dominique Feyer
11:34 Feature #33531: DateTimeConverter should support DateTime subclasses
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8770
Gerrit Code Review
11:32 Bug #33300: Validators should allow empty values
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8691
Gerrit Code Review
11:32 Revision 9b627954: [!!!][BUGFIX] Validators should allow empty values
Currently most of the provided validators do not accept empty
values.
This makes it impossible to have an optional em...
Bastian Waidelich

2012-02-09

20:01 Bug #33719: .Shortcuts symlinks use absolute path
The symlinks in the Web/_Resources/Static/Packages folder should also follow relative pathnames for an chrooted insta... Bernhard Fischer
17:45 Bug #33719: .Shortcuts symlinks use absolute path
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868
Gerrit Code Review
14:48 Bug #33719: .Shortcuts symlinks use absolute path
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868
Gerrit Code Review
08:16 Bug #33719: .Shortcuts symlinks use absolute path
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868
Gerrit Code Review
15:02 Revision 81a8da3e: [TASK] Remove CGL DocBook XML files from package
They have been transformed to reST long ago…
Change-Id: I8b8b364ca03cfb6a6a7957f1aa1b8d0dcdc24542
Related: #29318
Karsten Dambekalns
14:57 Bug #27989: Wrong check in our atomic writes code
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8931
Gerrit Code Review
14:52 Bug #27989 (Under Review): Wrong check in our atomic writes code
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8931
Gerrit Code Review
11:22 Bug #27989 (Accepted): Wrong check in our atomic writes code
Karsten Dambekalns
13:36 Bug #33811 (Resolved): Not all caches are flushed (automatically) if needed
At least these are affected:
* Fluid template cache (e.g. when changing a ViewHelper class)
* XLIFF / locale cach...
Karsten Dambekalns
11:21 Bug #33562: FileBackend didnt use defaultLifetime if lifetime is null
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8786
Gerrit Code Review
11:21 Revision a0ee47d1: [BUGFIX] Fix default lifetime use in cache backend
This fixes the FileBackend so it actually uses the default lifetime
in cases it should do that.
ArrayAccess detectio...
Karsten Dambekalns
11:07 Bug #33789: PersistentObjectConverter may break with < PHP 5.3.4
Patch set 13 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5992
Gerrit Code Review
10:53 Revision b1cbf2ba: [BUGFIX] Makes account expiry work as expected
The expiry if an account was checked in a way that left an account
enabled until after it's last day. Now it will exp...
Karsten Dambekalns
10:53 Bug #33707 (Under Review): It is possible to authenticate with an expired account
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8926
Gerrit Code Review
09:38 Bug #33707 (Accepted): It is possible to authenticate with an expired account
Karsten Dambekalns
10:21 Bug #33621 (Needs Feedback): Filebackend reports error when trying to rename file due to concurrent proccesses
Karsten Dambekalns
10:21 Bug #33621: Filebackend reports error when trying to rename file due to concurrent proccesses
Thanks! Aside from a simple issue with your code (forgot to throw a new exception), this seems fine. Two "theoretical... Karsten Dambekalns
09:53 Bug #33621 (Accepted): Filebackend reports error when trying to rename file due to concurrent proccesses
Karsten Dambekalns
09:44 Bug #33621: Filebackend reports error when trying to rename file due to concurrent proccesses
Hm, I had this in the pipeline: https://review.typo3.org/8786 Karsten Dambekalns
09:40 Bug #33639 (Rejected): Error in mySQL Statements with columnnames like "from" or "char"
As per the recommendations in Doctrine 2, no automatic quoting is done. You are advised to not use reserved words as ... Karsten Dambekalns
09:32 Task #33801 (Rejected): Add a "mixed" type converter
Problem:
"mixed" is used as a default type in case you don't explicitly provide proper type but there is no such con...
Ondrej Hatala

2012-02-08

21:56 Bug #33719: .Shortcuts symlinks use absolute path
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868
Gerrit Code Review
17:02 Bug #33789 (Under Review): PersistentObjectConverter may break with < PHP 5.3.4
Patch set 12 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5992
Gerrit Code Review
16:57 Bug #33789: PersistentObjectConverter may break with < PHP 5.3.4
Ok, this is indeed caused by https://review.typo3.org/5992 Karsten Dambekalns
16:22 Bug #33789 (Resolved): PersistentObjectConverter may break with < PHP 5.3.4
The use of @newInstanceArgs()@ inside the @PersistentObjectConverter@ breaks for PHP older than 5.3.4 if a class has ... Karsten Dambekalns
14:47 Bug #33720 (Resolved): \TYPO3\FLOW3\I18n\Xml\* Namespace doesn't exists
Applied in changeset commit:5572609a0e9a5087a1185e3c0b30227c4618345d. Tolleiv Nietsch

2012-02-07

23:19 Bug #33720: \TYPO3\FLOW3\I18n\Xml\* Namespace doesn't exists
Nice catch! Karsten Dambekalns
23:18 Bug #33720: \TYPO3\FLOW3\I18n\Xml\* Namespace doesn't exists
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8869
Gerrit Code Review
23:18 Revision 5572609a: [BUGFIX] Fixes wrong \TYPO3\FLOW3\I18n\Xml\* namespace use
The mentioned namespace doesn't exist and the resulting error
e.g. breaks the Admin package.
Along the way some doc ...
Tolleiv Nietsch
14:12 Feature #32106: Support for Object source in PropertyMapper
Hey,
any news already? :)
Greets, Sebastian
Sebastian Kurfuerst
14:11 Major Feature #33258: Implement support for Assetic
Did anybody start working on a concept yet? Sebastian Kurfuerst
12:00 Feature #33581: Extend configuration/context
Added new version of patch as flow3.php patched part was missing Peter Russ
02:46 Task #33589 (Resolved): Issue warning / error when trying to @inject into private property
Applied in changeset commit:837fc9800a6530274df2b2d575afdc20c24280f0. Adrian Föder

2012-02-06

22:11 Bug #33720 (Under Review): \TYPO3\FLOW3\I18n\Xml\* Namespace doesn't exists
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8869
Gerrit Code Review
22:09 Bug #33720 (Resolved): \TYPO3\FLOW3\I18n\Xml\* Namespace doesn't exists
The namespace is used but never defined - the suggested patch changes the 1.5 occurrences to use the right Classes... Tolleiv Nietsch
21:36 Bug #33719 (Under Review): .Shortcuts symlinks use absolute path
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868
Gerrit Code Review
21:34 Bug #33719 (Closed): .Shortcuts symlinks use absolute path
The symlinks in the .Shortcuts folder use absolute paths. This is an issue in chrooted environments where the path to... Rens Admiraal
21:30 Revision 837fc980: [TASK] Throw exception when private properties have @Inject
Properties annotated for injection do obviously have to
be at least protected in order for the proxy class
having acc...
Adrian Föder
21:29 Task #33589: Issue warning / error when trying to @inject into private property
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8794
Gerrit Code Review
17:46 Bug #33707: It is possible to authenticate with an expired account
After some investigation I found out that this feature works actually, but only on day basis.
I would assumed that i...
Patrick Pussar
15:24 Bug #33707 (Resolved): It is possible to authenticate with an expired account
authenticationManager->authenticate() works even with expired account: account->getExpirationDate() is in the past. Patrick Pussar
16:08 Feature #33710 (New): Configuration based on Domain
i know you can change the Configuration based on the Context through apache config and htaccess.
I'm currently for...
Marc Neuhaus
14:58 Task #33705 (Resolved): Inconsistent spelling of sub package and class names
The FLOW3 package contains, for historical reasons, sub packages and in consequence namespaces and class names which ... Robert Lemke
07:05 Bug #33698 (Under Review): ObjectAccess::setProperty must work with numeric indexes
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8857
Gerrit Code Review
07:04 Bug #33698 (Resolved): ObjectAccess::setProperty must work with numeric indexes
When trying to set a value to an @ArrayAccess@ object with a numeric index, the ObjectAccess::setProperty() method th... Sebastian Kurfuerst

2012-02-03

13:38 Bug #33645 (Under Review): Wrong return value getPasswordHashingStrategyAndIdentifier()
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8826
Gerrit Code Review
12:29 Bug #33645 (Resolved): Wrong return value getPasswordHashingStrategyAndIdentifier()
getPasswordHashingStrategyAndIdentifier is supposed to return an array, but in case a strategy identifier has been re... Karsten Dambekalns

2012-02-02

23:37 Feature #26783: Support settings via OS Environment
How should we proceed with this? The reviews show a dislike for it, I would love to discuss the points shortly, I sti... Christian Müller
20:22 Bug #33639 (Rejected): Error in mySQL Statements with columnnames like "from" or "char"
The mySQL Statements must be escaped with `columnname`
SQLSTATE[42000]: Syntax error or access violation: 1064 You...
Jan Roth
18:33 Bug #31905: getInterceptedRequest is NULL but ACL matches correctly DENY
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7635
Gerrit Code Review
14:44 Bug #33601 (Resolved): Logging of incorrect doc comments in reflection is broken
Applied in changeset commit:9d06eed31b3980193470f87bb2a91c48f3750ffe. Karsten Dambekalns
10:00 Bug #33601: Logging of incorrect doc comments in reflection is broken
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8800
Gerrit Code Review
14:32 Bug #33621: Filebackend reports error when trying to rename file due to concurrent proccesses
Reported also fixes to TYPO3 s. http://forge.typo3.org/issues/33622 Peter Russ
14:19 Bug #33621 (Resolved): Filebackend reports error when trying to rename file due to concurrent proccesses
Due to concurrent processes it happens that a temp-cache-file gets renamed while an other process is just writing it.... Peter Russ
10:00 Revision 9d06eed3: [BUGFIX] Fixed logging of incorrect doc comments in reflection
If one enables logIncorrectDocCommentHints nothing happens, as the
settings are not available in the ReflectionServic...
Karsten Dambekalns
02:43 Bug #33148 (Resolved): CLDR locale chaining doesn't work correctly
Applied in changeset commit:b2e4f3e94039135614bf8f3976da4f833cfaae42. Adrian Föder

2012-02-01

23:56 Feature #33581: Extend configuration/context
I have to agree with Peter there,... passwords and stuff in environment of the v-host is a big no no in most enterpri... Chris Zepernick
14:33 Feature #33581: Extend configuration/context
As operation might use different versioning system this will not work for us. Further it requires to restart the php-... Peter Russ
12:42 Feature #33581: Extend configuration/context
Regarding the FLOW3_STATIC_CONFIGURATION idea, please check https://review.typo3.org/5731 and see if that would work ... Karsten Dambekalns
12:40 Feature #33581: Extend configuration/context
Peter Russ wrote:
> why is patch set to no although there is one attached?
By accident, it's the default it seems...
Karsten Dambekalns
11:35 Feature #33581: Extend configuration/context
why is patch set to no although there is one attached? Peter Russ
10:32 Feature #33581 (Resolved): Extend configuration/context
So far FLOW3 context is limited to Development, Testing and Production.
Further there is no way that operation/admi...
Peter Russ
18:09 Bug #33601 (Under Review): Logging of incorrect doc comments in reflection is broken
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8800
Gerrit Code Review
18:07 Bug #33601 (Resolved): Logging of incorrect doc comments in reflection is broken
If one enables... Karsten Dambekalns
18:03 Revision c9457f2f: Merge "[BUGFIX] CLDR locale chaining doesn't work correctly"
Karsten Dambekalns
17:15 Task #33589: Issue warning / error when trying to @inject into private property
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8794
Gerrit Code Review
16:58 Task #33589: Issue warning / error when trying to @inject into private property
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8794
Gerrit Code Review
16:57 Task #33589 (Under Review): Issue warning / error when trying to @inject into private property
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8794
Gerrit Code Review
15:10 Task #33589 (Accepted): Issue warning / error when trying to @inject into private property
Adrian Föder
15:08 Task #33589: Issue warning / error when trying to @inject into private property
Hint: the exception should be thrown in the ConfigurationBuilder:... Robert Lemke
15:02 Task #33589 (Resolved): Issue warning / error when trying to @inject into private property
Class properties declared as private cannot be used for property injection by using the @FLOW3\Inject annotation beca... Robert Lemke
12:43 Bug #32873 (Accepted): Value changes for logged in account are not persisted due to session serialization
Karsten Dambekalns
12:35 Feature #33587 (New): Automatically remove unused Resources
Currently you have to delete unused Resources manually (resource & resource pointer records from the database and the... Bastian Waidelich

2012-01-31

22:19 Bug #27989: Wrong check in our atomic writes code
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8787
Gerrit Code Review
22:14 Bug #27989 (Under Review): Wrong check in our atomic writes code
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8786
Gerrit Code Review
22:14 Bug #33562 (Under Review): FileBackend didnt use defaultLifetime if lifetime is null
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8786
Gerrit Code Review
17:59 Bug #33562 (Accepted): FileBackend didnt use defaultLifetime if lifetime is null
Karsten Dambekalns
16:27 Bug #33562 (Resolved): FileBackend didnt use defaultLifetime if lifetime is null
FileBackend.php#L191... Mike Franke
17:58 Task #33268 (Under Review): Document new YAML behaviour for Upgrade to 1.1
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8781
Gerrit Code Review
17:39 Task #33268 (Accepted): Document new YAML behaviour for Upgrade to 1.1
Karsten Dambekalns
17:38 Task #33268: Document new YAML behaviour for Upgrade to 1.1
Bastian Waidelich wrote:
> Christopher Hlubek wrote:
>
> > The class would not match @MyPackage\Domain\Model\Poo@...
Karsten Dambekalns
17:30 Feature #29389: Provide simple code migration mechanism
Here is the concept: https://docs.google.com/document/pub?id=1ik5gp9TLfvV5yem04uSFkrOKBWIvPp-zGSgVrG9ePkY Karsten Dambekalns
17:26 Feature #29389 (Accepted): Provide simple code migration mechanism
Karsten Dambekalns
02:41 Task #32733 (Resolved): Rename i18n setting from locale to i18n
Applied in changeset commit:4542cd6ee47ba36892f098a94b070f570c974ba4. Karsten Dambekalns
02:41 Task #32736 (Resolved): Remove defaultLocale from i18n settings
Applied in changeset commit:4542cd6ee47ba36892f098a94b070f570c974ba4. Karsten Dambekalns

2012-01-30

18:25 Bug #33519: Route always appends a question mark even if the route already has arguments
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8766
Gerrit Code Review
18:24 Bug #33519: Route always appends a question mark even if the route already has arguments
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8766
Gerrit Code Review
18:24 Bug #33519: Route always appends a question mark even if the route already has arguments
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8766
Gerrit Code Review
18:17 Feature #33539 (Rejected): Append query arguments from RoutePart handlers
It would be nice if we could append query arguments from routepart handlers, e.g. a appendArguments method, where it ... Aske Ertmann
16:20 Bug #33300: Validators should allow empty values
As discussed in todays daily scrum meeting we'll change the validators as suggested in the first changeset (with some... Bastian Waidelich
16:05 Bug #33300: Validators should allow empty values
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8691
Gerrit Code Review
14:47 Bug #33300: Validators should allow empty values
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8691
Gerrit Code Review
14:26 Bug #33300: Validators should allow empty values
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8691
Gerrit Code Review
10:42 Bug #33300: Validators should allow empty values
Yeah, this is exactly, what I talked about in the first place. ;-)
I did some testing implementation (in extbase of ...
Felix Oertel
16:19 Task #33537 (Closed): Rename NotEmpty Validator to Required
The NotEmpty validator has a special meaning as it is executed for empty values (NULL or '') too. "NotEmpty" is a bit... Bastian Waidelich
15:58 Revision c2c171d4: [TASK] Adjust test fixture to Doctrine.ORM 2.2.0
The proxy interface in Common is now used, the fixture needs to
be adjusted.
Change-Id: I0ed772ad4d04cf80679f7f72669...
Karsten Dambekalns
15:46 Task #32733: Rename i18n setting from locale to i18n
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7444
Gerrit Code Review
15:46 Revision 4542cd6e: [TASK] Clean up i18n settings
This renames the locale settings section to i18n and changes the
defaultLocaleIdentifier setting to be defaultLocale....
Karsten Dambekalns
15:46 Task #32736: Remove defaultLocale from i18n settings
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7444
Gerrit Code Review
15:10 Revision 3f66b479: [TASK] Adjust to Doctrine 2.2 libraries
This adjusts the FLOW3 annotation driver and the ClassMetadata
implementation to the changes in Doctrine 2.
Also the...
Karsten Dambekalns
14:07 Feature #33531 (Under Review): DateTimeConverter should support DateTime subclasses
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8770
Gerrit Code Review
14:05 Feature #33531 (Resolved): DateTimeConverter should support DateTime subclasses
Currently DateTimeConverter only accepts target types of 'DateTime'.
It should be possible to convert sub-types too.
Bastian Waidelich
13:28 Feature #29564 (Closed): Change \TYPO3\FLOW3\Property\TypeConverter\DateTimeConverter in order to compose a DateTime property from two input fields
This was already implemented with #27417 (in changeset commit:f8f3fc23121f5959203903cb672e987b4aca57b0) Bastian Waidelich
13:21 Bug #33530 (Under Review): SubRequest should return format of root request
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8769
Gerrit Code Review
13:19 Bug #33530 (Closed): SubRequest should return format of root request
The SubRequest is currently missing getter and setter for the request format.
This is required for Fluid for example...
Bastian Waidelich
08:18 Revision 0736b6a4: [BUGFIX] Fix FLOW3 1.0 Functional Tests
I introduced an error inside the functional tests in https://review.typo3.org/#change,7715
and this followup commit f...
Sebastian Kurfuerst

2012-01-29

22:49 Bug #33519 (Under Review): Route always appends a question mark even if the route already has arguments
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8766
Gerrit Code Review
22:48 Bug #33519 (Resolved): Route always appends a question mark even if the route already has arguments
The Route class (MVC\Web\Routing\Route) always append additionalParameters etc., with a question mark in front even t... Aske Ertmann
21:50 Bug #33148: CLDR locale chaining doesn't work correctly
Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7761
Gerrit Code Review
21:46 Bug #33148: CLDR locale chaining doesn't work correctly
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7761
Gerrit Code Review
21:49 Revision b2e4f3e9: [BUGFIX] CLDR locale chaining doesn't work correctly
When collecting CLDR resource files, the required chaining
is corrected (i.e. de_DE.xml also requires de.xml to be
in...
Adrian Föder
21:31 Revision 92760a2b: Merge "[FEATURE] Add currentLocale to I18n translation service"
Karsten Dambekalns
18:34 Feature #32985 (New): Implement Processing Rules when merging numerically-indexed arrays
de-assigning because we do not have any consensus if we need that feature. Sebastian Kurfuerst
14:47 Bug #32700 (Resolved): Validation does not work for collections
Applied in changeset commit:dd6f600dca46366e3109a47488bc6c3b607105d9. Andreas Förthner
12:58 Bug #32700 (Under Review): Validation does not work for collections
Patch set 8 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5401
Gerrit Code Review
12:56 Bug #32700 (Resolved): Validation does not work for collections
Sebastian Kurfuerst
12:57 Revision dd6f600d: [FEATURE] Add validators for objects inside of collections
Model validation stops as soon as a property is of type collection. What one
would expect here is, that all objects i...
Andreas Förthner
12:35 Revision 721f8e71: [FEATURE] Add currentLocale to I18n translation service
Also add localeIdentifier validator.
Related: #33188
Releases: 1.1
Change-Id: Ia01ed36343a58d929371f3d0f6e419ac630e8502
Rens Admiraal

2012-01-28

15:18 Revision d76869a3: Merge "[BUGFIX] Make updating a Doctrine Lazy Loading Proxy work" into FLOW3-1.0
Sebastian Kurfuerst
15:17 Task #32735 (Resolved): Move Translations folder up one level
Sebastian Kurfuerst
15:16 Revision 4c91a0ce: Merge "[TASK] Move Translations folder up one level"
Sebastian Kurfuerst
15:10 Revision b372293f: Merge "[FEATURE] Add support for not mapped properties"
Sebastian Kurfuerst

2012-01-27

19:11 Feature #30933 (Under Review): Check for unique constraints on add()
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8733
Gerrit Code Review
18:21 Feature #30933: Check for unique constraints on add()
Actually moving the validation run from onFlush to prePersist / preUpdate could solve this. Worth a try. Karsten Dambekalns
17:43 Bug #33300: Validators should allow empty values
Bastian Waidelich wrote:
> I thought about this quite a while and I'm pretty sure the new behavior makes much more s...
Bastian Waidelich
12:52 Bug #33300: Validators should allow empty values
Felix Oertel wrote:
> Ps: I came up with some (EmailAddress OR Empty) syntax, but don't know if this is
> really ...
Bastian Waidelich
12:40 Bug #33300: Validators should allow empty values
Felix Oertel wrote:
> I would highly prefer to enable proper disjunction usage over breaking the current behavior....
Bastian Waidelich
12:20 Bug #33300: Validators should allow empty values
I went into the same problems two weeks ago and tried to come up with a good solution.
I would highly prefer to en...
Felix Oertel
16:58 Bug #33495 (Resolved): No alternative entry point type can be configured
Theoretically it is possible to use another type of entry point than the pre-configured WebRedirect. In practice, how... Robert Lemke
16:53 Revision dee07763: Merge "[TASK] Some cleanup around authentication"
Robert Lemke
16:40 Feature #3623 (Resolved): Implement authentication providers
Robert Lemke

2012-01-26

17:05 Feature #33469 (Rejected): Support for temporary Resources
Currently all resources need a Database, because the resource manager uses it in order to store resource pointers.
I...
Bastian Waidelich
15:51 Bug #33465 (New): Some vital commands to recover the system fail when recovery is needed
There are several ways to bring the system into an unstable state, like activating a package that depends on another,... Karsten Dambekalns
15:27 Bug #33055: AccessDeniedException instead of WebRedirect
The controller method is already annotated with @FLOW3\SkipCsrfProtection Patrick Pussar
15:24 Feature #5882 (Closed): Caching for Package Manager/Package file listing
Thanks a lot for the effort, Manuel. I took your suggestion and integrated it into the optimized class loading and bo... Robert Lemke
14:45 Task #33398 (Resolved): Use boolean instead of bool in docblocks (CGL)
Applied in changeset commit:a655ebced9c650a3d3ce1f754187616ebd24c833. Karsten Dambekalns
13:33 Bug #33460 (Resolved): Missing function visiblity declarations
I found a missing visiblity declaration in \TYPO3\FLOW3\MVC\Web\Routing\Route, and thought I'd search to see if there... Aske Ertmann
09:49 Revision ce81af94: Merge "[TASK] Replace bool by boolean in docblocks"
Sebastian Kurfuerst
02:43 Feature #33226 (Resolved): Add Getter for validator options and child validators
Applied in changeset commit:0acd9937f4cd9b638c436eeea144f05e6038dbdf. Bastian Waidelich

2012-01-25

17:39 Bug #33300 (Under Review): Validators should allow empty values
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8691
Gerrit Code Review
11:31 Feature #33226: Add Getter for validator options and child validators
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7819
Gerrit Code Review
11:27 Feature #33226: Add Getter for validator options and child validators
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7819
Gerrit Code Review
11:17 Feature #33226: Add Getter for validator options and child validators
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7819
Gerrit Code Review
11:31 Revision 0acd9937: [FEATURE] Add Getter for validator options and child validators
Validation options are set through the constructor but there is
currently no way of accessing the options afterwards....
Bastian Waidelich

2012-01-24

17:54 Bug #33417 (Resolved): kickstart:package does not update .Shortcuts
For some reason the .Shortcuts sub directory for the new package is not created after a new package has been kickstar... Robert Lemke

2012-01-23

21:50 Bug #33148: CLDR locale chaining doesn't work correctly
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7761
Gerrit Code Review
21:33 Bug #33148 (Under Review): CLDR locale chaining doesn't work correctly
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7761
Gerrit Code Review
21:32 Bug #33148: CLDR locale chaining doesn't work correctly
The use of @getParentLocaleOf()@ in the @CldrRepository@ is wrong - as it only contains Locales for which resources a... Karsten Dambekalns
21:34 Task #33398 (Under Review): Use boolean instead of bool in docblocks (CGL)
Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8657
Gerrit Code Review
21:23 Task #33398 (Resolved): Use boolean instead of bool in docblocks (CGL)
In some places _bool_ is used in docblocks (e.g. for return and param types) instead if the preferred _boolean_. Karsten Dambekalns
21:34 Revision a655ebce: [TASK] Replace bool by boolean in docblocks
A CGL conformity fix. Just because I noticed. :)
Change-Id: I8f2e1191ae8df2ce39b3d3987c421e525f913c41
Resolves: #333...
Karsten Dambekalns
21:08 Bug #33397: Difference for ManyToMany with JoinTable annotation and without
Thanks, I'll check. Karsten Dambekalns
20:53 Bug #33397: Difference for ManyToMany with JoinTable annotation and without
Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8656
Gerrit Code Review
20:51 Bug #33397 (Under Review): Difference for ManyToMany with JoinTable annotation and without
Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8656
Gerrit Code Review
20:44 Bug #33397 (Resolved): Difference for ManyToMany with JoinTable annotation and without
There is a difference between joinColumn creation with JoinTable annotation and without for ManyToMany relations. In ... Christian Müller
16:57 Feature #26783: Support settings via OS Environment
Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5731
Gerrit Code Review

2012-01-22

17:15 Feature #33371 (Resolved): HTTP 1.1 Support
Implement proper HTTP 1.1 support.
This issue is a stub for the corresponding FLOW3 1.1 key feature (until I come ...
Robert Lemke
 

Also available in: Atom