Activity
From 2012-01-20 to 2012-02-18
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 ...
-
- When a dateTime argument is set in a method the ResolveValidator breaks, he tries to create a DateTime (instead of a ...
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... -
- I have a domain model declared abstract with the following annotation:...
-
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 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9050 -
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...
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8931 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8931 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8787 -
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...
-
- FYI this is (kind of) how you can use those to output localized validation messages:...
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9045 -
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9047 -
- StringValidator uses the error id 1238108068, but that is used in StringLengthValidator as well.
-
- With the current version and #33970 applied this works quite nice. The respective Fluid code looks something like:
<p... -
-
- Well, repositories must be singleton, otherwise I don't even want to imagine what happens…
I changed the bug title... -
- Rens, could you take care of this? Yo are working on symlinks and Windows anyway, right?
The error is caused by th... -
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...
-
- What is this needed for? Any file functions can use the resource:// URL directly. To fetch the web URL, a method exis...
-
- Kira Backes wrote:
> Okay, but then you should throw an exception in case of a missing id attribute?
Good point, ... -
- Okay, but then you should throw an exception in case of a missing id attribute?
-
- This will never change, as XLIFF requires id attributes to be present. Plus, id-based translation is recommended as i...
-
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" ... -
- Applied in changeset commit:a0ee47d189f0a2141b6c8de2813d984c69a44eba.
-
- Applied in changeset commit:987a264f0322b1125ed41fcc26dd41b494b6b2ce.
2012-02-15
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9040 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9040 -
- When using plural forms, the quantity to be used for finding the correct plural form has to be given explicitly. This...
-
- Patch set 6 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9037 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8968 -
- It would be helpful to have an easy way to resolve the 'resource://...' path
to static resources.
Currently we end... -
-
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... -
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...
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 (... -
- 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...
-
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... -
- It is "fixable" by setting the scope of the repositories to Singleton
-
02:43 Task #33142 (Resolved): Ignore Tags / Annotations in external libraries, find some good solution
- Applied in changeset commit:cebbf8e2851b28ffaf32abafbb675e47af6ae783.
2012-02-13
-
- Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7747 -
- Applied in changeset commit:9b627954f289d793b808db2ac19edc6a04c25426.
-
- I have models with injected repositories and due to some change in the last 2 weeks in master I can no longer use FLO...
-
- If I want to translate completely without IDs I’m still forced to add IDs in the translation .xlf, because the id-les...
2012-02-12
-
- Sorry to re-post this bug, but it seems to have reappeared (or wasn't really fixed yet). I did some more testing with...
-
- Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986 -
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986 -
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...
2012-02-10
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8968 -
- When parsing XML files the exception thrown in case of errors does not differentiate between filesystem and parsing e...
-
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... -
14:52 Feature #33838: Add a way to configure Doctrine Mapping Type
- Same problem in 1.0.2
-
-
- As I need some JSON object storage in FLOW3, i need to use a custome Doctrine Mapping Type to handle to conversion be...
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8770 -
- Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8691
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...
-
- Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8931 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8931 -
-
- At least these are affected:
* Fluid template cache (e.g. when changing a ViewHelper class)
* XLIFF / locale cach... -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8786 -
- Patch set 13 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5992 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8926 -
-
-
- Thanks! Aside from a simple issue with your code (forgot to throw a new exception), this seems fine. Two "theoretical...
-
-
- Hm, I had this in the pipeline: https://review.typo3.org/8786
-
- As per the recommendations in Doctrine 2, no automatic quoting is done. You are advised to not use reserved words as ...
-
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...
2012-02-08
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Patch set 12 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5992 -
- Ok, this is indeed caused by https://review.typo3.org/5992
-
- The use of @newInstanceArgs()@ inside the @PersistentObjectConverter@ breaks for PHP older than 5.3.4 if a class has ...
-
14:47 Bug #33720 (Resolved): \TYPO3\FLOW3\I18n\Xml\* Namespace doesn't exists
- Applied in changeset commit:5572609a0e9a5087a1185e3c0b30227c4618345d.
2012-02-07
-
- Nice catch!
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8869 -
- Hey,
any news already? :)
Greets, Sebastian -
- Did anybody start working on a concept yet?
-
12:00 Feature #33581: Extend configuration/context
- Added new version of patch as flow3.php patched part was missing
-
- Applied in changeset commit:837fc9800a6530274df2b2d575afdc20c24280f0.
2012-02-06
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8869 -
- The namespace is used but never defined - the suggested patch changes the 1.5 occurrences to use the right Classes...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- The symlinks in the .Shortcuts folder use absolute paths. This is an issue in chrooted environments where the path to...
-
- Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8794 -
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... -
- authenticationManager->authenticate() works even with expired account: account->getExpirationDate() is in the past.
-
- i know you can change the Configuration based on the Context through apache config and htaccess.
I'm currently for... -
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 ...
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8857 -
- When trying to set a value to an @ArrayAccess@ object with a numeric index, the ObjectAccess::setProperty() method th...
2012-02-03
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8826 -
- getPasswordHashingStrategyAndIdentifier is supposed to return an array, but in case a strategy identifier has been re...
2012-02-02
-
- How should we proceed with this? The reviews show a dislike for it, I would love to discuss the points shortly, I sti...
-
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... -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7635 -
- Applied in changeset commit:9d06eed31b3980193470f87bb2a91c48f3750ffe.
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8800 -
- Reported also fixes to TYPO3 s. http://forge.typo3.org/issues/33622
-
- Due to concurrent processes it happens that a temp-cache-file gets renamed while an other process is just writing it....
-
- Applied in changeset commit:b2e4f3e94039135614bf8f3976da4f833cfaae42.
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...
-
- As operation might use different versioning system this will not work for us. Further it requires to restart the php-...
-
- Regarding the FLOW3_STATIC_CONFIGURATION idea, please check https://review.typo3.org/5731 and see if that would work ...
-
- Peter Russ wrote:
> why is patch set to no although there is one attached?
By accident, it's the default it seems... -
- why is patch set to no although there is one attached?
-
- So far FLOW3 context is limited to Development, Testing and Production.
Further there is no way that operation/admi... -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8800 -
- If one enables...
-
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8794 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8794 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8794 -
-
- Hint: the exception should be thrown in the ConfigurationBuilder:...
-
- Class properties declared as private cannot be used for property injection by using the @FLOW3\Inject annotation beca...
-
-
- Currently you have to delete unused Resources manually (resource & resource pointer records from the database and the...
2012-01-31
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8787 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8786 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8786 -
-
16:27 Bug #33562 (Resolved): FileBackend didnt use defaultLifetime if lifetime is null
- FileBackend.php#L191...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8781 -
-
- Bastian Waidelich wrote:
> Christopher Hlubek wrote:
>
> > The class would not match @MyPackage\Domain\Model\Poo@... -
- Here is the concept: https://docs.google.com/document/pub?id=1ik5gp9TLfvV5yem04uSFkrOKBWIvPp-zGSgVrG9ePkY
-
-
- Applied in changeset commit:4542cd6ee47ba36892f098a94b070f570c974ba4.
-
- Applied in changeset commit:4542cd6ee47ba36892f098a94b070f570c974ba4.
2012-01-30
-
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8766 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8766 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8766 -
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 ...
-
- As discussed in todays daily scrum meeting we'll change the validators as suggested in the first changeset (with some...
-
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8691 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8691 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8691 -
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 ... -
- The NotEmpty validator has a special meaning as it is executed for empty values (NULL or '') too. "NotEmpty" is a bit...
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7444 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7444 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8770 -
- Currently DateTimeConverter only accepts target types of 'DateTime'.
It should be possible to convert sub-types too. -
- This was already implemented with #27417 (in changeset commit:f8f3fc23121f5959203903cb672e987b4aca57b0)
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8769 -
- The SubRequest is currently missing getter and setter for the request format.
This is required for Fluid for example...
2012-01-29
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8766 -
- The Route class (MVC\Web\Routing\Route) always append additionalParameters etc., with a question mark in front even t...
-
- Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7761 -
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7761 -
- de-assigning because we do not have any consensus if we need that feature.
-
14:47 Bug #32700 (Resolved): Validation does not work for collections
- Applied in changeset commit:dd6f600dca46366e3109a47488bc6c3b607105d9.
-
- Patch set 8 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5401 -
2012-01-28
2012-01-27
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8733 -
- Actually moving the validation run from onFlush to prePersist / preUpdate could solve this. Worth a try.
-
- Bastian Waidelich wrote:
> I thought about this quite a while and I'm pretty sure the new behavior makes much more s... -
- Felix Oertel wrote:
> Ps: I came up with some (EmailAddress OR Empty) syntax, but don't know if this is
> really ... -
- Felix Oertel wrote:
> I would highly prefer to enable proper disjunction usage over breaking the current behavior.... -
- I went into the same problems two weeks ago and tried to come up with a good solution.
I would highly prefer to en... -
- Theoretically it is possible to use another type of entry point than the pre-configured WebRedirect. In practice, how...
-
2012-01-26
-
- Currently all resources need a Database, because the resource manager uses it in order to store resource pointers.
I... -
- There are several ways to bring the system into an unstable state, like activating a package that depends on another,...
-
- The controller method is already annotated with @FLOW3\SkipCsrfProtection
-
- Thanks a lot for the effort, Manuel. I took your suggestion and integrated it into the optimized class loading and bo...
-
- Applied in changeset commit:a655ebced9c650a3d3ce1f754187616ebd24c833.
-
- I found a missing visiblity declaration in \TYPO3\FLOW3\MVC\Web\Routing\Route, and thought I'd search to see if there...
-
- Applied in changeset commit:0acd9937f4cd9b638c436eeea144f05e6038dbdf.
2012-01-25
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8691 -
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7819 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7819 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7819
2012-01-24
-
- For some reason the .Shortcuts sub directory for the new package is not created after a new package has been kickstar...
2012-01-23
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7761 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7761 -
- The use of @getParentLocaleOf()@ in the @CldrRepository@ is wrong - as it only contains Locales for which resources a...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8657 -
- In some places _bool_ is used in docblocks (e.g. for return and param types) instead if the preferred _boolean_.
-
- Thanks, I'll check.
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8656 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8656 -
- There is a difference between joinColumn creation with JoinTable annotation and without for ManyToMany relations. In ...
-
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5731
2012-01-22
-
- Implement proper HTTP 1.1 support.
This issue is a stub for the corresponding FLOW3 1.1 key feature (until I come ...
2012-01-20
-
- Patch set 7 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/6596 -
- Patch set 6 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/6596 -
- Check the configuration in the TYPO3.TcPdf package, it simply adds stuff. Worked for me, so unless I missed something...
-
- Peter Beernink wrote:
> However, when I read the code it looks like one has to copy the default list
> to the Con... -
- Applied in changeset commit:b03e9eaed2f8ba2a7f944dd09e4c54ee800bc07e.
-
- Applied in changeset commit:ad89e84eecd5170b0ce87ac340c1c564acc67983.
Also available in: Atom