==================== 2.0.0-beta1 ==================== ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Base Distribution ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ [TASK] Update composer.json with new dependencies ----------------------------------------------------------------------------------------- * Commit: `7cdcca7 `_ [TASK] Rename Vendor to Libraries ----------------------------------------------------------------------------------------- * Resolves: `#42013 `_ * Commit: `c46b3c3 `_ [TASK] adjust to FLOW3 renaming ----------------------------------------------------------------------------------------- * Commit: `749ce77 `_ [!!!][TASK] Composerify base distribution ----------------------------------------------------------------------------------------- Removes submodules in favor of installing via composer. Files installed via composer from the FLOW3 package are removed as well. * Related: `#41031 `_ * Commit: `3922aea `_ [TASK] Update phoenix project name ----------------------------------------------------------------------------------------- * Commit: `709e9a1 `_ [TASK] Do not recurse into submodules for Phoenix distribution ----------------------------------------------------------------------------------------- * Commit: `4d74d89 `_ [TASK] Add Phoenix distribution build surf configuration ----------------------------------------------------------------------------------------- Note: This needs the corresponding change in BuildEssentials as well. * Resolves: `#38500 `_ * Commit: `218bcbf `_ [TASK] Adjust release deployment descriptor ----------------------------------------------------------------------------------------- This adds the configuration needed to push tags based on the PUSH_TAGS environment variable and tweaks some tiny (style) issues. * Commit: `34b5bfb `_ [TASK] Mention PhpSession configuration change in Upgrading.txt ----------------------------------------------------------------------------------------- This informs about the PhpSession settings key change. * Related: `#37993 `_ * Commit: `839d054 `_ [BUGFIX] Remove dead symlinks from release archives ----------------------------------------------------------------------------------------- The default Surf configuration symlinks Data and Configuration to a shared folder. For release archives that leads to dead symlinks being in the archives. This changes disables both symlink tasks in the release deployment descriptor. * Fixes: `#37569 `_ * Commit: `96c47dd `_ [TASK] Adjust Surf deployment descriptor ----------------------------------------------------------------------------------------- The deployment descriptor used to release archives of FLOW3 needed a little adjustment. * Commit: `5971c56 `_ [TASK] Clean Upgrading.txt from 1.1 content ----------------------------------------------------------------------------------------- * Commit: `402ac49 `_ [TASK] Update composer.json with new dependencies ----------------------------------------------------------------------------------------- * Commit: `aa45f04 `_ [TASK] Set FLOW_VERSION_BRANCH to 2.0 ----------------------------------------------------------------------------------------- * Commit: `36817a6 `_ [TASK] Remove API annotation from View::canRender() ----------------------------------------------------------------------------------------- With #40888 View::canRender() is no longer called from within ActionController::resolveView(). As the framework no longer relies on that method to exist the @api annotation is removed. Besides this marks EmptyView deprecated. * Related: `#40888 `_ * Related: `#42176 `_ * Commit: `6f03a9a `_ [TASK] Update references in the Guide ----------------------------------------------------------------------------------------- Another update to references forvalidators, converters and Fluid ViewHelpers. * Commit: `14c861f `_ [TASK] Update Readme.txt and Upgrading.txt files ----------------------------------------------------------------------------------------- This updates the Readme.txt and Upgrading.txt files of TYPO3 Flow for the release of 2.0. * Commit: `80850d0 `_ [TASK] Update Package Management section ----------------------------------------------------------------------------------------- This adjusts the package management section of the TYPO3 Flow Guide to the composer changes. * Resolves: `#42726 `_ * Commit: `6606d48 `_ [!!!][TASK] Use \\ instead of _ for entity resources in Policies ----------------------------------------------------------------------------------------- When defining entity resources in a Policy.yaml file, the entity type had to be the class name with \\ replaced by _. With this change the regular class name is to be used instead. A code migration to adjust Policy.yaml files is part of the change. * Resolves: `#43629 `_ * Commit: `e6c6ac5 `_ [TASK] Update Quickstart and Getting Started installation instructions ----------------------------------------------------------------------------------------- Updates the download instructions to talk about composer instead of unpacking archives. * Commit: `6e18986 `_ [BUGFIX] Invoking CLI from a different directory doesn't work ----------------------------------------------------------------------------------------- When executing the TYPO3 Flow CLI from a directory that is not equal to the root path and if the FLOW_ROOTPATH environment variable is not set, Flow dies with an exception. This change fixes this by falling back to the current directory in that case. * Fixes: `#31642 `_ * Commit: `81ed883 `_ [!!!][BUGFIX] Route defaults have to be specified when creating URIs ----------------------------------------------------------------------------------------- Currently when creating URIs one can omit route values that are defined as defaults of the route. This is not the intended behavior though, because with a lot of routes in place this leads to the situation where it gets difficult to target a certain route. Thus for a route to match the given route values, one should have to specify all the routing defaults exactly as they are specified by the route (except for when a route part appears in the uriPattern). This is a breaking change if you used Router or UriBuilder to create URIs without specifying all defaults (see #43589 for details). Besides, all Fluid ViewHelpers that use the UriBuilder now require the action argument to be specified. * Fixes: `#43589 `_ * Commit: `f17a227 `_ [BUGFIX] Use correct DomCrawler (again) ----------------------------------------------------------------------------------------- The latest manifest update downgraded the DomCrawler, breaking some Fluid tests. * Related: `#43822 `_ * Commit: `0d62cf6 `_ [FEATURE] Add array and object support for NotEmptyValidator ----------------------------------------------------------------------------------------- This patch resolves the problem that empty arrays are not correctly validated when using the NotEmpty validation annotation. Arrays and Countables that are empty will now trigger an error. Follow-up patch to https://review.typo3.org/14707 * Resolves: `#42467 `_ * Commit: `d228933 `_ [BUGFIX] Handle missing - but registered - packages gracefully ----------------------------------------------------------------------------------------- In production a missing, but registered package leads to a fatal error. Handle this situation gracefully. Allow logging early on in the package manager. * Fixes: `#42342 `_ * Commit: `f0ef163 `_ [TASK] Replace references to "Flow 1.2" ----------------------------------------------------------------------------------------- This fixes some documentation/comments referring to "Flow 1.2" * Fixes: `#43839 `_ * Commit: `886da55 `_ [BUGFIX] Throw ViewNotFoundException if view could not be resolved ----------------------------------------------------------------------------------------- This improves the resolving of views by removing the canRender() checks from ActionController::resolveView as discussed in #42176. Besides this skips the creation of the obsolete NotFoundView in favor of throwing a ViewNotFoundException. This also adds overdue unit tests for the ActionController. * Fixes: `#40888 `_ * Fixes: `#40857 `_ * Related: `#42176 `_ * Commit: `9f99c84 `_ [FEATURE] Taggable sessions ----------------------------------------------------------------------------------------- This introduces the ability to tag a session and retrieve remote sessions by tag through the Session Manager. * Resolves: `#43832 `_ * Commit: `3a2c116 `_ [TASK] Prepare composer manifest for release ----------------------------------------------------------------------------------------- Adjust some dependencies to pull in new stable releases only, instead of the latest development version. * Related: `#43822 `_ * Commit: `7552f05 `_ [TASK] Update references in the Guide ----------------------------------------------------------------------------------------- Freshly rendered references for commands, validators, converters, ViewHelper and annotations. * Commit: `96cb52d `_ [BUGFIX] Fix race condition in session shutdown ----------------------------------------------------------------------------------------- This protects sessions against being revived through the shutdown method even though they were destroyed remotely in the meantime. This patch also contains a small modification and related test to make sure that incoming session cookies are not blindly sent back to the user agent in the response. Instead, a clean, new session cookie with the parameters set in Flow's settings is created. * Resolves: `#43673 `_ * Commit: `4dac593 `_ [TASK] Add reference for predefined constants to documentation ----------------------------------------------------------------------------------------- This adds a new page in Chapter V, Appendixes, listing the predefined constants Flow introduces, and their intended meaning. * Resolves: `#43609 `_ * Commit: `ab44a1b `_ [FEATURE] Allow configuration processing in code migrations ----------------------------------------------------------------------------------------- This adds the new method ``processConfiguration`` to AbstractMigration. It is given a configuration type, a processor and an optional flag stating whether or not the processed configuration should be saved. The processor is a Closure and is handed the parsed configuration of the package under migration, for each found file separately. If you need to modify the configuration, make sure to use pass-by-reference for the processor configuration parameter in addition to setting the $saveResult flag to TRUE. * Resolves: `#43089 `_ * Commit: `a3c0c8b `_ [TASK] Make system call to php binary reliable, especially on Windows ----------------------------------------------------------------------------------------- This solves execution of php binary on windows environments with paths containing whitespace and special characters (like round brackets). On windows, commands containing white spaces must be quoted, but quoted paths may not be escaped, which escapeshellcmd() wrongly does. This is solved by using escapeshellarg() on windows instead and also adds more verbose exception messages in the case of failure. * Resolves: `#35916 `_ * Commit: `17c31e3 `_ [BUGFIX] Fix session usage in functional test cases ----------------------------------------------------------------------------------------- This fixes a bug in the FunctionalTestCase which resulted in a fatal error in Session->start(). Testable HTTP is now always enabled. The enableTestableSecurity() method (deprecated since 1.1) has been removed as well. * Fixes: `#43590 `_ * Commit: `5629be5 `_ [TASK] Make getPropertyInternal() protected ----------------------------------------------------------------------------------------- The getPropertyInternal() method obviously is intended for internal use in the ObjectAccess class. This change makes it protected. * Resolves: `#43617 `_ * Commit: `8d7e2cf `_ [FEATURE] Destroy session explicitly in logout ----------------------------------------------------------------------------------------- This change moves the call to Session->destroy() from a signal to the AuthenticationProviderManager that already has the current session as a dependency. The "emitLoggedOut" signal is emitted before the session is destroyed to allow other slots to inspect the session. * Resolves: `#43575 `_ * Commit: `15e1aa0 `_ [BUGFIX] Fix arguments check in buildConstructorInjectionCode, allow NULL ----------------------------------------------------------------------------------------- It was not possible to pass NULL as constructor arguments, because of the use of isset() instead of array_key_exists(). Also a few other places needed to be changed and some tiny (unrelated) tweaks were made. * Fixes: `#38988 `_ * Commit: `798d2b3 `_ [FEATURE] Touch remote sessions and tag with account info ----------------------------------------------------------------------------------------- This introduces two new features: updating the last activity timestamp of a remote session through a new method "touch()" and tagging sessions with their respective account (if any). * Resolves: `#43491 `_ * Commit: `c268993 `_ [!!!][FEATURE] Remote session management ----------------------------------------------------------------------------------------- This introduces the foundations for remote session management – that is, inspection, modification and destruction of sessions which are not currently active in the browser running the current HTTP request This patch also REMOVES the PhpSession implementation and configures the new, Flow-native implementation to be the default session handler. If you provided specific configuration for session cookies through TYPO3:Flow:Session:PhpSession:*, you will need to adjust your settings to use options in TYPO3:Flow:Session:* instead. This change set also contains a new type converter which is capable of converting a session identifier to a session object. * Resolves: `#43376 `_ * Commit: `986ad3a `_ [TASK] Revert exclusion of classes with broken PSR-0 in DomCrawler ----------------------------------------------------------------------------------------- This reverts commit ee8f8e1d898c15549c22278e3fe80cd5702df182 (source is Id3fbb8ad5154c633b7153b4f8eef742f385b05a3). The fix is no longer needed, as DomCrawler has been fixed, see the pull request at https://github.com/symfony/DomCrawler/pull/7 * Related: `#41031 `_ * Commit: `4f162e2 `_ [BUGFIX] Regard wildcard in PropertyConfiguration shouldMap() ----------------------------------------------------------------------------------------- Using 'name.*' in property mapping configuration did not work for fields added on the client, because shouldMap() would reject those fields before the configuration was even checked. * Fixes: `#43387 `_ * Commit: `bd0acd4 `_ [BUGFIX] Check for installation of phpunit by composer ----------------------------------------------------------------------------------------- Ib4b6c68c03590444addb7c2e5af89b2a2d90051f introduced a regression for running functional tests when composer is used in the project, but PHPUnit is still installed using PEAR. This change adds an additional check to verify if phpunit is actually installed in the Libraries folder. By doing so the Bootstrap will fall back to the PEAR installation of phpunit. * Resolves: `#43377 `_ * Commit: `a495545 `_ [BUGFIX] Exclude parameters of the current request from created URIs ----------------------------------------------------------------------------------------- Since https://review.typo3.org/#/c/15261/ the arguments of the current/uri-building-origin request were added without although addQueryString was FALSE. The UriBuilder now only keeps the requestArguments on non-involved requestNamespaces. * Resolves: `#42770 `_ * Commit: `feeb87c `_ [TASK] Cosmetic fixes in Http Cookie and Response ----------------------------------------------------------------------------------------- This adds a missing type hint in Cookie and moves the status code array in Response to a method: that way all the status codes don't appear in debug backtraces anymore. * Commit: `75ba1c1 `_ [BUGFIX] Package loading must be ordered by dependencies ----------------------------------------------------------------------------------------- In order to ensure proper merging of configurations packages most be loaded ordered by dependencies. This should be handled by ordering PackageStates accordingly. * Fixes: `#42730 `_ * Fixes: `#28672 `_ * Commit: `f9100b0 `_ [!!!][BUGFIX] Respect limit/offset constraints when counting results ----------------------------------------------------------------------------------------- Until now QueryResult::count() discarded limit constraints that were active for the affected query, so $query->setLimit(3)->execute()->count() could return a value > 3. If the query had an offset Flow even threw an exception. This changeset adjusts the Query so that limit and offset constraints are taken into account. It furthermore unsets the offset in the query in order to circumvent the exception. This is a breaking change if you relied on Query::count() to always return *all* results regardless of a specified limit. In this case better generate multiple QueryResults (they're only created in-memory until you actually access them) * Fixes: `#42955 `_ * Commit: `d46eef4 `_ [TASK] Code cleanup ----------------------------------------------------------------------------------------- This change * fixes a number of CGL violations and typos * changes public to protected in PolicyService * removes an @api annotation * adds a @throws annotation * removes an unused use statement * Commit: `0695c46 `_ [FEATURE] Remove trailing slash from routes for empty subroutes ----------------------------------------------------------------------------------------- Flow throws an exception if a route contains a trailing slash in order to avoid duplicate slashes in URIs and to make it possible to add some kind of global default suffix to all URIs. When using sub routes and a prefix with a trailing slash in the uriPattern of the main route (e.g. uriPattern: 'blog/') this leads to an unwanted exception if there is an empty (fallback) subroute because the merged uriPattern would contain a trailing slash ("blog/"). This change strips trailing slashes from merged uriPatterns if the sub routes uriPattern is empty. * Resolves: `#31777 `_ * Commit: `4690433 `_ [BUGFIX] Fix handling of paths with dots in getLocalizedFilename() ----------------------------------------------------------------------------------------- When a path like resource://TYPO3.Neos/Public/JavaScript/ is handed to getLocalizedFilename(), it tries to check for the existence of resource://TYPO3.en.Neos/Public/JavaScript/ - which breaks, unless you happen to have a package TYPO3.en.Neos activated. This change fixes this by using the basename in the check for an extension. * Fixes: `#43195 `_ * Commit: `fefb00e `_ [TASK] Add doctrine /bin/ folder to .gitignore file ----------------------------------------------------------------------------------------- * Commit: `28c27ed `_ [TASK] Allow use of composer-installed PHPUnit in Testing context ----------------------------------------------------------------------------------------- The Testing context loads PHPUnit for functional tests. If using PHPUnit installed via composer, this needs to be handled differently than for a PEAR-installed setup. * Commit: `0e1ac1c `_ [BUGFIX] FlowSession: Fixed renewId() ----------------------------------------------------------------------------------------- This patch provides a new implementation of FlowSession's renewId() method. It previously loaded all data from the old cache entry in order to write it into a cache entry with the new session identifier. This effectively overwrote any existing session-scoped object which existed in the old cache data. The new implementation uses a session identifier independent internal storage identifier which doesn't change on renewing the public session identifier. This way we don't need to move around session data, but can simply store a new mapping between session id and storage id. This patch also lets the HTTP Request store only the baseUri instead of the whole settings array which leads to a smaller footprint and less information stored in a session (as the current request is, in most cases, part of the serialized session data). * Resolves: `#43110 `_ * Commit: `1b6267f `_ [TASK] Rename csrfStrategy and csrfTokens member variables ----------------------------------------------------------------------------------------- The csrfStrategy and csrfTokens member variables have weird names, since they protect against CSRF they should rather be named csrfProtectionStrategy and csrfProtectionTokens. This change adjusts the Security\\Context internals accordingly. * Commit: `28f170d `_ [FEATURE] Add logging of resource access decision voting ----------------------------------------------------------------------------------------- This change adds a logging aspect for decideOnResource() to log votes analogous to what is done for decideOnJoinPoint(). * Resolves: `#43091 `_ * Commit: `39dd293 `_ [TAKS] Tweak and clean up code migration ----------------------------------------------------------------------------------------- This change cleans up the code a little by fixing some CGL violations and moving some code around. Packages in Libraries are now skipped by default (was Vendors before). If available, composer's autoload.php is used instead of manually requiring class files. * Commit: `8a1e24b `_ [TASK] Fix tiny FLOW3 leftover in docblock ----------------------------------------------------------------------------------------- * Commit: `8daa495 `_ [TASK] Update documentation to composer changes ----------------------------------------------------------------------------------------- This updates the documentation to fix the most prominent errors and inconsistencies regarding package handling and file locations that were caused by the composer changes. * Related: `#41031 `_ * Commit: `98395c7 `_ [BUGFIX] Fix FlowSession initialization ----------------------------------------------------------------------------------------- This patch fixes the FlowSession initialization for non HTTP-Requests. Previously the initializeObject() method was trying to retrieve the HTTP request even from a CLI request handler, resulting in a fatal error. Additionally this patch now defines a default storage backend (FileBackend) for FlowSession. Therefore it's now sufficient to enable FlowSession in Objects.yaml for getting started. The garbage collection now is more reliable and triggered when a session is resumed instead of when the shutdownObject() method is called. * Resolves: `#42801 `_ * Commit: `beac827 `_ [BUGFIX] Fix composer manifest detection ----------------------------------------------------------------------------------------- The method findComposerManifestPaths() would return wrong results if the list of found JSON files was in the "wrong" order. Adding a call to asort() makes the longer paths come later in the list and fixes the behavior. Also the direct detection of a manifest is fixed and a useless condition is removed. * Fixes: `#42807 `_ * Commit: `bc8e604 `_ [TASK] Handle InheritanceType annotation more robust ----------------------------------------------------------------------------------------- When specifying InheritanceType but no heirs exist, no longer an exception will be thrown. Instead inheritance will be disabled to allow specifying InheritanceType "for future use". * Resolves: `#27283 `_ * Commit: `f08339d `_ [FEATURE] Allow arrays to be set via Object.yaml property values ----------------------------------------------------------------------------------------- This introduces the option to also set arrays as plain native values in Objects.yaml. This allows settings like the following:: Acme\\Foobar\\Domain\\Repository\\BazRepository: properties: defaultOrderings: value: creationDate: 'ASC' * Resolves: `#42539 `_ * Commit: `fd2178c `_ [BUGFIX] Fix missing argument and superfluous trailing slash ----------------------------------------------------------------------------------------- A few errors sneaked into Ifb6612a808e8cd8ee892de727dd961287a119152. * Commit: `2338ac6 `_ [TASK] Align signatures of methods involved in creating package ----------------------------------------------------------------------------------------- The docblock of PackageFactory::create() was wrong, and the manifestPath is now optional. PackageManager is adjusted as needed. * Commit: `ad141e0 `_ [!!!][FEATURE] Check validator options and make them self-documenting ----------------------------------------------------------------------------------------- Validators now must declare any options they accept using the class property $supportedOptions. It is indexed by option name and holds an array for each option giving the default value, a description, type and an optional required flag. This is used to check accepted options and to generate option documentation for the validator reference documentation. * Resolves: `#37820 `_ * Commit: `d374b12 `_ [BUGFIX] Throw proper exception in ConfigurationBuilder ----------------------------------------------------------------------------------------- Add a check if the return value of get_class_methods() used in autowireProperties() actually is an array. Throw an exception if it is not, as this should not happen. * Fixes: `#41809 `_ * Commit: `d488398 `_ [TASK] Add checks for composer manifest in Package ----------------------------------------------------------------------------------------- The existence of a Composer manifest has been a soft requirement, make it hard. * Commit: `2ff9a04 `_ [TASK] Cleanup use statements with leading backslash ----------------------------------------------------------------------------------------- An almost purely cosmetic issue, but still: use always takes fully qualified class names, thus no leading backslash should be used. * Commit: `662fed7 `_ [BUGFIX] Consider only a packages top-level composer manifest ----------------------------------------------------------------------------------------- For different scenarios it is allowed for a package to contain more than one composer-manifest. For example in tests or if a package bundles several other packages. Only the root-manifest should be evaluated by the PackageManager. * Fixes: `#42516 `_ * Commit: `a148ca1 `_ [FEATURE] Native PHP-based session handler ----------------------------------------------------------------------------------------- This introduces an alternative session implementation which is based on the backend modules of the Flow Cache framework. All cache backend implementations available for caching (PDO, Redis, Apc, Memcache etc) can now be used for storing session data. * Related: `#42366 `_ * Resolves: `#42403 `_ * Commit: `5b56ade `_ [TASK] Small design corrections in documentation part II ----------------------------------------------------------------------------------------- Tweaks the error output in the New.html template. Some other tiny corrections. * Commit: `e311c25 `_ [TASK] Make Mvc\\Controller\\Exception extend Mvc\\Exception ----------------------------------------------------------------------------------------- Formerly the exception subclassed FLOW3\\Exception directly. * Resolves: `#42785 `_ * Commit: `2c27eb1 `_ [FEATURE] Add simple array insight in HTML debug backtrace ----------------------------------------------------------------------------------------- In the visual HTML debug backtrace, now array arguments get an additional title attribute, allowing to see the runtime values of the particular array when hovered with the mouse. Such a tooltip would look like:: array(2) integer 0 => Acme\\FooBar\\SomeObject singleton object proxy integer 1 => string "indexAction" (11) * Commit: `366174f `_ [FEATURE] Show package keys in schema migration listing ----------------------------------------------------------------------------------------- Additionally show the related package keys in the doctrine:migrationstatus command listing. For this, it brings a new PackageManager method ``getPackageOfObject()`` that allows retrieving a package by a package's member's class name. * Resolves: `#42011 `_ * Commit: `4b04b7d `_ [TASK] Fixed wrong filename in Documentation ----------------------------------------------------------------------------------------- The documentation used a wrong filename for a code listing (BlogRepository instead of PostRepository). * Commit: `417a571 `_ [FEATURE] Allow integer value as timestamp in DateTimeConverter ----------------------------------------------------------------------------------------- The DateTime TypeConverter cannot transform integer values to a DateTime object because of a strict check for string. Accept integer values and also assume plain digits to be a timestamp and use "U" as date format if no additional configuration is given. Additionally change the is_numeric() checks to ctype_digit() as is_numeric() would also allow float values. * Resolves: `#41989 `_ * Commit: `4c80098 `_ [FEATURE] Type converter for HTTP URIs ----------------------------------------------------------------------------------------- This provides a type converter, converting simple strings into their URI representations. This allows convenient type converting for use in action methods. * Resolves: `#42025 `_ * Commit: `b9fb686 `_ [FEATURE] Allow class constants in Configuration ----------------------------------------------------------------------------------------- Currently, only plain PHP constants are allowed for substitution in Configuration files. This changes the regular expression in order to match class constants too. So not only ``%PHP_VERSION%`` would be possible, but also, for example, ``%TYPO3\\FLOW3\\Core\\Bootstrap::MINIMUM_PHP_VERSION%``. * Resolves: `#41358 `_ * Commit: `50832ab `_ [BUGFIX] ObjectSerializer not capable of Collections ----------------------------------------------------------------------------------------- Serializing Objects containing properties being a Doctrine Collection fails with an exception since the ObjectSerializer is not capable of recognizing them. This change adds support for Collection properties in objects and arrays. * Resolves: `#42715 `_ * Commit: `a41d018 `_ [TASK] Small code cleanup in Http Request Handler ----------------------------------------------------------------------------------------- * Commit: `2cc6c24 `_ [FEATURE] Cache Frontend: getByTag() now returns identifiers ----------------------------------------------------------------------------------------- This changes the behavior of getByTag() to not only return a numeric array of entry values but now index them with their entry identifier. * Resolves: `#42761 `_ * Commit: `5cc198c `_ [BUGFIX] Fix unit test for PersistenceQueryRewritingAspect ----------------------------------------------------------------------------------------- The solution for #42601 broke one test in the PersistenceQueryRewritingAspectTest. * Fixes: `#42758 `_ * Commit: `3d94686 `_ [BUGFIX] Fix functional BrowserTest in Neos context ----------------------------------------------------------------------------------------- The functional BrowserTest in Flow (added as part of the solution to issue #42049) fails when run in a Neos context. This is caused by the different routes configuration. The change tweaks the route used in the test slightly. * Fixes: `#42756 `_ * Commit: `9736f94 `_ [TASK] Use Doctrine 2.3 ORM and adjust as needed ----------------------------------------------------------------------------------------- This adjusts our annotation driver to the changes in Doctrine 2.3 that were breaking. Also it adds the code needed to support new features: * NamedNativeQueries annotation * SqlResultSetMappings annotation * Table->options * DiscriminatorColumn->columnDefinition * AssociationOverrides annotation * AttributeOverrides annotation * CustomIdGenerator annotation Furthermore it requires doctrine/data-fixtures now, something we previously had packaged into our Doctrine.Common package. This change drops three changes we had applied to the stock packages before: * ORM - Throw exception if no identifier can be found in UnitOfWork * ORM - In proxies add docblocks and move __wakeup() call around * DBAL - Fix connection charset with MySQL for PHP < 5.3.6 (see https://review.typo3.org/4587). * Resolves: `#40863 `_ * Commit: `b807ceb `_ [TASK] Tweak exception message for not settable property ----------------------------------------------------------------------------------------- This explains the possible reason for a not settable property a bit more and describes how to solve the mentioned problem. * Commit: `667ff17 `_ [BUGFIX] Enforce Query Rewriting more reliably ----------------------------------------------------------------------------------------- This initializes the security context if it hasn't been initialized when the PersistenceQueryRewritingAspect becomes active. Previously it could happen that entities which should be covered by a policy are visible to anonymous users. * Resolves: `#42601 `_ * Commit: `9af3204 `_ [BUGFIX] Tweak help output, output version ----------------------------------------------------------------------------------------- The help output was a bit clumsy regarding the name of Flow and did not output the version (since it is missing in the manifest). * Commit: `b6e3faa `_ [TASK] Rename Phoenix to Neos ----------------------------------------------------------------------------------------- This change adjusts some technically irrelevant uses of Phoenix to Neos, just for completeness. * Related: `#41668 `_ * Commit: `c45b234 `_ [!!!][BUGFIX] Http\\Uri constructor throws exception on invalid Uri ----------------------------------------------------------------------------------------- The constructor of the Uri object now checks the return value of the parse_url method and throws an exception if this returns anything else than a valuable array, because in this case the given Uri is considered seriously malformed. This is considered breaking because prior to this change, the Uri just silently was created with all empty values. * Fixes: `#42511 `_ * Commit: `1b03b89 `_ [BUGFIX] Introduce HTTP Response::createFromRaw() ----------------------------------------------------------------------------------------- The Response::createFromRaw method halted if a Set-Cookie header arrives due to lacking handling. This brings an implementation to parse the Set-Cookie directive as described in RFC 6265 Section 5.2. The Response::createFromRaw method passes the Set-Cookie header data to this new implementation in order to create a Cookie object from it. * Commit: `c17a95d `_ [BUGFIX] Initialize resource wrapper before session ----------------------------------------------------------------------------------------- If you try to attach a resource-dependent object to a session, this will fail, as the _wakeup() method is called before the resource wrappers are initialized. This patch makes sure that the resource framework is initialized before the session framework in the standard runtime sequence of the bootstrap. * Fixes: `#42434 `_ * Commit: `84ef743 `_ [FEATURE] Identify key pair by fingerprint in RsaWalletService ----------------------------------------------------------------------------------------- This change uses SSH compatible fingerprints of the RSA public key instead of a random UUID to store keys. This allows to have a consistent reference of keys across systems. This change is backwards-compatible with the old behavior, because# stored keys can still be used with their generated UUID. * Resolves: `#42355 `_ * Commit: `8a706f2 `_ [BUGFIX] Support named indexes in multivalue translation labels ----------------------------------------------------------------------------------------- * Resolves: `#42169 `_ * Commit: `71318ee `_ [BUGFIX] Typo in setfilepermissions Script ----------------------------------------------------------------------------------------- This fixes the output of the setfilepermissions script to the correct spelling of TYPO3. * Fixes: `#42368 `_ * Commit: `c3371af `_ [BUGFIX] Fix sign method in RsaWalletService ----------------------------------------------------------------------------------------- Fixes the sign method to use the actual key string for openssl_sign and adds a unit test for that. Additionally a wrong return type is fixed in the interface. * Commit: `e63ab73 `_ [TASK] Tighten package key check a little ----------------------------------------------------------------------------------------- The pattern used to check for a valid package key was made less strict during composer integration. Although composer itself imposes almost no restrictions, packagist requires a "vendor/name" format. This changes thus adjusts the PATTERN_MATCH_PACKAGEKEY constant to require a dot in the package key. * Resolves: `#42319 `_ * Commit: `9762f62 `_ [BUGFIX] Make HTTP_HOST not always followed by port colon ----------------------------------------------------------------------------------------- If there is no port given, a trailing colon for the HTTP_HOST is not allowed to follow. * Fixes: `#42241 `_ * Commit: `53a6fde `_ [TASK] CGL-Documentation - Add a notice about byte order mark ----------------------------------------------------------------------------------------- * Fixes: `#42164 `_ * Commit: `96f4778 `_ [TASK] Remove oneline exception for throw statements ----------------------------------------------------------------------------------------- Removes the exception for throwing exceptions in oneliners, which is not allowed anymore. * Resolves: `#42168 `_ * Related: `#39977 `_ * Commit: `580b963 `_ [BUGFIX] Workaround issues with multiline commit messages ----------------------------------------------------------------------------------------- In core migration functionality, particularily on windows systems, the commit message is cropped at the first occurrence of a newline character. This leads into trouble when the core migration tool searches for the Migration: ... footers in the commit history as they won't be there then. * Related: `#41848 `_ * Commit: `77f148c `_ [FEATURE] Let HTTP browser follow Location redirects ----------------------------------------------------------------------------------------- Since Http's Browser object should behave like a real browser, Location headers are now followed (as soon as they come with a 3xx response code as of RFC2616 14.30). This patch includes a safeguard that prevents following infinite redirect loops and an option to disable the automatic redirection altogether. * Resolves: `#42049 `_ * Commit: `e90a00d `_ [BUGFIX] Apply PSR-0 code migration only to non-PSR-0 packages ----------------------------------------------------------------------------------------- This change adds a condition to prevent the code migration to move classes inside the Classes directory if it is already in PSR-0 layout. Additionally an already existing composer.json will not be overwritten and a safeguard was added to AbstractMigration that checks if a file that should be deleted actually exists. * Related: `#41848 `_ * Commit: `03d6d6f `_ [BUGFIX] Correct path check in HTTP\\Cookie constructor ----------------------------------------------------------------------------------------- A variable name has been corrected in the constructor of the Cookie class. * Related: `#39836 `_ * Commit: `9f40790 `_ [FEATURE] Include technical details to exceptions in dev context ----------------------------------------------------------------------------------------- Since the implementation of custom error renderers (#39423) 404 errors don't show any technical details to the user, even in development context. This change passes the exception to the fluid template which can render it, if configured. Furthermore this adds default rendering options and configures all exceptionHandlers to include technical details in development context. * Resolves: `#42135 `_ * Commit: `be181ef `_ [TASK] Documentation: Fix link to CGL on one page document ----------------------------------------------------------------------------------------- The link was renamed during the FLOW3 to TYPO3 Flow replacement but the site package is not yet renamed. * Commit: `cf4f0ee `_ [BUGFIX] CSRF token is not appended for actions with mixed case characters ----------------------------------------------------------------------------------------- The CrsfProtectionAspect no longer appends the CSRF token when the target action contains uppercase characters (e.g. someSpecialAction). This change fixes this by resolving the correctly cased method name before checking for the SkipCsrfToken annotation in the aspect. * Fixes: `#42083 `_ * Commit: `9ea304b `_ [BUGFIX] Respect explicit empty subpackage in uriFor with subrequests ----------------------------------------------------------------------------------------- This change just checks for the empty string in the routing arguments and sets a controller argument for building the URL. That way no subpackage argument will be overriden from the subrequest. * Fixes: `#42082 `_ * Commit: `8ebb3f6 `_ [BUGFIX] Fix path to database error template ----------------------------------------------------------------------------------------- The Settings.yaml contained an outdated path for the Fluid template which is supposed to display database errors. The message of the fatal error caused by this bug contained: 'Template file "resource://TYPO3.Flow/Private/Error/Default_Template.html" could not be loaded' * Related: `#39423 `_ * Commit: `ec4d422 `_ [TASK] Check for unexpected properties in Policy schema ----------------------------------------------------------------------------------------- This change disables unknown properties for resources and acl group keys to prevent mistakes. For example, the schema should fail if an entry is directly added to "resources" without using the "method" subkey. * Commit: `441957b `_ [BUGFIX] Fix composer manifest loading and unit tests ----------------------------------------------------------------------------------------- This change fixes 2 tests in PackageTest that failed when run from a package directory, because the $package->getComposerManifest() method did pick up the relative composer.json file inside that. Additionally some minor CGL cleanups in PackageTest. * Commit: `c85ccd7 `_ [BUGFIX] UriBuilder doesn't properly handle arguments of SubRequests ----------------------------------------------------------------------------------------- If you're creating a link inside a SubRequest that link currently doesn't add/remember the Arguments, Controller, etc from the parentRequest if it's nested more than one level: MainRequest->SubRequest->SubRequest This change fixes the mergeArgumentsWithRequestArguments to properly add all of parentRequests for links inside SubRequests. * Related: `#41571 `_ * Commit: `3b43e2a `_ [BUGFIX] Remove unnecessary call for code compilation ----------------------------------------------------------------------------------------- This change removes the forced compile command subrequest in a non Production context. Proxy classes are only initialized if the code cache is not up to date or with a special second check after file monitoring was run. This speeds up the development context quite enormous if no code was changed. * Fixes: `#41818 `_ * Commit: `c9f04ee `_ [BUGFIX] Tests fail when run from package path. ----------------------------------------------------------------------------------------- Composer manifests missing in some tests, and not read properly by package in others * Commit: `f442e2f `_ [TASK] Exclude classes with broken PSR-0 in DomCrawler ----------------------------------------------------------------------------------------- See https://github.com/symfony/DomCrawler/pull/7 * Related: `#41031 `_ * Commit: `ee8f8e1 `_ [BUGFIX] ActionRequest does not resolve correctly cased package key ----------------------------------------------------------------------------------------- ActionRequest resolves lowercased package key by looking for Package class instead of asking PackageManager. If no Package class exists in the given Package, the request fails. * Resolves: `#41657 `_ * Commit: `b744c35 `_ [TASK] Small code cleanup in Package ----------------------------------------------------------------------------------------- * Commit: `94a43b6 `_ [BUGFIX] Documentation mentions the NotEmptyValidator ----------------------------------------------------------------------------------------- The Part II (Getting Started), Validation chapter contains a note about the requirement and purpose of the NotEmptyValidator. * Fixes: `#41451 `_ * Commit: `d00e876 `_ [BUGFIX] Changed f:form.textbox to f:form.textfield in Documentation ----------------------------------------------------------------------------------------- * Commit: `0a78fb8 `_ [TASK] Make exception templates more configurable ----------------------------------------------------------------------------------------- At the moment only the file name can be set, but not the format, partial & layout paths are configurable. * Commit: `81c41cd `_ [BUGFIX] Adjust unit test for YamlSource to changed indentation ----------------------------------------------------------------------------------------- * Commit: `50c3e1c `_ [TASK] Use two spaces as yaml indentation instead of four ----------------------------------------------------------------------------------------- This was introduced when upgrading the Symfony Yaml package to the latest source with the move to composer. * Commit: `41c6352 `_ [TASK] make Package::getComposerManifest() public ----------------------------------------------------------------------------------------- This is needed for fixing the site import of setup * Commit: `4a53002 `_ [BUGFIX] fix unit tests ----------------------------------------------------------------------------------------- The test of Idc1d9d48998a0de0ecd07f834baa769ddf1e5ba4 was not adjusted to the rename. * Commit: `0146868 `_ [BUGFIX] Update composer type in migration ----------------------------------------------------------------------------------------- * Commit: `8c73d8e `_ [BUGFIX] Adjust Package Management to updated composer package types ----------------------------------------------------------------------------------------- This fixes issues f.e. with functional test runs of Phoenix. * Commit: `3f4b734 `_ [BUGFIX] Fix monitoring of changed class files ----------------------------------------------------------------------------------------- This change fixes the resolving of the class name from a changed file since the PSR-0 changes caused some problems with the assumption of the class name layout in that place. The change keeps the old behavior for test files since these are not yet updated to a new layout. * Resolves: `#41499 `_ * Commit: `3d0a981 `_ [BUGFIX] Remove unnecessary call to to sortAndSavePackageStates ----------------------------------------------------------------------------------------- This change should fix the saving of a new PackageStates.php file on every request which causes a rebuild of proxy classes and flushing of other caches through file monitoring. * Resolves: `#41497 `_ * Commit: `14a7882 `_ [BUGFIX] Fix Proxy Blacklisting ----------------------------------------------------------------------------------------- * Commit: `f4b26da `_ [BUGFIX] Try to fix composer loading problem ----------------------------------------------------------------------------------------- * Commit: `33acf04 `_ [!!!][TASK] Rename FLOW3 to TYPO3 Flow ----------------------------------------------------------------------------------------- Adjusts the code, comments, documentation as needed. A code migration to adjust packages and schema migrations are added as well. * Related: `#41466 `_ * Commit: `57526b7 `_ [!!!][TASK] Rename FLOW3 to TYPO3 Flow, move files ----------------------------------------------------------------------------------------- Moves files so names match the FLOW3 to Flow change. * Related: `#41466 `_ * Commit: `69d40a9 `_ [TASK] Some leftover case changes ----------------------------------------------------------------------------------------- This adjusts the case of some forgotten namespaces and classes. Also it marks a number of migrations as (correctly) applied. * Commit: `569fcb8 `_ [TASK] Adjust PackageManagerTest to "skip unnamed" change ----------------------------------------------------------------------------------------- With I5e29317bbada2b6aed13fe26b50c1ae1ed9c2c80 a check was added that lead to a failing unit test. So much for trivial stuff. :) * Commit: `adc2fba `_ [BUGFIX] Make sure generated composermanifest is valid ----------------------------------------------------------------------------------------- * Fixes: `#41449 `_ * Commit: `17af2c0 `_ [TASK] Skip "packages" without name in manifest ----------------------------------------------------------------------------------------- When using packages that have example "composer.json" files (like Doctrine ORM 2.3 in source form) this will avoid errors. * Commit: `d40e50a `_ [BUGFIX] Fix package / manifest detection ----------------------------------------------------------------------------------------- In production context class loading fails because the classloader has a wrong map for namespaces. This is caused by getNamespace() doing the wrong thing - because it finds no composer manifest. This change fixes this. * Fixes: `#41418 `_ * Commit: `d010f92 `_ [TASK] Copy dotfiles during composer install ----------------------------------------------------------------------------------------- Dotfiles like .htaccess need to be copied as well. * Related: `#41421 `_ * Commit: `8feeaa3 `_ [FEATURE] Allow copy of dotfiles in copyDirectoryRecursively() ----------------------------------------------------------------------------------------- Adds a new parameter to copyDirectoryRecursively(), namely the boolean $copyDotFiles. To enable this, also readDirectoryRecursively() now knows a new parameter named $returnDotFiles. * Resolves: `#41421 `_ * Commit: `9b63414 `_ [TASK] Remove leftover .Shortcuts handling ----------------------------------------------------------------------------------------- * Commit: `196e37d `_ [TASK] Fix flow3 commandline script ----------------------------------------------------------------------------------------- Shortens a docblock and removes define() left over from recent work. * Commit: `1a417b6 `_ [TASK] Move .gitignore to composer install default ----------------------------------------------------------------------------------------- * Related: `#41031 `_ * Commit: `ee4d1f8 `_ [TASK] Tweak install script behavior, add (example) configuration ----------------------------------------------------------------------------------------- With this change the composer post install/update script will discern between "Essentials", being files that will always be copied - even if they already exist in their target location - and "Defaults", being files that will be copied only if not already existing. This is used to add the example configuration files as essentials and a Routes.yaml as default. * Related: `#41031 `_ * Commit: `4179179 `_ [FEATURE] Allow keeping files in copyDirectoryRecursively() ----------------------------------------------------------------------------------------- This change adds a flag $keepExistingFiles to copyDirectoryRecursively() in Files. It defaults to FALSE and if set to TRUE allows to keep files already present in the target location. * Resolves: `#41378 `_ * Commit: `c7cce8d `_ [BUGFIX] Add .htaccess file to installed files ----------------------------------------------------------------------------------------- * Fixes: `#41369 `_ * Commit: `987b91a `_ [TASK] Update settings to new name of installer package ----------------------------------------------------------------------------------------- * Related: `#41031 `_ * Commit: `d303919 `_ [TASK] Update class exclusion to changed composer installer name ----------------------------------------------------------------------------------------- The installer change it's package key from christianjul.installers to flow3.composerinstallers so the exclusion needs to be adjusted. * Related: `#41031 `_ * Commit: `0ea33a3 `_ [TASK] Change dependency to require flow3/composer-installers ----------------------------------------------------------------------------------------- * Commit: `125ff01 `_ [TASK] Use composer version of installer ----------------------------------------------------------------------------------------- * Commit: `947608e `_ [TASK] Require christianjul/installers directly ----------------------------------------------------------------------------------------- * Related: `#41031 `_ * Commit: `9a68c0e `_ [TASK] Test commit for composer install ----------------------------------------------------------------------------------------- * Commit: `37492ff `_ [TASK] Change requirement to real composer installer ----------------------------------------------------------------------------------------- Also drop require-dev - only valid for root manifests * Commit: `e3f9f13 `_ [BUGFIX] Handle missing composer type gracefully ----------------------------------------------------------------------------------------- The type in a composer manifest is optional, so we need to check for it's existence before we use it. * Fixes: `#41334 `_ * Commit: `cb870e1 `_ [TASK] Install "Web/index.php" on Composer install ----------------------------------------------------------------------------------------- * Fixes: `#41310 `_ * Commit: `0f8b4d2 `_ [!!!][TASK] Add utility to get classes namespace entry-point ----------------------------------------------------------------------------------------- With PSR-0 the entry-point of the classes files needs prefixing of the pathified namespace. Add a getter, getClassesNamespaceEntryPath(), to resolve this. Also makes sure the namespace is only resolved once. Renames getPackageNamespace() to getNamespace() - adjust your code as needed. * Commit: `1594a52 `_ [BUGFIX] Allow excludeClasses setting for non-installed package ----------------------------------------------------------------------------------------- * Fixes: `#41157 `_ * Commit: `a952d97 `_ [BUGFIX] getClassesPath() returns path without trailing slash ----------------------------------------------------------------------------------------- * Fixes: `#41181 `_ * Commit: `1925ba3 `_ [TASK] Add code migration to "composerify" a package ----------------------------------------------------------------------------------------- The migration moves all class files into a PSR-0 compatible directory structure and adds a composer manifest; Package.xml is removed. * Related: `#41031 `_ * Commit: `6130251 `_ [FEATURE] Isolate migrations and provide package data ----------------------------------------------------------------------------------------- With this change migrations have access to the data of the target package (the package currently being migrated) and can be sure that each migration run is isolated from other runs (the internal state is reset for each execution). The command output is tweaked a little along the way. * Related: `#41031 `_ * Commit: `2e2d98f `_ [FEATURE] Add moveFile() and deleteFile() operations to migrations ----------------------------------------------------------------------------------------- Code migrations now can use deleteFile() and moveFile() to work on files. * Related: `#41031 `_ * Commit: `5950a76 `_ [FEATURE] Make metadata available to migrations ----------------------------------------------------------------------------------------- This makes metadata from Package.xml and composer.json available to migrations. * Related: `#41031 `_ * Commit: `bdc5a0e `_ [TASK] Fix PSR-0 compatibility for functional tests ----------------------------------------------------------------------------------------- * Related: `#41031 `_ * Commit: `9b9e92b `_ [TASK] Update unit tests to vfsStream > 1.0 ----------------------------------------------------------------------------------------- This adds "mikey179/vfsStream" to composer.json and adjusts the unit tests to the new vfsStream API. * Resolves: `#37813 `_ * Commit: `ef34261 `_ [BUGFIX] Fix failing tests in package management ----------------------------------------------------------------------------------------- * Fixes: `#41092 `_ * Fixes: `#41093 `_ * Commit: `91e1881 `_ [BUGFIX] PackageStates.php does not contains relative paths ----------------------------------------------------------------------------------------- * Fixes: `#41088 `_ * Commit: `9476151 `_ [TASK] Make package management use Composer ----------------------------------------------------------------------------------------- Refactor package management to use Composer, also relax constraints in order to utilize 3rd party composer / PSR-0 components. * Fixes: `#41031 `_ * Commit: `6fdd855 `_ [TASK] Exclude specific classes from object management ----------------------------------------------------------------------------------------- In order to exclude classes from object management - either for performance or compatibility reasons - create a configuration that allows specific classes to be excluded. Most be configurable outside of package in question to support 3rd party components * Related: `#41031 `_ * Commit: `6e88cac `_ [TASK] Handle exceptions thrown in ClassReflection ----------------------------------------------------------------------------------------- When a class cannot be loaded for reflection, it will be ignored and a message is logged. This makes sure that classes that are never used do not break the system. If they are indeed used later, things will still break, but even at a more helpful position. * Related: `#41031 `_ * Commit: `b811e14 `_ [TASK] Add Composer installer script and resources ----------------------------------------------------------------------------------------- Upon running 'install' or 'update' with composer, required paths and files outside of TYPO3.FLOW3 package must be created. * Related: `#41031 `_ * Commit: `f9b09a9 `_ [TASK] Add composer manifest ----------------------------------------------------------------------------------------- * Related: `#41031 `_ * Commit: `f227c30 `_ [TASK] Update referenced paths to reflect PSR-0 paths ----------------------------------------------------------------------------------------- See Idad621923ffccdb7b2742d9aba0ff22fdab3e192 for the files being moved. * Related: `#41031 `_ * Commit: `5c1bde4 `_ [!!!][TASK] Move all classes to PSR-0 compatible path ----------------------------------------------------------------------------------------- In order to be fully PSR-0 compatible the path below the classes directory should now be in a path identical to the namespace * Related: `#41031 `_ * Commit: `6a5d4d7 `_ [FEATURE] Provide flag for disabling link protection in UriBuilder ----------------------------------------------------------------------------------------- This introduces a new flag for the UriBuilder which allows to configure if a link built by the UriBuilder may be modified by some security mechanism or not. FLOW3's CSRF protection mechanism now considers this flag. By disabling link protection, it is now possible on a per-link basis, to generate a link without CSRF protection tokens for cases when it's clear that links are public. This patch also removes the CSRF protection of links used in ExtDirect services. This needs to be re-implemented in the ExtJS package. * Resolves: `#41137 `_ * Commit: `2058a64 `_ [FEATURE] Convenience function for registering routes in functional tests ----------------------------------------------------------------------------------------- This adds a new convenience function registerRoute() to the functional base testcase. * Resolves: `#41136 `_ * Commit: `6562a48 `_ [BUGFIX] Debugger now respects possible occurence of Closures ----------------------------------------------------------------------------------------- A trial isset() call for a property of a Closure resulted in a PHP error saying "Closure objects cannot have properties". To avoid this, the isset() statement has been replaced with a property_exists(). This is considered slower than isset(), like claimed in a User Contributed Note [1], but obviously the only workaround for these cases [2]. And speed is not an issue when the debugger is used… [1] http://www.php.net/manual/en/function.property-exists.php#97538 [2] https://bugs.php.net/bug.php?id=50146 * Fixes: `#41046 `_ * Commit: `15d3860 `_ [TASK] Adjust CsrfProtectionAspect unit test ----------------------------------------------------------------------------------------- * Related: `#40984 `_ * Commit: `e1f227c `_ [BUGFIX] Make sure autoloader is always unregistered ----------------------------------------------------------------------------------------- ClassReflection adds an autoloader to catch cases of unresolved classes. This autoloader needs to be unregistered reliably after use. * Fixes: `#40989 `_ * Commit: `dfc34e5 `_ [BUGFIX] Cleanup CSRF protection issues ----------------------------------------------------------------------------------------- * Fixes: `#40984 `_ * Commit: `4574114 `_ [TASK] Clarify parent request type in docblocks ----------------------------------------------------------------------------------------- Instead of just "object" this uses a more specific type in a few docblocks ("ActionRequest|HttpRequest"). * Commit: `a081efa `_ [TASK] Whitespace fix in class docblock ----------------------------------------------------------------------------------------- * Commit: `d9b71a6 `_ [BUGFIX] Display error message if "Data/" is not writeable ----------------------------------------------------------------------------------------- If the directory "Data/" or "Data/Persistent/" does not exist and could not be created by the bootstrap, an error message is now displayed which hints on using the setfilepermissions command. * Resolves: `#40890 `_ * Commit: `939181e `_ [BUGFIX] Parse target type before comparing the instance with it ----------------------------------------------------------------------------------------- The PropertyMapper chokes on already converted Objects if the targetType is a compositeType like Collection. * Resolves: `#40661 `_ * Commit: `4168539 `_ [TASK] Small code cleanup in ReflectionService ----------------------------------------------------------------------------------------- * Commit: `5ea5915 `_ [BUG] Fix wrong escape sequence in yaml ----------------------------------------------------------------------------------------- Backslash is not a valid escape-character in yaml for single- quoted strings. see: http://yaml.org/spec/current.html#id2534365 * Resolves: `#40840 `_ * Commit: `604406e `_ [TASK] Custom error page: higher priority for class matches ----------------------------------------------------------------------------------------- All custom error rendering options should be evaluated for matches of the exception class name before trying to match against status code because the latter are often used as fallback options (for example, 404 for anything not found). The TYPO3.TYPO3 package wouldn't be able to display a specific "Missing Homepage" error message without this patch because FLOW3's 404 handling would kick in before. * Related: `#39423 `_ * Commit: `9b28224 `_ [TASK] Handle errors in subprocesses properly ----------------------------------------------------------------------------------------- To be able to properly handle missing class dependencies in ClassReflection a ClassLoader should be added to end of the ClassLoader chain that throws an exception if it's reached, meaning the class couldn't be loaded. Make executeCommand throw an exception on errors, rather than just echo any command output collected. * Resolves: `#39599 `_ * Commit: `e590a18 `_ [!!!][FEATURE] Custom Error Renderers ----------------------------------------------------------------------------------------- This extends the exception handlers so that they can render arbitrary templates based on the status code and/or class name of the thrown exception. The template to be used and variables that should be passed to the template are configured in so called "renderingGroups" in the options of the exception handler. This is a breaking change because it deprecates the provided NotFoundController & NotFoundView. This patch also contains the related documentation, including general information about FLOW3's error handling. Also included in this patch is an update to the YAML schema definitions. Furthermore, removed all hints on "FLOW3" from the production exception handler. * Resolves: `#39423 `_ * Commit: `20e41f3 `_ [FEATURE] Add identifier to Booting\\Sequence ----------------------------------------------------------------------------------------- The identifier is set in Bootstrap to "runtime" and "compiletime" respectively. If you wire a slot to the beforeInvokeStep and afterInvokeStep signals, the identifier will be handed over as second parameter after the step instance. * Resolves: `#38424 `_ * Commit: `e694792 `_ [TASK] Add logging to importResource() and createResource() ----------------------------------------------------------------------------------------- When importing or creating a resource fails, the reason is logged to the system log. * Commit: `228cb54 `_ [BUGFIX] Fix failing UnitTests for new AuthenticationController ----------------------------------------------------------------------------------------- * Fixes: `#40760 `_ * Commit: `6ba2ae1 `_ [BUGFIX] Always add CSRF token, if authenticated ----------------------------------------------------------------------------------------- This ensures that the CSRF token is also present if the main request is not restricted, but if the subrequests is. * Commit: `443d6f1 `_ [FEATURE] Provide a general abstract authentication controller ----------------------------------------------------------------------------------------- This can be used to be inherited from in a custom package, to enable developers to quickly implement their own authentication controller. * Related: `#39839 `_ * Commit: `3eb7b5c `_ [TASK] Fix return type documentation of createQuery ----------------------------------------------------------------------------------------- * Commit: `f7f4671 `_ [BUGFIX] Fix SQL logging in Production context ----------------------------------------------------------------------------------------- The logger factory in Objects.yaml refers to a Setting directive which with this change is present in Production context. Before, the Settings could not be found, resulting in a NULL argument where an array is expected as soon as query logging was enabled in Production. Some cleanup to logging settings along the way. * Fixes: `#40580 `_ * Commit: `007e41b `_ [FEATURE] Improved argument type watch in HTML debug backtrace ----------------------------------------------------------------------------------------- In the debug backtrace of method calls, this gives concrete information about the actual argument values in case of being boolean; and reveals the amount of array elements, if such. An array is now presented as ``array|2|`` for example, if it has two elements. Boolean values are not longer only displayed as ``boolean``, but as their concrete TRUE or FALSE value accordingly. Since the color scheme stays the same, it can still easily be distinguished from string values. * Resolves: `#39879 `_ * Commit: `d953765 `_ [BUGFIX] Session Timeout shouldn't be changed to 10 seconds ----------------------------------------------------------------------------------------- This reverts the changed session timeout setting introduced in If7d004c7. * Fixes: `#40657 `_ * Commit: `3ef0662 `_ [BUGFIX] InternalRequestEngine observes exception status code ----------------------------------------------------------------------------------------- The InternalRequestEngine set the status code in case of an exception to 500 in any case. With this change it will instead use the code defined in an exception, if available. * Fixes: `#40658 `_ * Commit: `4b479b2 `_ [BUGFIX] Prevent data from destroyed session from being loaded ----------------------------------------------------------------------------------------- This fixes a problem with session data which is being unserialized even though its session is about to be destroyed. Because the time of the last activity was stored in the session and needs to be retrieved in order to determine if the session is still valid, also all other session data is being unserialized by PHP. This results in session scope objects registering themselves at the Object Manager due to their code in __wakeup(). Now the session time out is not stored in the session anymore but in its own cookie. Therefore the session data is only unserialized if the session is still valid. * Resolves: `#37001 `_ * Commit: `1b409cc `_ [BUGFIX] Throw exception on CSRF token error ----------------------------------------------------------------------------------------- Currently, if a required CSRF token is missing or invalid, FLOW3 dies with a hard coded "Access denied!". This change disables the try/catch blocks in the RequestDispatchingAspect so that the access denied exception is actually rendered. This shouldn't pose a security issue as details are hidden in production context. * Related: `#27798 `_ * Commit: `1b3a9e2 `_ [BUGFIX] Fix request injection in security context ----------------------------------------------------------------------------------------- As inject* methods are excluded from the session lazy loading proxy mechanism, the injectRequest method of the security context didn't work correctly in requests with a session timeout. Renaming the method to setRequest solves the problem. * Related: `#37001 `_ * Commit: `8d1c137 `_ [FEATURE] Enable functional tests to be executed by non CLI SAPI ----------------------------------------------------------------------------------------- The FunctionalTestRequestHandler should be executable by non CLI requests to enable TestRunners like VisualPHPUnit. * Fixes: `#40273 `_ * Commit: `8d01cdf `_ [FEATURE] cURL request engine for HTTP browser ----------------------------------------------------------------------------------------- This provides a new request engine, based on cURL, which can be used with the virtual browser contained in FLOW3's HTTP foundation. It can be used to send requests and receive responses via HTTP. This patch also contains a few minor modifications in the RequestEngineInterface and InternalRequestEngine class. * Resolves: `#40414 `_ * Commit: `20a8c74 `_ [!!!][BUGFIX] Misused JoinColumns in ManyToMany ----------------------------------------------------------------------------------------- With this fix the JoinColumns are created and used in the same way regardless if you have a JoinTable annotation. This is (almost) a cosmetic fix as it works without, but can lead to confusion if looking at the database. Breaking change as the JoinColumns could change if you annotated with JoinTable. * Fixes: `#33397 `_ * Commit: `f8b0bb7 `_ [!!!][BUGFIX] Don't override orphanRemoval setting from Doctrine ----------------------------------------------------------------------------------------- In Doctrine 2 ORM, orphanRemoval is disabled by default. In FLOW3 we want orphanRemoval to happen, if the target is not an aggregate root, changing the default Doctrine behavior. Because we want developers to be able to override that override, we had a check in place, to favor the setting coming from the annotation. But since there is no way to check if the FALSE in the orphanRemoval of an annotation is the default or has been explitly set, this would fail to allow an override with FALSE. For many-to-many we thus assume that orphanRemoval is very rarely a useful thing, and thus skip our own default handling. If you want orphanRemoval on many-to-many associations, you now must explicitly enable it with the annotation option! * Fixes: `#40389 `_ * Commit: `ca60d48 `_ [BUGFIX] Fix Headers::has() for Cache-Control ----------------------------------------------------------------------------------------- The Http\\Headers::get() method now correctly returns TRUE if a cache control header exists. * Resolves: `#40372 `_ * Commit: `28fc7e7 `_ [TASK] Add Windows version hint to documentation ----------------------------------------------------------------------------------------- * Related: `#38038 `_ * Commit: `fe8c078 `_ [TASK] Detect inject methods more strictly ----------------------------------------------------------------------------------------- Currently every method starting with "inject" is considered for property injection by the ConfigurationManager. That includes methods named "inject" or "injectionFoo". This change makes the detection more strict to only accept methods that start with "inject" followed by an upper case character. * Related: `#39681 `_ * Commit: `45f872a `_ [FEATURE] Make WebRedirect EntryPoint less Routing dependent ----------------------------------------------------------------------------------------- The WebRedirect currently expects an option uri that points to a relative or absolute URI to redirect when trying to access a protected action. This change makes it possible to specify *route values* instead so that the Routing framework can determine the correct target URI depending on the Routes.yaml configuration. * Resolves: `#39837 `_ * Commit: `94c0c7b `_ [BUGFIX] Loop confuses variable check on migrating ----------------------------------------------------------------------------------------- When migrating code and not specifying a package to work on, the code would get confused after the first available migration, because the internal loop used $packageKey as well. * Related: `#40263 `_ * Commit: `d8d6fee `_ [TASK] Add changelog for FLOW3 1.1.0 ----------------------------------------------------------------------------------------- * Commit: `a47ccbf `_ [FEATURE] Allow mapping of indexed property names with placeholder ----------------------------------------------------------------------------------------- For multi-valued property types (Doctrine\\Common\\Collections\\Collection or array) the property mapper uses indexes as property names. To match the property mapping configuration for any index, the path syntax supports an asterisk as a placeholder:: $propertyMappingConfiguration ->forProperty('items.*') ->setTypeConverterOption( 'TYPO3\\FLOW3\\Property\\TypeConverter\\PersistentObjectConverter', \\TYPO3\\FLOW3\\Property\\TypeConverter\\PersistentObjectConverter::CONFIGURATION_CREATION_ALLOWED, TRUE ); This also works recursively, as in 'items.*.author'. * Resolves: `#37436 `_ * Commit: `faaee92 `_ Add changelog for FLOW3 1.0.6 ----------------------------------------------------------------------------------------- * Commit: `4fcbba8 `_ [BUGFIX] Fix reflection on PHP 5.3.16 ----------------------------------------------------------------------------------------- The fix for https://bugs.php.net/62715 landed in PHP 5.3.16, and because of https://bugs.php.net/62874 reflection fails (in Production context) with a pretty unhelpful exception: Fatal error: Uncaught exception … with message 'Parameter is not optional' in … This change adds an additional check with isOptional() to the relevant code part. * Fixes: `#40003 `_ * Commit: `2dbfd8f `_ [BUGFIX] Alphanumeric-Validator fails on umlauts ----------------------------------------------------------------------------------------- This change adds support for umlauts in the AlphanumericValidator, by changing the regex pattern to [:alnum:] and adding utf-8 mode flag. * Fixes: `#35295 `_ * Commit: `f1fa750 `_ [FEATURE] Control Validation Groups via annotations ----------------------------------------------------------------------------------------- This feature adds support for overriding validation groups in actions through @FLOW\\ValidationGroups annotation. * Resolves: `#36695 `_ * Commit: `46ea9e1 `_ [CLEANUP] Some CGL fixes ----------------------------------------------------------------------------------------- * Commit: `d554ec2 `_ [TASK] Increase branch version number to 1.2 ----------------------------------------------------------------------------------------- This updates the constant in Bootstrap. * Commit: `f7c29db `_ [TASK] Get rid of strtolower() in foreach loop ----------------------------------------------------------------------------------------- A tiny optimization I could not resist. * Commit: `41267d1 `_ [TASK] Fix markup errors on documentation ----------------------------------------------------------------------------------------- * Related: `#27561 `_ * Commit: `522057e `_ [FEATURE] Cookie management support ----------------------------------------------------------------------------------------- This implements the cookie management feature which previously only existed as a stub in the HTTP foundation. This patch also contains the corresponding tests and documentation. * Resolves: `#39836 `_ * Commit: `ea08462 `_ [WIP][TASK] New chapter: MVC ----------------------------------------------------------------------------------------- This adds a new chapter to The Definitive Guide. Also adjust some mentions of "DocComment" in other chapters and adds a stub for a section about HMAC to the security chapter. * Resolves: `#3881 `_ * Commit: `d961924 `_ [FEATURE] Validate email addresses with filter_var() ----------------------------------------------------------------------------------------- This change makes FLOW3 use filter_var instead of a regex check to validate email addresses. * Resolves: `#39714 `_ * Extbase Issue: `#6970 `_ * Commit: `bf26810 `_ [BUGFIX] PropertyMapper should be able to work with recursive already-converted objects ----------------------------------------------------------------------------------------- If f.e. an array of already converted objects is passed into the PropertyMapper, a fatal error will be thrown. Thus, we move the check from convert() to doMapping() such that it works recursively. * Resolves: `#39445 `_ * Commit: `8ac0366 `_ [FEATURE] Added some Signals for the Debug.Toolbar ----------------------------------------------------------------------------------------- - added an Signal to AbstractAdvice on Advice invocation - added an Signal to ActionRequest when it's dispatched - added getSignals to the SignalSlot\\Dispatcher to get all configured Signals with it's slots * Resolves: `#37879 `_ * Commit: `713ca78 `_ [FEATURE] Configurable CLI Subrequest environment variables ----------------------------------------------------------------------------------------- When debugging with xdebug, a major difficulty is that in development and functional tests, there are always two requests being executed: The main request and a CLI request inside. Now, most IDEs do not support the (optional) feature of multi-threaded PHP debugging, effectively stopping the sub-request and also the main request in this case. With this change it's possible to change xdebug configuration settings for the CLI subrequests making it possible to debug them individually. By default the remote port is now changed from 9000 to 9001 in Development and Testing context. So you can debug the subrequest by configuring your IDE to listen on port 9001. Apart from that, this change allows you to set arbitrary environment variables by adding entries to the TYPO3.FLOW3.core.subRequestEnvironmentVariables setting. * Resolves: `#39648 `_ * Commit: `cfa7069 `_ [BUGFIX] Fix warning about property injection in test ----------------------------------------------------------------------------------------- The DependencyInjectionTest testcase has a test called injectionInParentClassIsDoneOnlyOnceOnCreationOfObject, leading to FLOW3 trying to use the inject method for the property "ionInParentClassIsDoneOnlyOnceOnCreationOfObject"… This change renames the test method to fix this. * Fixes: `#39681 `_ * Commit: `a7fdef8 `_ [TASK] Support literal strings in boolean arguments ----------------------------------------------------------------------------------------- This change adds support for string literals in boolean comparisons. String literals have to be surrounded by single or double quotes and may contain escaped single or double quotes. * Resolves: `#6757 `_ * Commit: `3b47daf `_ [TASK] Add section about Uri class in HTTP chapter ----------------------------------------------------------------------------------------- This adds some explanation for the use of the Uri class. * Related: `#39424 `_ * Commit: `68d19d7 `_ [TASK] New chapter: HTTP ----------------------------------------------------------------------------------------- This adds a new chapter to The Definitive Guide * Resolves: `#39424 `_ * Commit: `a0db61b `_ [BUGFIX] Fix some outdated code (examples) in Part II ----------------------------------------------------------------------------------------- * Fixes: `#39374 `_ * Commit: `aafcbb6 `_ [BUGFIX] Disable proxy building for Http classes ----------------------------------------------------------------------------------------- This disables proxy building for (most of) the classes in the Http part of FLOW3. They are not needing it and some would circumvent the built proxy because they are instantiated statically. * Fixes: `#39005 `_ * Commit: `868e073 `_ [BUGFIX] Fix arguments rendering for dumped backtraces ----------------------------------------------------------------------------------------- When an exception was dumped to Data/Logs/Exceptions/…txt all arguments were doubled, once with parenthesis around them, once without. This change fixes this. * Fixes: `#38625 `_ * Commit: `09d4a72 `_ [BUGFIX] FLOW3's var_dump() still chokes on Doctrine collections ----------------------------------------------------------------------------------------- Using Doctrine\\Common\\Util\\Debug::export() to render doctrine collections leads to an exception because our Debugger expects a string and doctrine's debugger returns an array. This change fixes this by post processing the result. * Fixes: `#39286 `_ * Related: `#37481 `_ * Commit: `fd3eaff `_ [BUGFIX] Corrected annotation of assign method in ViewInterface ----------------------------------------------------------------------------------------- A view is basically intended to be able to get arbitrary values, hence data types assigned, so the annotation gets corrected to accomodate for this. * Fixes: `#39293 `_ * Commit: `9ede9cc `_ [TASK] New chapter: Command Line ----------------------------------------------------------------------------------------- This is a new chapter for The Definitive Guide. Also marked the executeCommand() method in the Scripts class as an @api function. * Resolves: `#38944 `_ * Commit: `8f9b449 `_ [BUGFIX] RsaWalletService: Save keys only if modified ----------------------------------------------------------------------------------------- This patch adds a check to the shutdown() method of the RsaWalletServicePhp to make sure that the keys are only saved to disk if they were actually modified. * Resolves: `#39267 `_ * Commit: `752e4a6 `_ [BUGFIX] Missing setting namespace in error message ----------------------------------------------------------------------------------------- When an invalid PHP binary is found, the error message hint at setting the path in settings misses the setting namespace TYPO3 in the beginning. * Fixes: `#39142 `_ * Commit: `32c8a20 `_ [TASK] Fix partial validation example ----------------------------------------------------------------------------------------- The partial validation example was flawed. This change updates the example to an actual working usecase. Credits to Ruud Alberts for finding. * Commit: `c95d732 `_ [TASK] Add changelog for FLOW3 1.1.0-rc2 ----------------------------------------------------------------------------------------- * Commit: `2f1a4ef `_ [TASK] Update references in documentation ----------------------------------------------------------------------------------------- This updates command, type converter and Fluid VH references. * Commit: `76c814b `_ [FEATURE] Catch PDOExceptions to prevent database information in debug backtrace ----------------------------------------------------------------------------------------- Add catching of \\PDOExceptions to the Doctrine Query class, and output the exception message & code, but prevent the debug backtrace leading up to it, since it contains the database information in the method calls. * Resolves: `#37360 `_ * Commit: `4df796a `_ [BUGFIX] Fix broken links StandardView template ----------------------------------------------------------------------------------------- The links pointed to URLs no longer working. * Fixes: `#39062 `_ * Commit: `2b2fcaa `_ [BUGFIX] Fix functional tests after FloatConverter change ----------------------------------------------------------------------------------------- Somehow two failing functional tests went unnoticed so far, broken with Ib95c52d958d765eb0a9cd20e415642186e25ee57. * Related: `#36333 `_ * Commit: `b83b9b4 `_ [!!!][BUGFIX] Correct RepositoryInterface ----------------------------------------------------------------------------------------- Fixes a docblock and adds __call() to the interface (thus the breaking change flag), because everyone probably would expect it to be there. * Fixes: `#38840 `_ * Commit: `f305aee `_ [!!!][BUGFIX] Fix return values of Doctrine\\Repository magic methods ----------------------------------------------------------------------------------------- Overrides __call from EntityRepository to make the methods return the expected QueryResultInterface. Also adds support for countBy(). This is marked as a breaking change because if you relied on the specific return values before, you need to adjust your code. Along the way the code inside __call() is simplified. * Fixes: `#38841 `_ * Commit: `934c2c4 `_ [FEATURE] FloatConverter uses Locale parser ----------------------------------------------------------------------------------------- If configured using the Property Mapping Configuration, the FloatConverter now can also use the Locale parser to get the intended `float` value from a string input. * Resolves: `#36333 `_ * Commit: `d10d076 `_ [TASK] Correct typos in Coding Guidelines ----------------------------------------------------------------------------------------- This corrects some typos in PartV/CodingGuidelines of the documentation. * Related: `#27561 `_ * Commit: `d743972 `_ [!!!][TASK] Minor typo corrections in code ----------------------------------------------------------------------------------------- This trivially corrects some typos in code. As a consequence, the DataNotSerializeableException was renamed to DataNotSerializableException. * Commit: `c86356c `_ [TASK] Document configuration validation ----------------------------------------------------------------------------------------- * Resolves: `#38680 `_ * Commit: `9f4dba0 `_ [BUGFIX] Injections only on last level of class hierarchy ----------------------------------------------------------------------------------------- Property injections are no longer called for every level of your class hierarchy, which means they are only executed once now. * Fixes: `#31179 `_ * Commit: `f9330b4 `_ [BUGFIX] Fix order of monitoring and proxy building ----------------------------------------------------------------------------------------- During bootstrap file monitoring would be run too late, so that while change detection worked fine, the results were not used correctly: Reflection data was updated, but that happened after DI proxy building, so that e.g. added Inject annotations would not have the expected effect. This change moves file monitoring before reflection initialization to fix this. * Fixes: `#37574 `_ * Commit: `d355326 `_ [TASK] Add TypeConverter reference ----------------------------------------------------------------------------------------- Tweaks the DateTimeConverter docblock to be proper reST. * Resolves: `#38646 `_ * Commit: `16e1493 `_ [TASK] Add HTTP status code to exceptions ----------------------------------------------------------------------------------------- Currently if an exception is rendered with either Debug- or ProductionExceptionHandler the HTTP status is set to "500 Internal Server Error". This change adds a property "statusCode" to the FLOW3 base Exception allowing to set a custom HTTP status code. This also sets status codes for some of the FLOW3 exceptions * Related: `#27798 `_ * Commit: `2a3bfd1 `_ [TASK] Create Reflection manual page ----------------------------------------------------------------------------------------- * Resolves: `#38561 `_ * Commit: `205d209 `_ [TASK] Tweak TypeConverter extends statements ----------------------------------------------------------------------------------------- Use short class names for extends in TypeConverters. * Related: `#38646 `_ * Commit: `69d2705 `_ [!!!][TASK] Remove FLOW3's magic timezone setting behavior ----------------------------------------------------------------------------------------- FLOW3 used to magically set a default timezone if there's none set (in order to avoid the PHP notice being thrown, resulting in an exception). Because every default time zone will be the wrong one for most people; and to raise awareness of having to care about the system's time zone, this behavior is dropped. This change is marked breaking because in cases where no timezone is set in PHP configuration, FLOW3 will no longer work. * Resolves: `#37438 `_ * Commit: `3f3b1e0 `_ [BUGFIX] Allow PublishingConfiguration to be persisted ----------------------------------------------------------------------------------------- Any resource publishing configuration attached to a resource could not be persisted, because the needed mapping information for Doctrine was missing and the target type of the relation was an interface. This change adds an AbstractPublishingConfiguration and the needed annotations, plus a schema migration. * Fixes: `#32826 `_ * Commit: `9dd8a61 `_ [BUGFIX] Fix indentation in "Contributing to FLOW3" ----------------------------------------------------------------------------------------- * Fixes: `#38884 `_ * Commit: `91cb311 `_ [FEATURE] Add flushConfigurationCache to ConfigurationManager ----------------------------------------------------------------------------------------- This allows to remove the cached configuration in Production context. * Resolves: `#38859 `_ * Commit: `42702ac `_ [BUGFIX] Adjust execute bit on some files ----------------------------------------------------------------------------------------- This fixes the execute bit (and setfilepermissions.sh) so that no differences are created when setting permissions on a fresh clone. * Fixes: `#36049 `_ * Commit: `f91996c `_ [TASK] Some code cleanup, CSRF settings documentation ----------------------------------------------------------------------------------------- Does some code (style) cleanup and adds descriptions for CSRF options in Settings.yaml. * Related: `#34875 `_ * Commit: `f406f7c `_ [BUGFIX] Have Doctrine\\Repository::findAll() return a QueryResult ----------------------------------------------------------------------------------------- This makes the behavior of Doctrine\\Repository consistent and allows the use of findAll() e.g. in combination with widgets. With this change, findAll() also observes the default ordering of a repository. * Fixes: `#35035 `_ * Fixes: `#38381 `_ * Commit: `9825cdb `_ [BUGFIX] Fix timezone override side effect in DateTimeConverter ----------------------------------------------------------------------------------------- Specifying a timezone override in DateTimeConverter would lead to wrong times due to the timezone being applied to the finished date instance, not upon creation. * Fixes: `#35369 `_ * Commit: `49196a7 `_ [FEATURE] Support tearDown for functional tests in generic persistence ----------------------------------------------------------------------------------------- Implement a tearDown method in the generic persistence according to the Doctrine PersistenceManager. The call is delegated to the underlying backend. * Resolves: `#38785 `_ * Commit: `af46cd9 `_ [TASK] Clean up some code ----------------------------------------------------------------------------------------- Fixes some comments, style issues and API usage. * Related: `#37574 `_ * Related: `#34875 `_ * Commit: `d57781a `_ [BUGFIX] MemcachedBackendTest fail ----------------------------------------------------------------------------------------- MemcachedBackendTest doesn't use TYPO3\\FLOW3\\Core\\ApplicationContext to initialize MemcacheBackend. * Fixes: `#38713 `_ * Commit: `b9f907d `_ [BUGFIX] Remove f:uri.resource title parameter in Guide ----------------------------------------------------------------------------------------- The resource management documentation described a title parameter for the ResourceViewHelper that ceased to exist a long time ago. * Fixes: `#38382 `_ * Commit: `d799913 `_ [BUGFIX] Reset InternalRequestEngine for each functional test ----------------------------------------------------------------------------------------- Currently all functional tests share the same instance of \\TYPO3\\FLOW3\\Mvc\\Routing\\Router because the InternalRequestEngine is of scope singleton. This leads to unexpected results and failing tests if tests add custom routes. This change fixes this by making the ObjectManager forget the singleton instance of the request engine after each testrun. This also reverts the changes that were introduced with I5a5bf17d00c5be05748a18122dae42989f429221 as they did not have the expected effect. * Fixes: `#38435 `_ * Commit: `cff94d7 `_ [TASK] Add annotation reference to documentation ----------------------------------------------------------------------------------------- * Resolves: `#38669 `_ * Commit: `086ee1e `_ [TASK] Add documentation to annotation classes ----------------------------------------------------------------------------------------- * Related: `#38669 `_ * Related: `#34875 `_ * Commit: `e4fbaad `_ [BUGFIX] Fix use of Target in annotation classes ----------------------------------------------------------------------------------------- * Fixes: `#38667 `_ * Commit: `fbe64c7 `_ [FEATURE] Extract authenticateAccount functionality ----------------------------------------------------------------------------------------- This extracts the portion that is responsible for the actual account authentication in order to use this exclusively in own tests. * Resolves: `#37470 `_ * Commit: `6bbe20c `_ [BUGFIX] Skip abstract classes in ensureAggregateRootInheritanceChainConsistency ----------------------------------------------------------------------------------------- When using an abstract base class for domain models, the checks done in ensureAggregateRootInheritanceChainConsistency() would fail, even though it is fine to have an abstract base class not being aggregate root. * Fixes: `#31522 `_ * Commit: `2d62b1e `_ [TASK] Use Inject annotation instead of tag ----------------------------------------------------------------------------------------- The ConfigurationBuilder still used getPropertyNamesByTag(), this change replaces that with getPropertyNamesByAnnotation(). * Resolves: `#38622 `_ * Commit: `761e287 `_ [TASK] Functional tests for Routing Framework ----------------------------------------------------------------------------------------- Adds testing routes to the provided Testing/Routes.yaml file and adds tests covering mathing and resolving of arbitrary routes. * Resolves: `#37608 `_ * Commit: `6c9633f `_ [BUGFIX] Reset the router for each functional test ----------------------------------------------------------------------------------------- Currently all functional tests share the same instance of \\TYPO3\\FLOW3\\Mvc\\Routing\\Router because the InternalRequestEngine is of scope singleton. This leads to unexpected results and failing tests if tests add custom routes. This change fixes this by reinitializing the request engine for each test case that has testableHttpEnabled set to TRUE. * Fixes: `#38435 `_ * Commit: `3bb6604 `_ [BUGFIX] BaseUri contains default HTTPS port ----------------------------------------------------------------------------------------- By resolving #37770 now the BaseUri and thus the base tag when using contains the HTTP port if it's not equal to 80. This adjusts Request and Uri to omit the port if it is equal to 443 and the request scheme is https. * Fixes: `#38547 `_ * Related: `#37770 `_ * Commit: `5835407 `_ [TASK] Add "Introduction" to the guide ----------------------------------------------------------------------------------------- Added introduction section for the guide. * Related: `#38037 `_ * Related: `#27561 `_ * Commit: `439702c `_ [TASK] Add changelog for FLOW3 1.1.0-rc1 ----------------------------------------------------------------------------------------- * Commit: `291cb17 `_ [BUGFIX] Fix wrong check in atomic writes code ----------------------------------------------------------------------------------------- Fixes a wrong check in Cache\\FileBackend and RsaWalletServicePhp as well as a bunch of one-line if blocks. * Fixes: `#27989 `_ * Related: `#34875 `_ * Commit: `1355fcd `_ [FEATURE] Inform about lock expiry in CLI ----------------------------------------------------------------------------------------- When a CLI command is not executed because the FLOW3 system is locked, the time remaining until lock expiry is shown in the message to the user. * Resolves: `#38419 `_ * Commit: `849e9e1 `_ [TASK] Add "Essential Design Patterns" to the guide ----------------------------------------------------------------------------------------- Re-uses some AOP verbage from part III of the guide. * Related: `#38037 `_ * Related: `#27561 `_ * Commit: `90cad6b `_ [TASK] Document Routing feature "appendExceedingArguments" ----------------------------------------------------------------------------------------- * Resolves: `#35394 `_ * Commit: `8dde9f2 `_ [BUGFIX] IdentityRoutePart should accept identity array ----------------------------------------------------------------------------------------- IdentityRoutePart::matchValue() returns an identity array in the format array('__identity' => '') (see #37403). This change ajusts resolveValue() to accept arrays in that format, too. This is required for widgets that fetch arguments from the current request and pass them to the UriBuilder. * Resolves: `#38404 `_ * Commit: `23a2b37 `_ [TASK] Adjust filename spelling to match rest of package ----------------------------------------------------------------------------------------- * Related: `#35298 `_ * Commit: `91141a3 `_ [TASK] Remove trailing colon from example headlines ----------------------------------------------------------------------------------------- It just looked weird this way… * Related: `#38037 `_ * Commit: `63421e0 `_ [BUGFIX] Fix Objects.yaml className example in guide ----------------------------------------------------------------------------------------- The guide still stated that setting className in Objects.yaml was possible for classes, but it works only for interfaces. * Fixes: `#37648 `_ * Commit: `dc396d0 `_ [BUGFIX] Fix code documentation of DatetimeParser ----------------------------------------------------------------------------------------- The parser code documentation would state the parse methods expect \\DateTime instances, but in fact strings are expected. * Fixes: `#34050 `_ * Commit: `7024268 `_ [TASK] Provide conventions for ids in XLIFF files ----------------------------------------------------------------------------------------- * Resolves: `#32606 `_ * Commit: `f4f841b `_ [TASK] Update and link CGL on one page ----------------------------------------------------------------------------------------- * Resolves: `#36651 `_ * Commit: `20f273f `_ [TASK] Show up hints for nesting inline ViewHelpers ----------------------------------------------------------------------------------------- This adds a section with some examples about how to nest and use inline ViewHelpers. Besides, it does some minor typo and formatting corrections. * Resolves: `#34749 `_ * Commit: `28ab103 `_ [BUGFIX] Log route values if a route can't be resolved ----------------------------------------------------------------------------------------- This re-introduces logging if a route could not be resolved. This behaviour was lost with I9605731f78cb1be9b0fb5c439d7149a676b6ccb7 * Related: `#38172 `_ * Commit: `c110361 `_ [TASK] Update "CGL on one page" ----------------------------------------------------------------------------------------- * Resolves: `#36651 `_ * Commit: `3633f65 `_ [TASK] Code cleanup ----------------------------------------------------------------------------------------- Does code cleanup, mostly to documentation, but also tweaks unused variables, missing return statements and similar things. * Related: `#34875 `_ * Commit: `3128201 `_ [TASK] Add "Architectural Overview" to the guide ----------------------------------------------------------------------------------------- Based on the introduction from the old DocBook manual, tweaked and amended. * Resolves: `#29266 `_ * Commit: `fd62a02 `_ [TASK] Some tiny tweaks to the 1.1.0b3 changelog ----------------------------------------------------------------------------------------- * Commit: `4ae9c66 `_ [TASK] Add changelog for FLOW3 1.1.0-beta3 ----------------------------------------------------------------------------------------- * Commit: `38e6e22 `_ [BUGFIX] Only map file uploads to arguments if not empty ----------------------------------------------------------------------------------------- Currently the HTTP request converts the FILES superglobal to request arguments even if no file has been uploaded. This leads to an exception when submitting a form without uploading a file because the property mapper receives a corrupt source array in that case. This adjust the HTTP request to only map values from the FILES superglobal to arguments if a file has been uploaded. This also tweaks the ResourceTypeConverter to accept an empty source. Besides the converter now only outputs filer errors (for too large or corrupt files) to the user. Server errors (e.g. non-existing/read-only tmp directory) are logged to the systemLogger. * Fixes: `#38184 `_ * Commit: `38fcda5 `_ [BUGFIX] Inform about possible cause for "No class schema found" ----------------------------------------------------------------------------------------- When mapping meta data for Doctrine an error about "No class schema found for …" could occur. It would not tell in what model the mapping was taking place, leaving the user to guesswork. This adds output of the currently examined class to the exception. * Fixes: `#37547 `_ * Commit: `a88b400 `_ [TASK] Add context information to cache clearing commands ----------------------------------------------------------------------------------------- When using the flow3:cache:flush commands only the current cli context is cleared. This is not clear when using the commands and this change add the context information to the command output. * Resolves: `#38221 `_ * Commit: `44e04bf `_ [TASK] Fix section about Resources in FLOW3 manual ----------------------------------------------------------------------------------------- This fixes the Fluid code snippet of the resource upload example in the Resource Management section. Besides this adds a sample domain model to make this easier to understand. * Related: `#38037 `_ * Commit: `e444f6a `_ [TASK] Documentation corrections, each minor ----------------------------------------------------------------------------------------- Some additional corrections, basically typo corrections, for the manual. * Related: `#38037 `_ * Commit: `13b891d `_ [BUGFIX] Routing cache only works for one request ----------------------------------------------------------------------------------------- Currently the routing uses a TransientMemory cache backend by default. Besides, objects in the routing values are converted to hashes using spl_object_hash() - which generates a new hash for every request. This change configures the RouterCachingAspect to use a SimpleFile cache backend and adjusts the aspect to use the persistence manager in order to convert objects to their identifiers before creating cache entries. This also adds a signal to the FunctionalTestCase base class that flushes the routing caches on teardown. Furthermore this adds two convenience methods getLastMatchedRoute() and getLastResolvedRoute() to the router which should make debugging and testing easier. * Fixes: `#38172 `_ * Commit: `c7386f4 `_ [TASK] Update Fluid ViewHelper reference ----------------------------------------------------------------------------------------- This adds format.identifier documentation and a depreciation notice for identity.json to the Fluid VH reference. * Related: `#38037 `_ * Commit: `a96ea73 `_ [BUGFIX] Non-standard ports are now considered by HttpRequest ----------------------------------------------------------------------------------------- Ports that are not 80 for HTTP or 443 for HTTPS were simply not passed through to the Request when being created by the create() method. Also Uri will now set the port to 443 for HTTPS if no port is given in the string handed to the constructor. * Fixes: `#37770 `_ * Commit: `d7106f9 `_ [TASK] Comment fixes for accuracy and grammar ----------------------------------------------------------------------------------------- * Commit: `c2e90ad `_ [BUGFIX] Make PhpSession read settings from "PhpSession" ----------------------------------------------------------------------------------------- The settings for the PhpSession are read from the key "PhpSession" now, to ensure consistency. Before that they were read from "PHPSession". * Fixes: `#37993 `_ * Commit: `96de170 `_ [BUGFIX] Uri respects query and fragment even without path ----------------------------------------------------------------------------------------- Corresponding to the appropriate RFC 3986, section 3.3 the query and fragment indeed may be present also if no path is given. * Fixes: `#37903 `_ * Commit: `75d9852 `_ [TASK] Make extending GenericObjectValidator easier ----------------------------------------------------------------------------------------- This change moves the check for already validated objects out of the isValid() method and into validate() instead. Since in custom validators isValid() is the method to implement, this makes correct implementation less error-prone. * Related: `#37363 `_ * Commit: `ed3788b `_ [BUGFIX] Property Mapper chokes on already-converted objects ----------------------------------------------------------------------------------------- This happens for example when using the TYPO3.Form Project with a DatePicker, which already transforms the value to the correct data type. * Fixes: `#36988 `_ * Commit: `6695a4c `_ [BUGFIX] Correctly implement validator resolver short notation ----------------------------------------------------------------------------------------- Currently, the short notation for validators which has been introduced with Id807dbfa78ae1388aa8cd53bfc578893eade427c does only support syntax like TYPO3\\FLOW3:EmailAddress; but we should rather support TYPO3.FLOW3:EmailAddress as this is how we use package keys at all other places which are not PHP code. * Fixes: `#38154 `_ * Related: `#33862 `_ * Commit: `fdfca9e `_ [TASK] Add note to View chapter about default model properties ----------------------------------------------------------------------------------------- This note should be made seen the confusion around fixing #35894 * Related: `#35894 `_ * Commit: `a785bb6 `_ [BUGFIX] Make baseUri setting work for resource publishing ----------------------------------------------------------------------------------------- The baseUri setting was not observed in the filesystem resource publisher. * Fixes: `#37550 `_ * Commit: `92b4d99 `_ [BUGFIX] Use exception handling framework for CommandController ----------------------------------------------------------------------------------------- The CommandRequestHandler brought its own tiny exception handling with the main drawback exceptions not being logged anymore, unlike the Error\\AbstractExceptionHandler features. When handling a CommandRequest, a dedicated CommandExceptionHandler is involved, taking care of occuring exceptions. * Fixes: `#37995 `_ * Commit: `f0eaa57 `_ [FEATURE] Configuration validation through (adapted) JSON Schema ----------------------------------------------------------------------------------------- Implementation of a PHP array schema validator modeled using the JSON Schema specification as guideline. The following CLI commands which utilize the validator are provided: - configuration:validate validates the configuration of a certain type with respect to the provided schemata. The schema files are searched for in Resources/Private/Schema of all active packages. - configuration:generateschema is a kickstarter which generates a basic schema to be used as a starting point for further adjustments Furthermore, this change includes schemata for all FLOW3 settings and the other configuration types (Caches, Objects, Policy and Routes). * Resolves: `#31277 `_ * Commit: `1c8a80d `_ [BUGFIX] Internal Request Engine needs to clear security context ----------------------------------------------------------------------------------------- In order to get the security tokens matching the new request created in the internal request engine, the security context should be cleared using the "clearContext" method. This means that the tokens will be matched with the configuration from the routing based on the new request and not the parent request. This patch also contains a small enhancement which adds two additional headers to the virtual browser's response containing the code and message of a possible exception. * Fixes: `#37377 `_ * Commit: `f67078c `_ [TASK] Fix markup errors in FLOW3 manual ----------------------------------------------------------------------------------------- This change fixes another markup issue resulting in rendering glitches. * Resolves: `#38037 `_ * Commit: `70e012a `_ [TASK] Fix markup errors in FLOW3 manual ----------------------------------------------------------------------------------------- This change fixes a number of markup issues resulting in rendering glitches. * Resolves: `#38037 `_ * Commit: `2ce7ec8 `_ [TASK] Update templating chapter ----------------------------------------------------------------------------------------- Along with some style and markup tweaks this adds details on XML namespace declaration and XSD generation. * Resolves: `#38037 `_ * Commit: `ad55bfb `_ [TASK] Update validation chapter ----------------------------------------------------------------------------------------- Along with some style and markup tweaks this moves some sections around and adds details on avoiding recursive/duplicate validation. * Resolves: `#38037 `_ * Commit: `ffa7f56 `_ [TASK] Update object management chapter ----------------------------------------------------------------------------------------- This updates the section on custom factories. * Resolves: `#38037 `_ * Commit: `70cf2d8 `_ [TASK] Update persistence chapter ----------------------------------------------------------------------------------------- Some layout/markup tweaks and removed some outdated parts about the generic PDO persistence we had earlier. Also explain identity and value object handling in more detail. * Resolves: `#38037 `_ * Commit: `9ba6511 `_ [TASK] Update FLOW3 manual ----------------------------------------------------------------------------------------- This change removes the glossary and colophon from the guide and applies mostly markup changes to various parts. * Resolves: `#38037 `_ * Commit: `f951e29 `_ [BUGFIX] Discourage people to expose anything but the Web folder ----------------------------------------------------------------------------------------- This updates the Quickstart to explicitly push people into using Web as the document root for their web server. * Fixes: `#38016 `_ * Commit: `6545e47 `_ [TASK] Refactor constructor argument of ExceptionHandler implementors ----------------------------------------------------------------------------------------- Moves the redundant constructor method of each AbstractExceptionHandler implementor to the AbstractExceptionHandler itself. * Resolves: `#38002 `_ * Commit: `62df179 `_ [TASK] Session handling documentation ----------------------------------------------------------------------------------------- * Resolves: `#37668 `_ * Commit: `8d7f0f3 `_ [BUGFIX] Fix recursion detection in instantiateClass ----------------------------------------------------------------------------------------- When in ObjectManager->instantiateClass() the speedup case was used, recursion detection would potentially fail, as it returns early without removing the class name from the list of classes currently being instantiated. * Fixes: `#37997 `_ * Commit: `88988c5 `_ [TASK] Code cleanup without functional changes ----------------------------------------------------------------------------------------- This change fixes some comments, shuffles the order of things a little and adds some empty lines. * Commit: `6584400 `_ [BUGFIX] Remove unused configuration option from settings ----------------------------------------------------------------------------------------- This change removes a no longer used/supported option from the shipped Development configuration. * Fixes: `#37966 `_ * Commit: `9d72dba `_ [TASK] Update validator reference with correct option types ----------------------------------------------------------------------------------------- * Commit: `8ad0388 `_ [TASK] Add validator reference to the FLOW3 guide ----------------------------------------------------------------------------------------- * Commit: `ad190d4 `_ [TASK] Add CLI command reference to guide ----------------------------------------------------------------------------------------- Adds a reference for the CLI commands in the FLOW3 base distribution. * Commit: `51ff07d `_ [TASK] Add contributors list ----------------------------------------------------------------------------------------- * Commit: `de0a196 `_ [TASK] Update Fluid ViewHelper reference ----------------------------------------------------------------------------------------- This change updates the Fluid VH reference and does some tiny tweaks along the way. * Commit: `594b122 `_ [TASK] Tiny docblock fix in DoctrineCommandController ----------------------------------------------------------------------------------------- * Commit: `a915d02 `_ [BUGFIX] Corrected "Uri host" regular expression ----------------------------------------------------------------------------------------- The regular expression verifying the validity of a host name in Uri's setHost method was of an incorrect syntax leading to a PHP error message. Besides correcting that, this patch adds a Test to proof the intended functionality. * Fixes: `#37554 `_ * Commit: `9b04755 `_ [TASK] Add changelog for FLOW3 1.0.5 ----------------------------------------------------------------------------------------- * Commit: `083e453 `_ [TASK] Update code migration to do two more fixes ----------------------------------------------------------------------------------------- This adds adjustments for getOriginalRequestMappingResults() and getOriginalRequest()->getArguments(). * Resolves: `#37762 `_ * Commit: `65a3e7b `_ [BUGFIX] Packages can be frozen in Development only ----------------------------------------------------------------------------------------- Freezing packages was not possible in other contexts than Development with the related commands. But if a package was frozen according to PackageStates.php, it would advertise itself as frozen in all contexts. While this did not seem to have any real effects, it caused confusing output with the cache:flush command in Production context. This change makes the Package class ignore the frozen state in any context except Development and adds a check in the freezePackage() method of the PackageManager. * Fixes: `#37223 `_ * Commit: `4bff558 `_ [BUGFIX] Purge Doctrine proxies that are no longer needed ----------------------------------------------------------------------------------------- With this change the proxy target directory is emptied before Doctrine proxies are (re)built, removing outdated proxy classes. This solves fatal errors when (proxied) classes are removed, and thus can no longer be found when referenced from their heirs. * Fixes: `#26903 `_ * Commit: `2f2e3fc `_ [TASK] No longer freeze packages by default ----------------------------------------------------------------------------------------- Due to the possible pitfalls with frozen packages, no packages will be frozen by default. This way a developer has to learn about frozen packages before he can be confronted with them (and their pitfalls). * Resolves: `#37726 `_ * Commit: `cadcb61 `_ [BUGFIX] Integrity constraint violation deleting resources ----------------------------------------------------------------------------------------- If you have two resources using the same resource pointer, deleting a Resource will trigger deletion of the connected ResourcePointer which then leads to an integrity constraint violation. This patch tells doctrine NOT to cascade delete operations. This will lead to orphan resource-pointers which have to be deleted manually or on a regular base. * Fixes: `#34448 `_ * Commit: `02a7d84 `_ [TASK] Update Security Framework chapter ----------------------------------------------------------------------------------------- Quick update of the Security Framework chapter to reflect recent changes in FLOW3. Also some minor spelling error fixes. * Resolves: `#37672 `_ * Commit: `df033b0 `_ [TASK] document utility functions ----------------------------------------------------------------------------------------- * Resolves: `#37669 `_ * Commit: `2ef191c `_ [TASK] Documentation of Validation ----------------------------------------------------------------------------------------- * Resolves: `#37666 `_ * Commit: `8d40ad1 `_ [TASK] Make validator resolver public API ----------------------------------------------------------------------------------------- People should use it when validation is used in custom code instead of creating validators using new; thus the ValidatorResolver must be marked as public API. * Resolves: `#37665 `_ * Commit: `8182c52 `_ [TASK] improve Property Mapping Documentation ----------------------------------------------------------------------------------------- * Resolves: `#37664 `_ * Commit: `9be5bb0 `_ [TASK] Update Package.xml with new version number ----------------------------------------------------------------------------------------- * Commit: `cab86de `_ [TASK] Some tiny tweaks to the 1.1.0b2 changelog ----------------------------------------------------------------------------------------- * Commit: `57bbe65 `_ [TASK] Add changelog for FLOW3 1.1.0-beta2 ----------------------------------------------------------------------------------------- * Commit: `82590ff `_ [BUGFIX] Fix inconsistent Validate and IgnoreValidation behavior ----------------------------------------------------------------------------------------- The Validate annotation does an ltrim of "$" on the argument which the IgnoreValidation annotation did not do. This change makes the two behave the same by adding that trim to IgnoreValidation as well. * Fixes: `#37385 `_ * Commit: `24485a7 `_ [TASK] Make "all" parameter optional in package:refreeze ----------------------------------------------------------------------------------------- For freeze, unfreeze and refreeze the default value for the packageKey is now "all". * Resolves: `#37358 `_ * Commit: `bc84ef1 `_ [BUGFIX] PersistenceManager->add() now requires objects being new ----------------------------------------------------------------------------------------- The add method in repositories did also update existing entities, this can be dangerous, as it allows an attacker to misuse creation forms (i.e. a register form) and change existing entities. * Fixes: `#34527 `_ * Commit: `8d4b3c7 `_ [TASK] Log skipped persistAll() due to closed EntityManager ----------------------------------------------------------------------------------------- To avoid exceptions we only flush the EM if it is still open. To be able to spot errors more easily, this adds a log entry if the EM was closed and flushing was skipped. * Related: `#30934 `_ * Commit: `660ee7e `_ [TASK] Clean up some coding style issues ----------------------------------------------------------------------------------------- Add space to "foreach(". * Related: `#34875 `_ * Commit: `1ff13c4 `_ [TASK] Clean up some coding style issues ----------------------------------------------------------------------------------------- Add spaces to "){" * Related: `#34875 `_ * Commit: `bb514fd `_ [TASK] Create configuration directories only on save ----------------------------------------------------------------------------------------- The configuration directories for cached configuration were created inside the constructor of the ConfigurationManager. This caused the unit test to create directories outside the VFS. Moving the creation to the saveConfigurationCache method fixes that. Additionally cleans up some unit tests related to ApplicationContext and ConfigurationManager. * Related: `#33581 `_ * Commit: `cb141c7 `_ [BUGFIX] Keep existing PackageStates.php ----------------------------------------------------------------------------------------- A wrong check for the existence of the configured "packagePath" caused the loss of custom package states configuration on every request in development mode or initially in production. * Resolves: `#37607 `_ * Commit: `166b44e `_ [BUGFIX] Router overrides request arguments ----------------------------------------------------------------------------------------- With the reworked Request stack, the router kicks in after the action request have been built, overriding all arguments the respective route specifies. This works for the most cases, but leads to issues when you try to specify part of a complex target argument via the route and part via request. For example if you update a domain model via request body but specify the model itself through the URI like http://domain/products/product-1 - now the Identity-RoutePart detects "product-1" and turns it into the corresponding __identity-array but then the Router kicks in and overrides the "product" argument completely resulting in a new object to be created by the property mapper. This change adjusts the Router to recursively merge arguments with the route values instead, with precedence to the route values. * Fixes: `#37445 `_ * Commit: `170733f `_ [BUGFIX] Avoid validating instances more often than needed ----------------------------------------------------------------------------------------- The GenericObjectValidator uses a static variable to store objects being validated. Objects were still validated over and over because at the end of isValid() they were detached again. This change replaces that detach() call with another concept. Now any validator implementing the new ObjectValidatorInterface can be given an SplObjectStorage to keep track of instances already validated. If you reset that container when it makes sense (before you actively validate objects) you can avoid validating single instances more than once in one validation run but still validate at various points on your logic flow. If you have implemented your own object validator you should implement the new ObjectValidatorInterface - the easiest way is to extend the GenericObjectValidator, if you haven't done that so far. * Fixes: `#37363 `_ * Related: `#34154 `_ * Commit: `26e4b3f `_ [BUGFIX] Fix IP address validation ----------------------------------------------------------------------------------------- The IP address validation done with filter_var() worked on the development machine running PHP 5.3.12, however, it failed on the integration server running PHP 5.3.5. Apparently, the filter_var options for FILTER_VALIDATE_IP behave differently in these versions. This patch removes the validation for IPv4 / IPv6 addresses because they didn't make sense anyway. In that regard: hooray for PHP behavior changes. They actually exploit bad application code. Sometimes. * Related: `#37584 `_ * Commit: `744c759 `_ [FEATURE] Provide way to determine client IP address ----------------------------------------------------------------------------------------- This adds a method getClientIpAddress() to the HTTP Request class. It considers various kinds of environment variables which are present when using proxies or certain server setups for determining the actual client IP address. * Resolves: `#37584 `_ * Commit: `66bcdf5 `_ [!!!][FEATURE] Content Negotiation for Media Types ----------------------------------------------------------------------------------------- This improves the support for detecting the Media Type (that is, "format" in an ActionRequest context) based on the request's Accept header. This implementation supports Quality Values and Media Ranges as specified in RFC 2616. The way to define the supported formats of a controller has changed: Previously a class property $supportedFormats contained a list of filename extensions which would be supported. This property is not supported anymore. Instead, controllers may now specify a list of IANA Internet Media Types in a property $supportedMediaTypes. The default setting for these supported media types in ActionController is "text/html". You'll need to adjust your controllers accordingly if you relied on the $supportedFormats feature. Also note that the format in ActionRequest is now NULL by default. The ActionController makes sure to set the correct format, but if you created your own ActionRequest for a special purpose, you need to take care of setting it yourself. * Resolves: `#37561 `_ * Commit: `03b6d85 `_ [BUGFIX] Save relative package path in PackageStates.php ----------------------------------------------------------------------------------------- Using absolute paths in the PackageStates.php causes problems in a chroot environment where the absolute path is different in CLI then in the HTTPD process. Also relative paths allow to put PackageStates.php under version control to maintain a specific state even when paths differ on development and production systems. * Fixes: `#37220 `_ * Related: `#33719 `_ * Commit: `531178a `_ [Bugfix] Property mapper should distinguish between error and NULL ----------------------------------------------------------------------------------------- currently, when a nested type converter returns NULL, this is silently caught by the property mapper. Thus, it is not possible to reset f.e. an integer value to NULL again. This change fixes that behavior, and adds a functional test for it. * Related: `#35136 `_ * Commit: `5badcdb `_ [!!!][TASK] Switch from MIME to Media Types ----------------------------------------------------------------------------------------- The term MIME type is outdated, at least if used in a web context. The correct term is "Internet Media Type". Furthermore, our list of MIME types (or media types) was not up to date. This patch introduces a new utility class "MediaTypes" which replaces "FileTypes". Along with the new class comes a script which allows the core team to conveniently update the list of Media Types and filename extensions. This is a breaking change as the FileTypes class is deprecated with it. The old methods are still available but should not be used anymore. A code migration to use the new ones instead is shipped with the change. * Related: `#37402 `_ * Related: `#33371 `_ * Commit: `ef3e8fb `_ [!!!][BUGFIX] Float and Integer converters do not correctly handle errors and empty values ------------------------------------------------------------------------------------------ When an argument of a controller action is annotated with @param integer $var, the validation never fails if a string is passed to the argument. This is because the IntegerConverter implicitly casts the string to an integer. The same problem exists with floats. This change fixes this by checking the value and returning an error object if it is not numeric. Besides this tweaks the FloatConverter to accept float and integer values as input and it adds a bunch of unit & functional tests. Furthermore, empty strings are now correctly converted as NULL values. This is a breaking change if you relied upon the old behavior that empty values are converted to the number 0. * Fixes: `#35136 `_ * Commit: `a915647 `_ [BUGFIX] Make our var_dump() usable with Doctrine collections ----------------------------------------------------------------------------------------- For Doctrine\\Collection instances we use Doctrine's export() method now to render the dump. * Fixes: `#37481 `_ * Commit: `c5497de `_ [TASK] Remove debug output in log on configuration changes ----------------------------------------------------------------------------------------- * Related: `#34875 `_ * Commit: `cec2133 `_ [FEATURE] Add getHttpResponse() to HttpRequestHandler ----------------------------------------------------------------------------------------- This adds a getter method to the HTTP request handler which allows for retrieving the response corresponding to the currently handled request. * Resolves: `#36696 `_ * Commit: `7e1087a `_ [FEATURE] Make HTTP responses standards compliant ----------------------------------------------------------------------------------------- This adds a new API method to the HTTP response which is called by a request handler to assure conformity with RFC 2616 and other related specifications. While there are a lot of rules to consider, this patch only introduces a first set of important checks which are all explained by a corresponding test. * Related: `#33371 `_ * Commit: `63283d9 `_ [BUGFIX] Set property mapping configuration in RestController ----------------------------------------------------------------------------------------- Since Iac7bbb2a58ad890701fff2b0ad6b16a0e0b15bba we use a whitelist-based approach to configure property mapping. This change hooks into the create/update action of the RestController and sets the required property mapping configuration. * Related: `#37402 `_ * Related: `#36776 `_ * Commit: `94d958a `_ [FEATURE] Allow configurable identity key in JsonView ----------------------------------------------------------------------------------------- Providing _exposeObjectIdentifier option is not restricted to result in __identity anymore, but is configurable to be an arbitrary string. * Resolves: `#34414 `_ * Commit: `29ffc63 `_ [BUGFIX] Add hint on validator empty handling to migration ----------------------------------------------------------------------------------------- * Fixes: `#37231 `_ * Commit: `d511b04 `_ [BUGFIX] Fix Wrong exception message in Repository->remove() ----------------------------------------------------------------------------------------- The message would say add() was involved, which is wrong. * Fixes: `#37448 `_ * Commit: `57e1e95 `_ [FEATURE] Response setExpires() / getExpires() ----------------------------------------------------------------------------------------- This adds two new convenience methods to the Response API which allow for setting and retrieving the Expires HTTP header. * Related: `#33371 `_ * Commit: `60c22ce `_ [FEATURE] Make Message / Response setters chainable ----------------------------------------------------------------------------------------- This makes the setter methods of the Message and the Response class chainable. * Related: `#33371 `_ * Commit: `149a012 `_ [FEATURE] More convenience methods in Response ----------------------------------------------------------------------------------------- This adds a few more convenience methods for setting headers and cache control directives in the Response object: * setDate() / getDate() * setLastmodified() / getLastModified() * setMaximumAge() / getMaximumAge() * setSharedMaximumAge() / getSharedMaximumAge() * Related: `#33371 `_ * Commit: `b18a9a5 `_ [FEATURE] Support for HTTP Cache-Control headers ----------------------------------------------------------------------------------------- This adds support for the Cache-Control header. The "Headers" object now handles the Cache-Control header in a special way by keeping track of the various cache directives separately. Those can be set through the method setCacheControlDirective() and get('Cache-Control') returns a rendered version based on the previously set directives. Alternatively, a whole Cache-Control header can also be set through set() / setHeader(). The given field value will then be parsed and the individual cache directives are stored internally. This patch also corrects the behavior of set() and get() to convert date / time values always to GMT. Also added a related note in the comments of the respective API methods. This patch also adds the new API methods setPrivate() and setPublic() to the Response class. * Related: `#33371 `_ * Commit: `fb8b064 `_ [BUGFIX] Fix error message about command controller classes ----------------------------------------------------------------------------------------- If you create a CommandController which is not in the "Controller" directory of a package or where the name does not end in "CommandController" you will get an error saying "Invalid controller class name [...] 1305100019". This change makes the error message a little more helpful. * Fixes: `#33139 `_ * Commit: `5756509 `_ [FEATURE] Add @depends annotation to ignored ones ----------------------------------------------------------------------------------------- To allow test dependencies in Unit Tests, the @depends annotation is added to the ignored annotation array. * Resolves: `#37475 `_ * Commit: `d84ffe2 `_ [FEATURE] Add hasRole() method to Account entity ----------------------------------------------------------------------------------------- At the moment the roles still have to be persisted as strings in the serialized roles property in the database. Already converting all roles to objects would require some more refactoring. This change is tested on already existing, and on new users. * Related: `#37451 `_ * Commit: `a79904d `_ [FEATURE] Support for HTTP PUT/DELETE arguments ----------------------------------------------------------------------------------------- This adds transparent argument support for arguments in PUT and DELETE requests which are passed through the request body. Depending on the content type of the request, the message body is parsed and then mapped into arguments for further use in controllers. The following content content types are currently supported: * application/x-www-form-urlencoded * application/json * xml * Resolves: `#36913 `_ * Related: `#37402 `_ * Related: `#33371 `_ * Commit: `26f83dd `_ [TASK] Common base class Message for Request / Response ----------------------------------------------------------------------------------------- This introduces a new class "Message" which acts as a common parent class for the HTTP Request and Response. It provides shared functionality such as setting and getting headers, charset handling and more. It is in line with the terminology in RFC 2616. * Related: `#37259 `_ * Commit: `7ac583f `_ [TASK] Temporarily remove nested response code ----------------------------------------------------------------------------------------- This temporarily removes the code in HTTP Response which tackles support of nested responses. This feature needs to be implemented only when most of the other features, including Cache Control, are in place. * Related: `#33371 `_ * Commit: `238efb0 `_ [FEATURE] Support for HTTP Age and Date header ----------------------------------------------------------------------------------------- This patch adds API methods and internal handling of the HTTP "Date" and "Age" headers, according to RFC 2616. It also introduces the use of the "Now" singleton as a time reference for age calculations. * Related: `#37259 `_ * Commit: `a1c37e3 `_ [TASK] Annotation cleanup for http response __toString method ----------------------------------------------------------------------------------------- * Related: `#34875 `_ * Commit: `a2e8c45 `_ [TASK] IdentityRoutePart should return __identity array ----------------------------------------------------------------------------------------- IdentityRoutePart::matchValue() currently returns the identifier of the ObjectPathMapping record directly. This makes it difficult to merge arguments with data from the requestBody (e.g. for PUT requests). This change adjusts the route part so that it sets $this->value to an array in the format array('__identity' => '') * Fixes: `#37403 `_ * Related: `#37402 `_ * Commit: `02fc35b `_ [FEATURE] Fluent Interface for PropertyMappingConfiguration methods ----------------------------------------------------------------------------------------- Setter and other methods not intended to return particular values now return $this in order to provide a Fluent Interface that allows chaining of multiple calls to the same object. * Resolves: `#34866 `_ * Commit: `2eeca51 `_ [BUGFIX] Http Response can be cast to string again ----------------------------------------------------------------------------------------- This reverts the removal of the __toString() method for the Http Response. We currently rely on this behavior in the ActionController which combines response content by simple string concatenation. * Commit: `aa303c3 `_ [TASK] HTTP Response: Status Code and Content Type ----------------------------------------------------------------------------------------- This adds a few new methods to the HTTP Response class - including the related unit tests. * Related: `#37259 `_ * Commit: `1bf8bbd `_ [FEATURE] Introduce "Now" as unique time reference ----------------------------------------------------------------------------------------- This patch introduces a new class called "Now" which inherits DateTime and has the scope Singleton. It can be used in all cases where a reliably comparable value of "now" is needed or where time needs to be mocked in related tests. * Resolves: `#37376 `_ * Commit: `7f9fb7d `_ [BUGFIX] Nested arrays in Annotation options are no longer dropped ----------------------------------------------------------------------------------------- When giving a nested array of options to an annotation the rendered form in proxy classes was cut off. * Fixes: `#37193 `_ * Commit: `5a2cbd0 `_ [BUGFIX] credentialsSource in Account should be nullable ----------------------------------------------------------------------------------------- Not all authentication mechanisms need a "credentials source" (e.g. the typo3.org SSO provider), so the column for the property should be nullable. * Fixes: `#37341 `_ * Commit: `d3e991d `_ [BUGFIX] Fix wrong hint in code migration ----------------------------------------------------------------------------------------- * Fixes: `#37224 `_ * Commit: `5d3a102 `_ [FEATURE] Day, month, year keys in DateTimeConverter source array ----------------------------------------------------------------------------------------- This patch introduces additional possible array keys for the source array of the DateTimeConverter. Now not only a string representing the DateTime can be passed, but array keys of the single day, month and year. * Resolves: `#35195 `_ * Commit: `01d918d `_ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ TYPO3.Fluid ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ [TASK] Update composer.json with new dependencies ----------------------------------------------------------------------------------------- * Commit: `8ad378a `_ [TASK] Remove API annotation from View::canRender() ----------------------------------------------------------------------------------------- With #40888 View::canRender() is no longer called from within ActionController::resolveView(). As the framework no longer relies on that method to exist the @api annotation is removed. * Related: `#40888 `_ * Related: `#42176 `_ * Commit: `06ac5bc `_ [BUGFIX] Allow one of action and actionUri to be omitted in Form ViewHelper ----------------------------------------------------------------------------------------- With I03fa30be2522ee53d5a8f184b57afe539ed4f991 the action argument of the Form ViewHelper was made required. But in fact you can override the action attribute by specifying the "actionUri" argument. In that case the "action" argument should not be required. This removes this requirement again. But if neither "action" nor "actionUri" is specified, an exception will be thrown. This also removes the obsolete method setFormActionUri() * Related: `#43589 `_ * Commit: `8bc51f2 `_ [TASK] Prepare composer manifest for release ----------------------------------------------------------------------------------------- Remove the version field, so versions can be read from tags in git instead. Fixes the description slightly. * Related: `#43822 `_ * Commit: `af16656 `_ [!!!][BUGFIX] Mark action argument required in routing related ViewHelpers ----------------------------------------------------------------------------------------- Currently you can omit the action attribute when creating a link/URI in Fluid. If no action was specified the default action of the matching route was assumed. With #43589 the behavior changed and you'll have to specify all defaults of a route. This change makes the action argument of follwing ViewHelpers required: * f:link.action * f:uri.action * f:form This is a breaking change if you created a link without specifying the action. In this case change your Fluid code from to (or whatever action is the default of the target controller). * Related: `#43589 `_ * Commit: `c9d4c4a `_ [TASK] Avoid use of getPropertyInternal() ----------------------------------------------------------------------------------------- The getPropertyInternal() method obviously is intended for internal use in the ObjectAccess class. This change replaces it's use with a call to getProperty(). * Related: `#43617 `_ * Commit: `c06652a `_ [TASK] Removed testable HTTP flag from tests ----------------------------------------------------------------------------------------- Testable HTTP is now always enabled, thus the flag to switch it on can be removed. * Related: `#43590 `_ * Commit: `2a188cb `_ [BUGFIX] Make name attribute of LayoutViewHelper optional ----------------------------------------------------------------------------------------- This change removes the required = TRUE from the registerArgument() call for the name attribute of LayoutViewHelper. This matches the fallback behavior in the postParseEvent() method. * Fixes: `#43579 `_ * Commit: `5ce7ad5 `_ [!!!][FEATURE] Allow Fluid arrays only in ViewHelper arguments ----------------------------------------------------------------------------------------- This change greatly enhances the JavaScript compatibility of Fluid. Fluid Arrays are a subset of the JavaScript object syntax, making it hard to work with them in mixed HTML/JavaScript documents. For example before this change, the following JavaScript Object was parsed by Fluid:: var uris = { endPoint1: '{f:uri.action(…)}', endPoint2: '{f:uri.action(…)}', }; Before this change, the above snippet resulted in the following template, as the generated array was casted to a string: var uris = Array With this change, Fluid only parses arrays which are used inside ViewHelper arguments, such that an array inside normal text is not converted anymore. Breaking Change =============== This change is only breaking in very rare cases where one relied on the inner contents of the ViewHelper being an array, f.e. if one used the debug ViewHelper as follows:: {key1: 'value1', key2: 'value2'} … or if anybody wrote custom ViewHelpers which use this convention. Relying on that would be very fragile anyways, as the insertion of a single space character before the opening curly bracket or after the closing one would cast the array back to a string. ViewHelpers which were written like this should be re-written to take the array as ViewHelper argument:: * Resolves: `#43356 `_ * Commit: `ba09ec5 `_ [!!!][FEATURE] Allow ResourceViewHelper to accept resource:// paths ----------------------------------------------------------------------------------------- This enables the path argument of the Uri.ResourceViewHelper to accept also resource URIs in the format "resource://Package.Name/Public/...". This functionality used to be achievable with the former uri argument but was removed with I92dccba6b5acd623ff33eb538e62d0682f00b95e. This is marked breaking because it introduces Exceptions in case of incorrect argument usage or values. * Commit: `5504bee `_ [BUGFIX] Fix Doc comment in link.email ViewHelper ----------------------------------------------------------------------------------------- Fixes the code example in the classes doc comment and replaces a minor whitespace issue * Fixes: `#43121 `_ * Commit: `d21bd9e `_ [TASK] Add member for PaginateController::objects ----------------------------------------------------------------------------------------- Adds declaration for the $objects member in the controller of the pagination widget. * Commit: `5695651 `_ [BUGFIX] Fix broken WidgetTest ----------------------------------------------------------------------------------------- The fix for #42511 broke the functional WidgetTest in Fluid, because it relied on "broken" URIs being accepted. The AJAX URI is now prefixed with "http://localhost/" to make the test work again. * Fixes: `#42746 `_ * Related: `#42511 `_ * Commit: `65ea61e `_ [!!!][FEATURE] I18n support for uri.resource VH in Fluid ----------------------------------------------------------------------------------------- This adds the (already documented) localization of resources to the uri.resource VH. Any path will be localized if possible, unless the feature is switched off using the "localize" argument. The change is marked as breaking because it removed the "uri" argument formerly available to the VH. * Resolves: `#32339 `_ * Resolves: `#39792 `_ * Commit: `e9a21a4 `_ [BUGFIX] Fluid should depend on the composer installer ----------------------------------------------------------------------------------------- * Commit: `913272d `_ [TASK] Adjust to FLOW3 renamed to TYPO3 Flow ----------------------------------------------------------------------------------------- Adjusts the code, comments, documentation as needed. * Related: `#41466 `_ * Commit: `284e8f8 `_ [TASK] Update composer type to new naming scheme ----------------------------------------------------------------------------------------- In accordance with the rename to TYPO3 Flow, the composer-type is now ``typo3-flow-*`` * Related: `#41466 `_ * Commit: `45a3117 `_ [TASK] Change dependency to require flow3/composer-installers ----------------------------------------------------------------------------------------- * Commit: `5682214 `_ [TASK] Use composer version of installers ----------------------------------------------------------------------------------------- * Commit: `8061eeb `_ [TASK] Add christianjul/installers dependency ----------------------------------------------------------------------------------------- * Related: `#41031 `_ * Commit: `4449c2a `_ [TASK] Update unit tests to vfsStream > 1.0 ----------------------------------------------------------------------------------------- This adjusts the unit tests to the new vfsStream API. * Related: `#37813 `_ * Commit: `e42dfb0 `_ [TASK] Make PSR-0 and composer compatible ----------------------------------------------------------------------------------------- Moves all classes into a PSR-0 directory structure and adds a composer manifest instead of Package.xml. * Related: `#41031 `_ * Commit: `dfab7ff `_ [FEATURE] Allow using Locale in Currency and Date ViewHelper ----------------------------------------------------------------------------------------- This enables switching the use of the current active Locale in Format.Currency and Format.Date ViewHelper. * Resolves: `#33143 `_ * Commit: `afdbc77 `_ [TASK] Whitespace fix in class docblock ----------------------------------------------------------------------------------------- * Commit: `f20e61d `_ [TASK] Code documentation fix on return types ----------------------------------------------------------------------------------------- The variable container accepts any type, not only objects. * Commit: `bbb9951 `_ [TASK] Enhance exception for duplicate variables ----------------------------------------------------------------------------------------- The exception message for 'Duplicate variable declarations!' now contains the variablename which is doubled. * Resolves: `#40828 `_ * Commit: `438da61 `_ [FEATURE] Format.JsonViewHelper escapes tag characters per default ----------------------------------------------------------------------------------------- Since occuring plain greater-than or less-than characters in HTML context could be dangerous (even CDATA sequences can be stopped with this), the tags are now converted to their hexadecimal representation. Since this is standard JSON syntax, this is considered not to be a breaking change. * Resolves: `#39886 `_ * Commit: `3d6a932 `_ [TASK] Support literal strings in boolean arguments ----------------------------------------------------------------------------------------- This change adds support for string literals in boolean comparisons. String literals have to be surrounded by single or double quotes and may contain escaped single or double quotes. * Resolves: `#6757 `_ * Commit: `125413c `_ [BUGFIX] TemplateParser unquoteString does not trim escaped quotes correctly ----------------------------------------------------------------------------------------- This change fixes wrong trimming of trailing quotes in escaped sequences by replacing trim() calls with a preg_replace() alternative. * Fixes: `#39604 `_ * Commit: `009b1fc `_ [FEATURE] Reserved variables represent their meaning when accessing ----------------------------------------------------------------------------------------- All reserved variables, ie true, false, on, off etc. now represent their intended value when accessing them. * Resolves: `#39410 `_ * Commit: `3000719 `_ [BUGFIX] Action URI of the form ViewHelper is not reset ----------------------------------------------------------------------------------------- Form ViewHelpers store the action URI attribute for performance reasons. But because instances of ViewHelpers are reused, this value needs to be reset. Besides this removes an obsolete property "formActionUriArguments" * Fixes: `#39456 `_ * Related: `#11636 `_ * Commit: `6945389 `_ [BUGFIX] Make TextareaViewHelper HTML5 aware ----------------------------------------------------------------------------------------- This makes the arguments "rows" and "cols" of the form.textarea ViewHelper optional. * Fixes: `#31189 `_ * Commit: `2ed2091 `_ [TASK] Make reflectionService property in AbstractViewHelper protected ----------------------------------------------------------------------------------------- The property "reflectionService" in the AbstractViewHelper should be protected instead of private to allow usage of the reflection service in view helpers without having to inject again with another property name. * Resolves: `#39255 `_ * Commit: `4b3c096 `_ [TASK] Correct StripTags test because of unintended incompletion result ----------------------------------------------------------------------------------------- A test's data provider's method name began with the character sequence 'test', was hence regarded as test to be executed. This resulted into an "Incomplete" result when running the tests. * Commit: `96f77ee `_ [BUGFIX] Make use of dashes in attribute names possible ----------------------------------------------------------------------------------------- When using e.g. data-foo in VH tags, the parser would choke. This change fixes this. * Fixes: `#38592 `_ * Commit: `88d7486 `_ [BUGFIX] Move functional test configuration to Testing ----------------------------------------------------------------------------------------- This moves the functional test configuration to the Testing context, to avoid errors in other contexts. * Related: `#38873 `_ * Commit: `50370f1 `_ [BUGFIX] Remove eval from StandaloneViewTest ----------------------------------------------------------------------------------------- Removes the use of eval from the StandaloneViewTest so that now a Fixture class is used that extends the StandaloneView. * Fixes: `#38873 `_ * Related: `#31179 `_ * Commit: `ec1b904 `_ [BUGFIX] Replace placeholders in fallback for id translation ----------------------------------------------------------------------------------------- When using id based translation and no translation is defined in XLIFF, the tag value is used as a fallback in TranslateViewHelper. With this change, the fallback is passed to label based translation, making placeholder replacement work also if no translation does exist yet. * Fixes: `#37387 `_ * Commit: `96189df `_ [BUGFIX] Remove (wrong) use of Internal annotation ----------------------------------------------------------------------------------------- The internal annotation is used to declare command methods as internal, not to mark things as not being public API. Furthermore, it is not allowed to be used on properties and with #38667 fixed Fluid would cause exceptions. * Related: `#38667 `_ * Commit: `55e055c `_ [TASK] Code cleanup ----------------------------------------------------------------------------------------- Does code cleanup, mostly to documentation, but also tweaks unused variables, missing return statements and similar things. * Related: `#34875 `_ * Commit: `edd10b4 `_ [BUGFIX] Only render hidden upload fields if a resource is available ----------------------------------------------------------------------------------------- The UploadViewHelper always creates hidden fields for filename and resource pointer so that previously uploaded files are not lost if validation errors occur. This change makes sure that those hidden fields are only rendered if a file has been uploaded before because otherwise the property mapper tries to create a Resource object and fails with an exception (see #38184). * Related: `#38184 `_ * Commit: `630992b `_ [TASK] Add query parameters for form action as hidden fields ----------------------------------------------------------------------------------------- This change makes the form viewhelper usable with method GET by adding query parameters of the action as hidden fields to the form. Otherwise the browser would replace them by the form values (http://www.w3.org/TR/html401/interact/forms.html#h-17.13.3.4). * Resolves: `#11636 `_ * Commit: `488b238 `_ [BUGFIX] Fix unusable checkbox/radio if formObject is NULL ----------------------------------------------------------------------------------------- When using a form object inside of a form view helper inside an index action template and a checkbox is referring to a property $sth of $someFormObject, the form object can be NULL with code like:: indexAction(\\Some\\Type $someFormObject = NULL){ ... } Initially calling the index action makes someFormObject NULL. If the submitted form had validation errors someFormObject is NULL, too. This possibility was ignored in a call for the formObject from within the CheckboxViewHelper as well as from the RadioViewHelper. So you would get an exception telling you No value found for key "TYPO3\\Fluid\\ViewHelpers\\FormViewHelper->formObject" This change fixes that bug and thus also sets the checkbox and radio "checked" attribute properly when a validation error occurred. * Fixes: `#35894 `_ * Commit: `cbe9304 `_ [FEATURE] Add a ViewHelper for entity identifier retrieval ----------------------------------------------------------------------------------------- This adds a ViewHelper "format.identifier" that returns the plain identifier for a given entity. * Resolves: `#33494 `_ * Commit: `c8d53e9 `_ [BUGFIX] ViewHelper exceptions should not be caught in testing context ----------------------------------------------------------------------------------------- While it is correct that ViewHelper exceptions are silently logged in production context, they should be displayed in development and testing context; and not only in development context. * Fixes: `#37671 `_ * Commit: `166e7e5 `_ [TASK] Update Package.xml with new version number ----------------------------------------------------------------------------------------- * Commit: `575adc2 `_ [TASK] Standalone View: set default format "html" ----------------------------------------------------------------------------------------- Since the introduction of Content Negotiation to FLOW3 (I7d753155aa72f41e3df85b076e8067bcbfe7ac04), the default Format of the ActionRequest is not "html" but undefined. The Standalone View relied on the default "html" format and thus now needs to set it itself. * Related: `#37561 `_ * Commit: `db77eca `_ [BUGFIX] hidden fields should be rendered in extra section ----------------------------------------------------------------------------------------- Currently form.select and form.checkbox ViewHelpers render hidden fields before the actual input field in order to force submission of a value. This leads to the situation that labels wrapped around the field do not correspond to the actual form field, but to the hidden field. This is fixed by letting the form ViewHelper taking care of rendering the requried hidden fields in the already existing section on top of the form. * Fixes: `#26481 `_ * Fixes: `#13342 `_ * Commit: `f25fa5f `_ [BUGFIX] Avoid empty output of TranslateViewHelper ----------------------------------------------------------------------------------------- If translation by ID is used but no translation is found, the ID will now be returned if the view helper tag has no content. Before this change it would just return nothing (printable, at least). * Fixes: `#35041 `_ * Commit: `68e3bad `_ [FEATURE] Allow access to attributes in TagBuilder ----------------------------------------------------------------------------------------- * Resolves: `#37460 `_ * Commit: `afaaceb `_ [TASK] Disable escapingInterceptor in Format.Json ViewHelper ----------------------------------------------------------------------------------------- Because the provided ViewHelper children will never be put through to the template but only be used for source value retrieval, the escapingInterceptor gets disabled. * Related: `#12442 `_ * Commit: `50dcde3 `_ [FEATURE] Adds a format.json ViewHelper ----------------------------------------------------------------------------------------- Provide a ViewHelper that enables putting JSON out of an array or object. Basically wraps PHP's native json_encode() method. * Resolves: `#12442 `_ * Commit: `87ea437 `_ [TASK] Clean up some coding style issues ----------------------------------------------------------------------------------------- Add spaces to "){" and to "foreach(", fixes some docblocks and indentation. Also avoids a possible uninitialised variable warning. * Related: `#34875 `_ * Commit: `1a68840 `_ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ TYPO3.Party ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ [TASK] Update composer.json with new dependencies ----------------------------------------------------------------------------------------- * Commit: `587018d `_ [TASK] Prepare composer manifest for release ----------------------------------------------------------------------------------------- Remove the version field, so versions can be read from tags in git instead. * Related: `#43822 `_ * Commit: `e15b0c2 `_ [TASK] Cleanup use statements with leading backslash ----------------------------------------------------------------------------------------- An almost purely cosmetic issue, but still: use always takes fully qualified class names, thus no leading backslash should be used. * Commit: `aae5f6e `_ [BUGFIX] Fix down migration for Mysql, Version20120429213446 ----------------------------------------------------------------------------------------- * Fixes: `#41482 `_ * Commit: `930a1d4 `_ [BUGFIX] Party should depend on the composer installer ----------------------------------------------------------------------------------------- * Commit: `4f814fe `_ [TASK] Adjust to FLOW3 renamed to TYPO3 Flow ----------------------------------------------------------------------------------------- Adjusts the code, comments, documentation as needed. * Related: `#41466 `_ * Commit: `483d978 `_ [TASK] Update composer type to new naming scheme ----------------------------------------------------------------------------------------- In accordance with the rename to TYPO3 Flow, the composer-type is now ``typo3-flow-*`` * Related: `#41466 `_ * Commit: `4e0c4b3 `_ [TASK] Adjust composer manifest ----------------------------------------------------------------------------------------- * Related: `#41031 `_ * Commit: `1b33e69 `_ [TASK] Change dependency to require flow3/composer-installers ----------------------------------------------------------------------------------------- * Commit: `9ef4f5f `_ [TASK] Add christianjul/installers dependency ----------------------------------------------------------------------------------------- * Related: `#41031 `_ * Commit: `1961f20 `_ [TASK] Make PSR-0 and composer compatible ----------------------------------------------------------------------------------------- Moves all classes into a PSR-0 directory structure and adds a composer manifest instead of Package.xml. * Related: `#41031 `_ * Commit: `cbe85f3 `_ [TAKS] Whitespace fix in class docblock ----------------------------------------------------------------------------------------- * Commit: `6201a38 `_ [TASK] Code cleanup ----------------------------------------------------------------------------------------- Tweaks a single constructor docblock. * Related: `#34875 `_ * Commit: `ac08ac5 `_ [TASK] Adjust validators to avoid validating more than needed ----------------------------------------------------------------------------------------- Corresponds to If2975bb2e4b79fd7227bda9c7c65b32a7f210e8e. * Related: `#34154 `_ * Commit: `356939c `_ [TASK] Update Package.xml with new version number ----------------------------------------------------------------------------------------- * Commit: `ed0a605 `_ [TASK] Make usagetype on ElectronicAddress nullable ----------------------------------------------------------------------------------------- The usage property has no default value and thus should be nullable to make "any" usage possible. Adds a functional test (thanks Adrian Föder) as well. * Related: `#35059 `_ * Commit: `e1d0e7b `_ [TASK] Adjust validators to avoid validating more than needed ----------------------------------------------------------------------------------------- * Related: `#34145 `_ * Commit: `eece558 `_ [BUGFIX] Duplicate email address throws exception ----------------------------------------------------------------------------------------- Since #29547 is resolved, trying to store an email address that already exists throws an exception. This change drops the unique indexes for "identifier", "type" and "usagetype" from the ElectronicAddress domain model. * Fixes: `#37266 `_ * Related: `#29547 `_ * Commit: `ed1f4b8 `_ [TASK] Clean up some coding style issues ----------------------------------------------------------------------------------------- Add space to "foreach(". * Related: `#34875 `_ * Commit: `6445053 `_ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ TYPO3.Kickstart ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ [TASK] Update composer.json with new dependencies ----------------------------------------------------------------------------------------- * Commit: `c2c556a `_ [TASK] Prepare composer manifest for release ----------------------------------------------------------------------------------------- Remove the version field, so versions can be read from tags in git instead. * Related: `#43822 `_ * Commit: `3c7e9c5 `_ [BUGFIX] Fix strpos() parameters in controller kickstart ----------------------------------------------------------------------------------------- strpos() parameters were specified in reverse order which broke creation of controllers for subpackages. * Fixes: `#43423 `_ * Commit: `97c03c5 `_ [BUGFIX] Fix type hint of collections in generated model classes ----------------------------------------------------------------------------------------- This makes sure that, when using collections with element type hints, only the collection class name ends up in the type hint of the generated setter. * Fixes: `#29791 `_ * Commit: `3060ff7 `_ [BUGFIX] Fix error message about invalid package key ----------------------------------------------------------------------------------------- When specifying an invalid package key in the kickstart commands, the error message now includes a note about the vendor namespace that has to be added. * Fixes: `#43194 `_ * Commit: `7a4886a `_ [BUGFIX] Fix case of class names to generate ----------------------------------------------------------------------------------------- This makes sure the created controller class names and references are upper cased in order to avoid misleading exceptions if the controller was specified lower cased. * Fixes: `#43144 `_ * Commit: `ee5f026 `_ [TASK] Update composer type to new naming scheme ----------------------------------------------------------------------------------------- In accordance with the rename to TYPO3 Flow, the composer-type is now ``typo3-flow-*`` * Related: `#41466 `_ * Commit: `4280efb `_ [TASK] Adjust to FLOW3 renamed to TYPO3 Flow ----------------------------------------------------------------------------------------- Adjusts the code, comments, documentation as needed. * Related: `#41466 `_ * Commit: `5ee2ca0 `_ [TASK] Add dependency to require flow3/composer-installers ----------------------------------------------------------------------------------------- * Commit: `c86dcb7 `_ [TASK] Update to write in PSR-0 compatible path ----------------------------------------------------------------------------------------- * Fixes: `#41180 `_ * Commit: `3cd555a `_ [TASK] Make PSR-0 and composer compatible ----------------------------------------------------------------------------------------- Moves all classes into a PSR-0 directory structure and adds a composer manifest instead of Package.xml. * Related: `#41031 `_ * Commit: `e23e727 `_ [TAKS] Whitespace fix in class docblock ----------------------------------------------------------------------------------------- * Commit: `e7c7a48 `_ [BUGFIX] kickstart:package gives wrong hint for valid package key ----------------------------------------------------------------------------------------- The kickstarter wrongly claims that a valid package key may contain underscores. This patch corrects the respective messages. * Fixes: `#38999 `_ * Commit: `211dcab `_ [TASK] Code cleanup ----------------------------------------------------------------------------------------- Does cleanup to code documentation. * Related: `#34875 `_ * Commit: `d6228ba `_ [TASK] Update Package.xml with new version number ----------------------------------------------------------------------------------------- * Commit: `ed26ddf `_ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ TYPO3.Welcome ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ [TASK] Update composer.json with new dependencies ----------------------------------------------------------------------------------------- * Commit: `662a7be `_ [BUGFIX] Adjust tests to required action arguments ----------------------------------------------------------------------------------------- Adjusts tests to changed requirements introduced with I03fa30be2522ee53d5a8f184b57afe539ed4f991 * Related: `#43589 `_ * Commit: `7ad7b6c `_ [TASK] Prepare composer manifest for release ----------------------------------------------------------------------------------------- Remove the version field, so versions can be read from tags in git instead. Fixes the description slightly. * Related: `#43822 `_ * Commit: `c7697af `_ [TASK] Update logo to TYPO3 Flow ----------------------------------------------------------------------------------------- * Resolves: `#42766 `_ * Commit: `f05652e `_ [TASK] Update composer type to new naming scheme ----------------------------------------------------------------------------------------- In accordance with the rename to TYPO3 Flow, the composer-type is now ``typo3-flow-*`` * Related: `#41466 `_ * Commit: `44a146a `_ [TASK] Adjust to FLOW3 renamed to TYPO3 Flow ----------------------------------------------------------------------------------------- Adjusts the code, comments, documentation as needed. * Related: `#41466 `_ * Commit: `9a04182 `_ [TASK] Adjust composer manifest ----------------------------------------------------------------------------------------- * Related: `#41031 `_ * Commit: `79fade8 `_ [TASK] Make PSR-0 and composer compatible ----------------------------------------------------------------------------------------- Moves all classes into a PSR-0 directory structure and adds a composer manifest instead of Package.xml. * Related: `#41031 `_ * Commit: `a18695b `_ [TASK] Update Package.xml with new version number ----------------------------------------------------------------------------------------- * Commit: `f889660 `_