Activity
From 2012-02-14 to 2012-03-14
2012-03-14
-
23:14 Bug #34864 (Rejected): inproper escape of attribute names
- hi all,
I named an attribute "order" in my model and thus ran into problems running "./flow3 doctrine:update", sin... -
22:21 Bug #26903: Flush Doctrine metadata cache with FLOW3 caches
- Metadata is a problem that is probably solved, but proxy classes stay around for too long. I deleted a package and th...
-
21:40 Feature #34827: Precompiled reflection data for rather static packages
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9644 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9644 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9644 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9644 -
11:46 Feature #34827 (Resolved): Precompiled reflection data for rather static packages
- Packages which are usually not touched during development (such as FLOW3, Fluid etc.) don't need to be monitored for ...
-
- Patch set 14 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9623 -
- Patch set 13 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9623 -
- Patch set 12 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9623 -
- Patch set 9 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9623 -
- Patch set 8 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9623 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9653 -
19:28 Feature #30256 (Accepted): Package help in CLI
- Showing some mercy to Julle ;)
-
16:38 Feature #30256: Package help in CLI
- Nooooo! I still want this :)
The use-case is this: I know I want to use some command in the kickstarter package, but... -
16:14 Feature #30256: Package help in CLI
- Shouldn't we even close this, with the per command help screen we support already?
-
- This extends the CLI to display a list of all commands of a
specified package including their short description.
Also... -
19:33 Bug #29582: you should be able to pass any kind of objects to action methods
- seems easier in java, in php may be with apc then if it is attached to apc is not a good idea to have inside the fram...
-
- The interesting question is, how is this being called?
-
- not only debugging, maybe you need to introduce a new feature in a live system (you could change the environment in a...
-
- I don't really see the need for this. Debugging on a live machine is bad all the time. Maybe we could introduce a con...
-
- There are other ways to work with your use case. If we create a string to array converter it should actually try to c...
-
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9622 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9622 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9622 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9633 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9633 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9633 -
- See also #34817
-
- I close it due to the Http change from Robert, if it still needs love after we have that chnage we can reopnen a ticket.
-
- We have two more tickets for this, see https://review.typo3.org/#/c/9651/ so I close this, please refer to one of the...
-
- You could use the ProxyClassBuilder directly to build your models from metadata, maybe that is already enough?
-
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9648 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9648 -
15:05 Bug #34837 (Resolved): flow3:cache:flush should care about the .Shortcuts state
- Currently, after renaming a package path (name; from @Foo@ to @Acme.Foo@) manually, @flow3:cache:flush@ resulted in a...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9651 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9651 -
16:24 Feature #30423: Rendering template of other action without forward
- Well, everytime you want to render a template that is not named by the action or you don't want to forward to another...
-
- What would be a use-case for this again?
-
- Actually the logging is not shutdown at all atm. Maybe we could shut it down after the BootstrapShuttingDown Signal?
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9647 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9647 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9647 -
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9650 -
- I will drop this ticket in favor of #29547 which is more concise in its title. The issue is the same.
-
-
- Well, actually it's not about the output. A validator is a prototype that is configured with options. To differentiat...
-
- With the current state of (validation) errors you can pretty much do all that in your output by using the code and ar...
-
- Should we still keep this then?
-
- As I see on the mailing list you provide such a script already, so I would like to close this ticket as the script it...
-
- Is this still the case? From what I see the behavior of remove and clear should be the same.
-
- Okay, the issue would be resolved for me with the new AOP optimizations.
Christian Mueller wrote:
> I will close ... -
- I will close this one, the results of the code sprint are tagged with #9568, and should include this optimization too.
-
- Isn't the current state enough with calling fixFilePermissions after move_uploaded_file()
-
- For me the current behaviour makes sense, as you want a cleanly defined state for the DB in testing. What would be a ...
-
- This is handled in the RequestDispatchingAspect and should work.
-
- Indeed it is obsolete.
-
- Persistable Objects should best be stored in a Doctrine Collection like the ArrayCollection. Then persistence will pe...
-
- Christopher, do you think this is still valid, I did not do an in-deep debugging but looking at current master code, ...
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9461 -
- This works with Doctrine Persistence so I will Reject it for now, as we probably won't work on the Generic Persistenc...
-
- It seems to me this is obsolete with the PropertyMapper.
-
- The "new" PropertyMapper catches this error already and gives a resonable message.
-
- In the meantime the method has changed quite a lot and the limit IMHO makes sense otherwise the output will totally b...
-
- As the currently proposed change introduces a new method to the SessionInterface I would regard this as breaking chan...
-
-
08:33 Feature #34816 (New): Long text encryption
- It should be possible to encrypt / decrypt a long plain text using the TYPO3\FLOW3\Security\Cryptography\RsaWalletSer...
-
- Applied in changeset commit:d8ee47023607e65be59f30ea6c43bc37042f96dd.
2012-03-13
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9629 -
- The fix for the "./flow3" invocation hint on Windows introduced a "ghost command" because methods in command controll...
-
- The fix for #33048 in Ia8561a7878b7dd9ba8a6f3bb926c9da10b46bd0c caused a
ghost command to appear in all command contr... -
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9624 -
- HTML in the code that is output as part of the stack trace is
escaped now.
Change-Id: I0b2a1bcf5ee72a3212dec8b6541f5... -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9622 -
-
- The test improvements are in review now, I will close this, we could decide to deprecate the TextValidator at some po...
-
- Hi,
could you please try https://review.typo3.org/#/c/9606/ with your environment? I am not sure if that works ou... -
- In some cases, @get_class($object)@ returns the Doctrine proxy class; and not the actual, intended class name.
In ... -
- Applied in changeset commit:fec2ff945c3b1a73e0811637fe4bb2ecbce8e6df.
-
- Applied in changeset commit:a29012b669e90a7e39a4727514538ace5a77eb80.
-
- Applied in changeset commit:7e567e8d38e5a24f052a7da4fa7ae0fd657540e3.
-
02:38 Bug #34608 (Resolved): ResourceStreamWrapper::seek gives exactly the wrong bool value
- Applied in changeset commit:461bb056be2d6855aa3def46b4dcbe18fca28cd7.
-
02:38 Bug #34620 (Resolved): Wrong return value for getAccessDecisionVoters()
- Applied in changeset commit:7e055f0b2c7e2d0f92992afd0c97007b50ef4aac.
2012-03-12
-
- This has been removed. The JsonView in the FLOW3 package sets the header correctly, you should do the same if you dec...
-
- Patch set 20 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Patch set 19 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Patch set 18 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/9528 -
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9589 -
- Fix return value to array.
Fix type hinting for var $securityContext and $authenticationManager
Change-Id: I05223d15... -
-
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9587 -
- Calling fseek will always return the wrong value, as fseek
return 0 on success otherwise -1.
The stream_wrapper expec... -
-
- What speaks against symlinking the Framework directory to your projects?
-
- Take a look at Packages/Framework/TYPO3.FLOW3/Classes/Persistence/Aspect/PersistenceMagicAspect.php there we introduc...
-
- Patch set 2 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9584 -
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9584 -
- Annotations like @FooBar(amount=2) lost their parameter value
in renderAnnotation(). This change fixes that.
Change-... -
- Yep, just use your IDE to generate the getters and setters, then you benefit from code completion. If you don't like...
-
-
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9582 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9476 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9476 -
- When using flow3.bat on Windows the usage information would tell the
user to call "./flow3", which doesn't fit the Wi... -
-
- With Doctrine we have JOINS, so this is closed.
-
-
-
-
- When using flow3.bat on Windows the usage information would tell the
user to call "./flow3", which doesn't fit the Wi... -
- Patch set 14 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/5992 -
- Things which have been deprecated prior to the 1.0.0 release are
removed with this change:
* Cache\AbstractFrontend-... -
-
- As Bastian said should be checked on output.
-
- Adrian Föder wrote:
> I'll take care, fine?
More than fine ;)
-
- that's fine, thanks Bastian; I think the examples from the FAQ could almost be transferred 1:1.
I'll take care, fine? -
- Adrian Föder wrote:
> It would be useful to have some more examples how to nest different variables,
> array and Vie... -
- It would be useful to have some more examples how to nest different variables, array and ViewHelpers; for example, ho...
2012-03-10
-
- That matches my expectations to a large degree - sounded like something not really related to our code.
Have fun w... -
10:18 Bug #33297: flow3 package:list throws an error
- Well, i updated apache2, PHP and FLOW3 Versions last weeks and now the error doesnt occur anymore :/
I will try to...
2012-03-09
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9528 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9528 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9528 -
- Patch set 17 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/9518 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9518 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9518 -
- The PackageManager needs to update the shortcuts when saving
package states. Besides that the PackageManager should i... -
- Could you tell us the used PHP version and the amount of files and directories in this folder? And additionally try a...
-
- Maybe a bug in a specific PHP version causing some overflow (Too many open files) when there is a file with #?
Kar... -
16:42 Bug #33158: Problems setting PHP binary path on Windows
- thanks for noticing ;-) and fixed the example... :)
-
- Rens Admiraal wrote:
> Giovanni: are you sure you've no indent errors in your Settings.yaml? I've a 2008 IIS environ... -
- just as a feedback; in my case also the even worse varation with spaces and backward slashes *works*:...
-
- Giovanni: are you sure you've no indent errors in your Settings.yaml? I've a 2008 IIS environment using:...
-
- The Package.php template will generate clean use statements with
same usage of backslashes.
Change-Id: I30e711549119... -
- (I felt free to set to "Accepted" because I think there's enough agree for the need for it)
Sebastian, I read a bi...
2012-03-08
-
19:06 Bug #34675 (Rejected): NotFoundView is not used for missing actions
- When an action is missing instead of an exception the NotFoundView should be used.
-
- Should be injected in the ActionController so a user can overwrite it and show its custom NotFoundView when an action...
-
- Makes sense - could the problem be distilled into a functional test?
-
- When running an application (that uses Collections) in Production context the following exception is thrown:...
-
10:35 Bug #34466: Roles are not cleared after every test in functional tests
- Yes, have the fix already here. Will be pushed together with some content security functional tests...
2012-03-07
-
- Sure, will have a look at it.
@Richard: could you be so kind to add a note with the exact windows version you use? -
- Sure, will have a look this week
-
- Rens, could you check this?
-
-
- The old text can serve as a basis for the "Architectural Overview" section of part 3 of the guide.
-
- I created a file named @#test.html@ in the public resources folder and called @package:list@ - worked fine. I suspect...
-
-
- Yes just like:
#test.html
or something similar -
- Could you give me some more details to reproduce this? Like an offending filename? Thanks!
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9476 -
-
-
- Andi, I set this to accepted - right?
-
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9461 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9461 -
17:23 Feature #34648 (Resolved): Implement a way to authenticate with a file based encrypted key
- For the setup wizard we need a file based encryped key provider. For this we need a password token which only has a p...
-
-
- Robert, is this still a valid issue considering your MVC refactoring? Feel free to close, if not :)
-
- Considered as fixed.
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9145 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9145 -
-
- Right, Doctrine doesn't differentiate between add and update in it's API. So we'd need to do this "on our side".
-
-
- Assigning to Robert as the MVC refactoring should solve this "along the way".
-
-
-
- Ok, here's the ViewHelper which needed this:...
-
- This change would affect *all* users and require adjustment of *all* standard controllers. The advantage is unclear, ...
-
-
- Recent improvements have shown the caching we have in place does the job.
-
- Our approach to keeping things simple means: one configuration format is enough. Especially if not knowing YAML is th...
-
- Maybe this could be achieved with a ViewHelper:...
-
- It would be nice to have this in 1.1 - but I need input from the JS folks on this!
-
-
- Patch set 16 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Applied in changeset commit:ca5b3b9a535ff29a4e34c70bbeb68612d135b0f1.
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9430 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9430 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9430 -
- The return value of the function getAccessDecisionVoters() in the AccessDecisionVoterManager should return a array. I...
-
- Fix return value to array.
Fix type hinting for var $securityContext and $authenticationManager
Change-Id: I75f82560... -
- 11:33 Revision 62d0629d: [TASK] Fix variable annotations (missing prepending backslashes)
- Releases: 1.1
Change-Id: I06d0344ad389e978dd92b8d110b6ae60406184fc
2012-03-06
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9427 -
- The ResourceStreamWrapper does:
@return fseek($this->handle, $offset, $whence);@
The documentation to fseek say... -
- Calling fseek will always return the wrong value, as fseek
return 0 on success otherwise -1.
The stream_wrapper expec... -
- Karsten Dambekalns wrote:
> > If I get it right, this means that the streamWrapper is only instantiated the *first... -
- Bastian Waidelich wrote:
> If I get it right, this means that the streamWrapper is only instantiated the *first* tim... -
- While writing unit tests for the StreamWrapperAdapter, I came upon this issue: @StreamWrapperAdapter::stream_lock()@ ...
-
- That solution still has a flaw, I can’t see all validation errors no more, because the object validation is missing n...
-
-
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9420 -
- Applied in changeset commit:fe76887f75aa80b82be3104ef1f51e065dd774f0.
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9405 -
- The rename function uses the variable $path which should be $path_from. No error occurred so far because the rename f...
-
- Use the proper variable name to create the stream wrapper.
Change-Id: Ia6527b6c95fbe1fcd6cb546a67b7d4b6ac621f66
Fixe... -
- Patch set 15 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Use the propper variable name to create the stream wrapper.
Change-Id: Icc0ced55caf92d8cd23507b55e1413b9783358e2
Fix... -
-
- Applied in changeset commit:e1f1478eba905740584e9990a68cbf6b7d4c0b4c.
-
- Applied in changeset commit:999b95082c1b5695f698a7f640e9eeaec51d9231.
2012-03-05
-
- Patch set 2 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 ... -
19:37 Bug #34545 (Resolved): Hint at wrong usage of @ORM\MappedSuperclass not displayed
- Doctrine 2 allows usage of @MappedSuperclass without @Entity, but in FLOW3 @Entity annotation is necessary. That's wh...
-
- Patch set 14 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Karsten, the NotFoundController way isn't involved in every NotFoundView invokation; the NotFoundView is used every t...
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9296 -
- Under what circumstances is this needed - the NotFoundController sets the status to 404, so out of the box it works a...
-
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9387 -
- StringValidator uses the error id 1238108068, but that is
used in StringLengthValidator. This change makes the former... -
-
- The default NotFoundView, which renders a 404 template,
sets now also the HTTP status to 404.
Change-Id: I01e77d0b3b... -
- The add method in repositores does also update existing entities, this is a dangerous security flaw, as it allows an ...
-
- Hi Sebastian,
indeed I thought about it in "idle time" ;-), I'll write down a few things.
Basically, the design...
2012-03-02
-
- If you do not call authenticateRoles() in one test this will not end in no roles being authenticated as expected. Ins...
-
- Currently method security acls follow a whitelist approach and content security acls follow a blacklist approach. Thi...
2012-03-01
-
-
17:01 Bug #34448 (Resolved): Integrity constraint violation on FLOW3 resource tables
- I have an own model/class/controller, called "award" with a property mapped to the standard FLOW3 resource table.
... -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9297 -
02:38 Bug #33698 (Resolved): ObjectAccess::setProperty must work with numeric indexes
- Applied in changeset commit:d473ee2f0e9409959b594d1198bd4d225d8b367c.
-
- Applied in changeset commit:95e0ca4941f447f898f84298e529a78c44976dc3.
-
02:38
Task #34304 (Resolved): Add abstract classes for entrypoint, provider & token in security authentication
- Applied in changeset commit:2a7a67fca12db09aa970549f36b34a50446d4a26.
-
- Applied in changeset commit:3f288553a845c904596dd51454cbc90a3e89e06d.
-
- Applied in changeset commit:27f00f62768c2b322e87d815e5bef9f5bd2bb2ea.
-
- Applied in changeset commit:74838ac1f5e96f097aac0addaad89940c10b766f.
2012-02-29
-
- Patch set 9 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/6596 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9312 -
- Currently there is no helper method to find the correct class name of a given object. The reflection service should p...
-
- Resolves: #34426
Releases: 1.1
Change-Id: I768171f2f1d402be0846c7f794669ffb7d1125e2 -
- The change to validate collections introduced unwanted behavior
by validating lazy annotated and not yet initialized ... -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9174 -
- The ObjectManager refers to $this->context which doesn't exist.
-
- The ObjectManager refers to $this->context which didn't exist.
Change-Id: I9754acce672472642c71afa23f0bc0041e882279
... -
- Change the behavior of hasRole so it has an early return
when the authenticationManager is not yet authenticated
(and... -
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9302 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9260 -
- AuthenticationManager::isAuthenticated now checks for resumable
and new sessions before returning FALSE, so it works ... -
-
- AuthenticationManager::isAuthenticated now checks for resumable
and new sessions before returning FALSE, so it works ... -
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9290 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9290 -
- Fixes: #34399
Related to: #34398
Releases: 1.1
Change-Id: Ia370abe31f6c08d1f9a4cf2993ef8fb2b1ffb6b7 -
-
-
16:56 Task #8923: Provide a Nginx Server Configuration for FLOW3
- I had to modify the resource url regexp like this to get it working:...
-
- Patch set 5 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9242 - 15:06 Revision 2a7a67fc: [TASK] Add abstract classes for entrypoint, provider & token in security authentication
- Change-Id: Ie3a0a8c3fe43689a70c14e5882c3f5e0d4233250
Resolves: #34304
Releases: 1.1 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9297 -
- The configuration setting for the MVC JsonView allows exposing an object's persistence identifier.
Currently, that c... -
- Duplicate of #34154 with fix under review at http://review.typo3.org/9180
-
- This is a case when 2 domain objects are associated with ManyToMany relation. If the owning side object is created/up...
-
- Andi, I think some better error checking would be helpful here, to detect missing/wrong role names.
-
- This is a duplicate of #33932 and a fix is under review at https://review.typo3.org/9125
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9296 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9296 -
- The NotFoundView currently sends itself out with HTTP status 200, saying it was a 404 error. It should instead say 40...
-
01:12 Bug #34404 (New): JsonView transformObject does not respect _descendAll configuration
- When configuring a jsonView for a domain object, the '_descendAll' configuration is currently not taken into account,...
2012-02-28
-
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9242 -
- Some methods were missing their visibility declarations, this
change adds them.
Change-Id: I88ee085111b38366fc0af5ca... -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9293 -
- FrontendInterface should be public, Route protected (as noted above), the Name fixture public. The others are anonymo...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9290 -
- The default authentication provider was always configured at the first position and could not be removed; which lead ...
-
- Fixes doc comments issues, like missing or wrong @param
annotations. Removes @author annotations.
Change-Id: I785e73... -
- Applied in changeset commit:899eb3f37229244f2938f84d52c655b80c91d759.
2012-02-27
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9242 -
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9262 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9261 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9260 -
- Duplicate of issue #31677
-
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9256 -
- Currently the doc comment of a class is replaced with
"Autogenerated Proxy Class" when the proxy class is generated.
... -
-
02:55 Bug #34293 (Resolved): Typo in TYPO3\FLOW3\Session\Aspect\SessionObjectMethodsPointcutFilter::matches()
- Applied in changeset commit:55150fe526b60d0200b6afd40731b8c36cef1bc4.
2012-02-26
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9242 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9242 -
- There's a lot of duplicated code in the security authentication, we should create some abstract classes to prevent this.
-
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9222 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9220 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9220 -
- l.74: __desctruct should read __destruct
<code class="php">
if (preg_match('/^__wakeup|__construct|__desctruct|__... -
- The filter for proxyable methods in SessionObjectMethodsPointcutFilter skips
the method '__desctruct' instead of '__d... -
- The filter for proxyable methods in SessionObjectMethodsPointcutFilter skips
the method '__desctruct' instead of '__d...
2012-02-25
-
- Patch set 8 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/6596
2012-02-24
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9180 -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9180 -
- Change-Id: I2589eb210140265a829933baeb7d22c0c6f89412
Releases: 1.0 -
- Change-Id: I942d9356d6c7b05418a1fd1c6f005ba108e765de
Releases: 1.0 -
-
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9121 -
- Currently the doc comment of a class is replaced with
"Autogenerated Proxy Class" when the proxy class is generated.
... -
- Applied in changeset commit:b484e6a4f3725091765cdf9c898b81c03245d3d9.
-
- Applied in changeset commit:e7141cb693168c2785c0b7e075ead8c73b96194a.
-
- Applied in changeset commit:7323b900fb1c96e14f36e2cc7b67b064dc01d912.
-
- Applied in changeset commit:9ccbdb00a87a21796f28212c480a35112d014196.
-
- Applied in changeset commit:b24339c953375fc738fca7ad364af78c558186b2.
-
- Applied in changeset commit:2b0b5b96657f731bcdef8ecd7ccbcf3edd4cc5de.
2012-02-23
-
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9121 -
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9149 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9149 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9149 -
- Patch set 4 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9148 -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9148 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9148 -
- If a translation is requested but no (exact) match is found,
a different locale may take effect (e.g. en instead of e... -
- If translateById() is able to find a translation but is not given
any arguments to embed in the output, NULL is retur... -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9147 -
- Patch set 2 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9146 -
- Patch set 4 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 had to be given explicitly. This... -
- To control the behavior of the i18n component a way to
configure the "current" locale, fallback rules for resolving
o... -
- Allows to set a fallback order for locale resolving. If not set,
the implicit inheritance of locales will be used. Th... -
2012-02-22
-
- Currently if you connect a $classname + $methodname as slot it will instantiate the class to call $methodname on the ...
-
- Applied in changeset commit:873bdd5376defbd863929258b2e52f63b8b73dbe.
-
- Patch set 6 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8986 -
14:34 Revision 8fca8ee3: [FEATURE] Shorthand syntax for custom validators
- This change allows the Validate annotation for properties to
be given a value like "Mypkg:My". This is equivalent to ... -
- Patch set 3 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8770 -
- Patch set 3 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 thro... -
-
-
- Patch set 3 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/9149 -
- If @translateById()@ is able to find a translation but is not given any arguments to embed in the output, NULL is ret...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9148 -
- If a translation is requested but no (exact) match is found, a different locale may take effect (e.g. en instead of e...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9147 -
- Currently the only fallback applied when resolving translations is the implicit chain in locale, e.g. en_UK > en
I... -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9146 -
- The behavior of the i18n component needs to be configurable, down to per request in some cases. Thus a configuration ...
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9145 -
- To use a @Resource@ with file operations, one can use the @resource://…@ scheme and the @__toString()@ method of any ...
-
- This also happens when loading a single entity with lots of connected entities (even when those entities are connecte...
2012-02-21
-
- In the onFlush method of the PersistenceManager all objects are validates, this is a problem especially for many conn...
-
15:37 Bug #33158: Problems setting PHP binary path on Windows
- The flow3.bat also has a path to php.exe. So you should also change this to your local php path to make this work.
-
- Hi.
Jacob Floyd wrote:
> Do we really want to support using session.use_trans_sid?
I don't give a dime about t... -
12:06 Feature #34133: RFC: Handle Semicolons in Path part of URIs as Scoped Path Parameters
- Karsten Dambekalns wrote:
> Without having read everything yet, I'd like to point to the arg_separator.* directives ... -
- …moved to original description.
-
- Without having read everything yet, I'd like to point to the arg_separator.* directives of PHP (see http://www.sitepo...
-
- h1. Scoped Path Parameters
The URI RFCs hint at what I think will make an awesome, unique, feature for Flow3, and ... -
- Here is a better solution so you can still get all errors for all properties and you still won’t get an exception eve...
-
- In the case of models with non-optional constructor arguments, you get an exception when one of those subproperties h...
-
- Patch set 13 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/8868 -
- Patch set 11 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Patch set 10 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Patch set 9 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868
2012-02-20
-
- Patch set 1 for branch *FLOW3-1.0* has been pushed to the review server.
It is available at http://review.typo3.org/9129 -
- 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... -
-
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9125 -
- Annotations like @FooBar(amount=2) lost their parameter value
in renderAnnotation(). This change fixes that.
Change-... -
- Patch set 1 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/9121 -
- The Proxy Class replaces the class doc comment of the original class with ...
-
- This also causes the DateTimeValidator to not work
2012-02-19
-
- Patch set 8 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- Patch set 7 for branch *master* has been pushed to the review server.
It is available at http://review.typo3.org/8868 -
- @robert - I've seen/heard your comments in the scrum protocols/meeetings about this HTTP work you're doing.
I've b...
2012-02-17
-
- 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:...
-
- 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 -
- 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 -
- Adds some "required" attributes to the XLIFF files.
Change-Id: Ib30f5653197229c5d2062eba2c0020ca6e191216
Resolves: #... -
- 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.
-
- StringValidator uses the error id 1238108068, but that is
used in StringLengthValidator. This change makes the former... -
- 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...
-
- Change-Id: I8c8c4d19b673d02339369d2a16961441ad65bdc5
Resolves: #33970
Releases: 1.1 -
- 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 -
- This fixes the FileBackend so it actually uses the default lifetime
in cases it should do that.
ArrayAccess detectio... -
-
- Patch set 2 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 did not
differentiate between filesystem and parsing er... -
-
-
- It would be helpful to have an easy way to resolve the 'resource://...' path
to static resources.
Currently we end... -
-
- Johannes,
we "plan" anything you like :) We're currently in the absolut early stage of collecting some ideas and imp... -
- In the TYPO3\FLOW3\Object\Proxy\Compiler::renderAnnotation function, there is currently no case for numeric annotatio...
2012-02-14
-
- 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
-
- Applied in changeset commit:cebbf8e2851b28ffaf32abafbb675e47af6ae783.
Also available in: Atom