Activity
From 2011-12-21 to 2012-01-19
2012-01-19
-
21:02 Task #33142: Ignore Tags / Annotations in external libraries, find some good solution
- Karsten I haven't tested your code yet, but it looks like a good solution.
However, when I read the code it looks li... -
17:49 Task #33142: Ignore Tags / Annotations in external libraries, find some good solution
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7747 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7747 -
17:48 Task #33142 (Accepted): Ignore Tags / Annotations in external libraries, find some good solution
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7675 -
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/7861 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5731 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5731 -
10:27 Bug #33311 (Resolved): AuthenticationProviderManager::isAuthenticated does not work in authentication request
- If the client does not have a session and the client authenticates, then a new session is created. The method isAuthe...
2012-01-18
-
19:38 Bug #33300 (Resolved): 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...
-
- IMO TextValidator should be removed as it depends on the context whether a string is insecure or not (also see commen...
-
17:38 Bug #33297 (Closed): flow3 package:list throws an error
- Hi together,
thanks for FLOW3. I'm developing for about one year with it and now i discovered a bug which is not a... -
16:41 Bug #33293 (New): Injection to private variable results in injection of the the wrong class
- If you annotate a private declared class variable to inject some class this annotated class won't be injected. Rather...
-
- Applied in changeset commit:f53be7acc8707652a6d76a8dc302601ffd35d15c.
-
- Applied in changeset commit:68d7cf650364b32f17f0af7178147aa5ed1d4ac0.
-
03:26 Bug #32984 (Resolved): ArrayMergeRecursiveOverrule does not override arrays with simple types
- Applied in changeset commit:b7e76ffd7eed0fb6648d07ec0d5cf109f3ff8568.
-
- Applied in changeset commit:2fdc342d181d816f6a6ee41b7b30500f3a8741d7.
-
03:26 Feature #33112 (Resolved): Create .Shortcuts directory if it is missing
- Applied in changeset commit:8dd2ea5352ca1fafb38fbe1d2b6b26b6f432e985.
-
- Applied in changeset commit:9c421d6401df8d0e45ede739041507d38f9d2fe4.
2012-01-17
-
- Christopher Hlubek wrote:
> The class would not match @MyPackage\Domain\Model\Poo@ [...]
IMO the users shouldn'... -
16:41 Task #33268 (Resolved): Document new YAML behaviour for Upgrade to 1.1
- The YAML parser changed its behaviour in some intricate ways. One of it is the (now correct) processing of escape seq...
-
- We still need a clean solution to handle assets; and https://github.com/kriswallsmith/assetic would be a great fit to...
2012-01-16
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7819 -
- Validation options are set through the constructor but there is currently no way of accessing the options afterwards....
-
- I think, this one can be closed as a validator for malicious HTML doesn't make sense IMO:
The rules for *malicious* ... -
11:46 Bug #33055: AccessDeniedException instead of WebRedirect
- It is the first call to the Site. Meaning a user calls a domain like: http://mydomain
This url is linked to a site, ...
2012-01-14
-
11:47 Bug #33191 (Rejected): TypeConverter from string to array
- I discovered this bug by using a <f:form.select name="items" /> in a form. In case the user doesn't select anything, ...
2012-01-13
-
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7781 -
- The AuthenticationController and some related code show some tiny dents if looking closely :)
-
-
-
-
- Adrian Föder wrote:
> If this is considered common guideline, I suggest we should only allow customized annotation a... -
01:11 Bug #33158 (Closed): Problems setting PHP binary path on Windows
- Windows Server 2008 R2 SP1
Apache 2.2 PHP 5
I still set FLOW3.core.phpBinaryPathAndFilename in Configuration/Setti...
2012-01-12
-
17:57 Bug #33148: CLDR locale chaining doesn't work correctly
- Test is here: -https://review.typo3.org/#change,7750
Please disregard the "Depends on"; just see the test.-
https://r... -
- Simply said, the chaining in order to get also de.xml when de_DE.xml is asked for, doesn't work correctly.
The pro... -
- With the following constellation
Example.php
containing the class Example with a scope singleton
and
Exampl... -
- Well for now, we have afaik two possibilities how external libraries could get involved:
# if it's just a tiny helpe... -
13:09 Task #33142 (Resolved): Ignore Tags / Annotations in external libraries, find some good solution
- for now we have the ignore list of tags in Packages/Framework/TYPO3.FLOW3/Classes/Reflection/ReflectionService.php
... -
- cool Marc, thanks for the update :-)
Greetings, Sebastian -
11:27 Feature #32106: Support for Object source in PropertyMapper
- Hey Sebastian,
unfortunately i've been drowning in Work the last 3 Weeks :/
i promise to get back to this as so... -
- Hey Marc,
do you need any help on this, or anything else I can do to support you here?
Greets,
Sebastian -
- If you create a CommandController which is not in the /Controller/ directory of a package you will get an error sayin...
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7734
2012-01-11
-
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7507 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7735 -
- FLOW3 currently creates or updates the Packages/.Shortcuts/ directory only if the package states have changed. That i...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7734 -
- One example is @offsetGet()@ - Generic returns NULL if the key doesn't exist, Doctrine crashes. This needs to be chec...
2012-01-10
-
19:37 Bug #33055: AccessDeniedException instead of WebRedirect
- Did you try to call the protected action manually, or via a Fluid generated link?
I'm asking, because to call protec... -
- Did you try to call the protected action manually, or via a Fluid generated link?
I'm asking, because to call protec... -
12:01 Bug #33078: No Redirect to Login
- Similar to http://forge.typo3.org/issues/33055
-
- I think this is a Bug with the Security Subsystem.
I'm only getting a Exception when i'm not login or i've no righ... -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7723 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7723 -
- as of http://php.net/function.preg-match the function could also return FALSE if an error occurs (see Return Values a...
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5918 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/6859 -
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/7716 -
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/7715 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7513
2012-01-09
-
19:40 Bug #33048: flow3.bat gives wrong feedback
- Bastian Waidelich wrote:
> > You need to specify the .bat!
>
> leaving it out works for me in the default CMD and... -
- > You need to specify the .bat!
leaving it out works for me in the default CMD and msysgit interface on Windows 7 ... -
- Following the Unix rules we should make the built-in command output less verbose. We should introduce a generic verbo...
-
10:05 Feature #33058 (Rejected): Choice for XML Configuration
- Though YAML seems to be much simpler then XML, but companies whose staff is trained to use XML instead of YAML. It wo...
-
- After defining some restrictions via ACLs on a controller method and defining a WebRedirect I get an AccessDeniedExce...
2012-01-08
-
- No sorry, your wrong.
You need to specify the .bat!
Windows couldn't handle the request of flow3 help
This has two... -
- > The correct response should be:
> FLOW3 1.0.2 ("Development" context)
> usage: *flow3.bat* <command identifier>
... -
- Hi,
this is just a very little issue, but your flow3.bat gives some wrong feedback.
If i callflow3.bat I curre... -
- Hi there,
I found some configuration in some of the Framework Packages corresponding the persistence:core:context.... -
12:12 Task #33040 (Rejected): rename the default controller as DefaultController
- The code could be more readable if the default controller is renamed from StandardController to DefaultController
2012-01-06
-
- They also do not work in widget (e.g. pagination does not work in protected actions!)
-
- Patch set 7 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5401 -
- Patch set 6 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5401 -
16:38 Bug #33024 (Accepted): Exception when validating a float in a Model with the Number validator
- Warning: array_merge(): Argument #1 is not an array in /Data/Temporary/Development/Cache/Code/FLOW3_Object_Classes/TY...
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7513 -
- I have tested your soluction and works quite well
So the best way to use doctrine event listeners is:
//Event lis... -
- We are somewhat limited in the extensibility of translations with the current approach. The use case would be a packa...
2012-01-05
-
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/3245 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/3245 -
-
- Ok, I finally wanted to se if this goes away when not adding any event listeners. So I checked out the AbstractContro...
-
-
- Please, using cascade should never be needed. FLOW3 determines this from the aggregate boundaries.
In this particl... -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7680 -
11:00 Feature #32990 (Rejected): Account party property should be persistable through the account
- Before:
* @ORM\ManyToOne(inversedBy="accounts")
After:
* @ORM\ManyToOne(inversedBy="accounts",cascade={"all"})
... -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7681 -
- In https://review.typo3.org/5756 the default hashing strategy was changed to BCrypt.
Later, in https://review.typo... -
- Bastian Waidelich wrote:
> I'm not too happy with this, to be honest, for a few reasons (just IMO of course):
I a... -
- I'm not too happy with this, to be honest, for a few reasons (just IMO of course):
* arrayMergeRecursiveOverrule() i... -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7677 -
- *This is an improvement to the @Utility\Arrays::arrayMergeRecursiveOverrule@ function, leading to enormous flexibilit...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7676 -
- While working on some improvements for the Configuration subsystem, I stumbled over the following issue in arrayMerge...
-
-
- Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7421
2012-01-04
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7675 -
- Yes, in FLOW3 all entities must be marked as such. The entity declaration has more significance than only for persist...
-
- The solution that is also recommended by the Doctrine people is to not use reserved words. You can specify another co...
-
-
- If you are really running 1.0.0, please try again with (at least) 1.0.1, as this sounds like #30040, which has been f...
-
-
- I can reproduce the segmentation fault with the updated PoC code.
It occurs with current master and is not fixed w... -
-
- Sebastian Kurfuerst wrote:
> can you point me to the place where this is evaluated? I just grepped the full FLOW3 so... -
-
10:31 Bug #32959 (Resolved): Exception thrown on missing role definition
- Notice: Undefined index: Visitor in C:\f3cache\Development\Cache\Code\FLOW3_Object_Classes\TYPO3_FLOW3_Security_Polic...
2012-01-03
-
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7421
2012-01-02
-
- This should work without changing the scope annotation since it is prototype by intent (to allow different configurat...
-
- @\TYPO3\FLOW3\MVC\Controller\ControllerContext@ could use some cleanup:
- arguments (\TYPO3\FLOW3\MVC\Controller\Arg...
2011-12-29
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7635 -
- Scenario: Change accountIdentifier or credentialsSource on an account (\TYPO3\FLOW3\Security\Account), while being lo...
-
- You have to configure a FQCN for tokeClass config. This is not consistent with providerClass that doesnt need to use ...
2011-12-28
-
- Karsten Dambekalns wrote:
> Ok, what you want to do is set the default hashing strategy. But your approach is wrong....
2011-12-27
-
- And can get closed now, #26885 is resolved
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7421
2011-12-24
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7507 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7507 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7507 -
13:22 Bug #32829 (Resolved): Mapping to a subclass by __type property fails if subclass adds fields
- I have a hierarchy of classes for which an object should be created from a form. For that, I use the feature added wi...
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7515 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7515 -
- Sometimes, validation should only be executed on persist, sometimes on the controller invocation. Especially when bui...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7513 -
- Hey everybody,
I am currently experiencing an error with the Validate-On-Persist:... -
- already merged long ago
-
01:37 Bug #32827 (Closed): TYPO3\FLOW3\Exception thrown in file Bootstrap.php
- It seems like the PHP binary "C:/Program Files (x86)/EasyPHP-5.3.8.1/php/php.exe" cannot be executed by FLOW3. Set th...
2011-12-23
-
22:05 Bug #32826 (Resolved): Resource PublishingConfiguration is not persisted (missing Doctrine annotation)
- The publishingConfiguration property of the Resource does not get persisted. An abstract publishing configuration cla...
-
18:33 Bug #30536 (Resolved): URI builder does not work correctly with arguments of subrequests
- Applied in changeset commit:6629f53025e73b73d2c04d043f561637b1fa23be.
-
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7219 -
- Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/6596 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7499 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7499 -
- ... this should be enforced through a DB constraint.
2011-12-22
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7442
2011-12-21
-
- Applied in changeset commit:fa850f3efd00d37e5124a81c2d809ffdf79349df.
-
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/7471 -
- The same error as we already had it for master now appears on the FLOW3-1.0 branch.
On the CI server we see errors... -
- Applied in changeset commit:12eea665d94747288a82a9a0e02094ac6ab8d42b.
-
- Patch set 4 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/7293 -
- Applied in changeset commit:f95aa8385b91967eb342513d08dbdbc4f87cb38e.
-
- Patch set 2 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/7439 -
- Applied in changeset commit:6ca1f9b2734e083f0f707af0d2f83f7bf5da87de.
-
- Applied in changeset commit:6ba03e4810791eb15f43282789b8dd29d8d57c09.
-
- Applied in changeset commit:d5533b0bb05c88853df170a4b988984e459552c3.
-
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/7464 -
- Applied in changeset commit:38270b0d017279ee6d03441e00f740b18160a709.
-
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/7466 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/7436 -
- ah, ok, thanks a lot... That's funny because my "solution" also works... but thanks; I'll change it.
Do you need f... -
- Ok, what you want to do is set the default hashing strategy. But your approach is wrong. Change the default instead i...
-
- add. information:
@Objects.yaml@ is set to ...
Also available in: Atom