8.0.7 (2022-11-03)
Overview of merged pull requests
BUGFIX: 3371 shortcut translations
closes: #3371
## 1
> the dutch translation f.x. doesnt have correctly formatted <br/> tags. See f.x.: 'shortcut.clickToContinueToParentNode' and compare the spanish < br / > to the english source <br />. A search for < br / > reveals multiple cases. The problem is, that this invalid html (due to the space) and will be rendered as text (< br / >).
before in NL: !`image <https://user-images.githubusercontent.com/85400359/198111355-d5a4d9f5-f1b5-4b83-8f4a-1f3c914b4955.png>`_
after: !`image <https://user-images.githubusercontent.com/85400359/198111385-575bd6b0-4044-436d-a145-e14d4934c778.png>`_
## 2
> this translation id doesnt exist: 'shortcut.toSpecificTarget' so the value / fallback This is a shortcut to a specific target: is always used.
It has been missed in 2014 (since Neos 2) ^^ https://github.com/neos/neos-development-collection/commit/`02f8ff96e1ecd3dfbd9a356c4c6f55bb565916c7 <https://github.com/neos/neos-development-collection/commit/02f8ff96e1ecd3dfbd9a356c4c6f55bb565916c7>``_#diff-``d1bec08a4f5050d463e4f7af3d61b767974d858d <https://github.com/neos/neos-development-collection/commit/d1bec08a4f5050d463e4f7af3d61b767974d858d>`_3d56ced85581ca55c27126dcR6
before in DE (or any other language): !`image <https://user-images.githubusercontent.com/85400359/198111487-39e7c03c-c132-4896-a12d-f25da087e07b.png>`_
after: nothing to see here yet - but now we can translate via weblate :D
Upgrade instructions
Review instructions
Check the shortcode rendering in the backend. If you switch your user language - flush the fusion cache - as this is not calculated into the shortcut cache:
./flow cache:flushone Neos_Fusion_Content
Packages:
Neos
BUGFIX: fix typos in privilegeTargets
Review instructions
Fixes small spelling errors in the priviligeTargets
Packages:
Neos
BUGFIX: DI conflict in DataSource `Call to a member function log Throwable() on null <https://github.com/neos/neos-development-collection/pull/3931>`_
closes #3858
when one throws an exception in a datasource, it fails logging it and an uncaught error is thrown, because the ThrowableStorage is not injected:
Call to a member function log Throwable() on null
this seems due to a dependency injection conflict with inheritance.
this pr only prevents this conflict from happening
the controller that handles datasources extends the AbstractServiceController
which injects the throwableStorage
but the AbstractServiceController also extends the ActionController which injects the throwableStorage via an inject method into a private property
- ```php
- /**
@var ThrowableStorageInterface
*/
private $throwableStorage;
- /**
Injects the throwable storage.
@param ThrowableStorageInterface $throwableStorage
@return void
*/
public function injectThrowableStorage(ThrowableStorageInterface $throwableStorage) {
$this->throwableStorage = $throwableStorage;
}
this combined doesnt work and results in the @Flow\\Inject in the AbstractServiceController not working
solutions are:
fix the flow DI? (assuming this is a issue)
rename the variable (what id did now)
use also
AbstractServiceController::injectThrowableStorageand call parent / not the@Flow\\Injectannotationuse a protected property in
ActionControllerand reuse it
## update 1 this is indeed defined behaviour from flow: https://flowframework.readthedocs.io/en/8.1/TheDefinitiveGuide/PartIII/ObjectManagement.html#property-injection > If a setter method exists for the same property, it has precedence.
Upgrade instructions
Packages:
Neos
BUGFIX: Node::countChildNodes($nodeTypeConstraints) filter doesn’t work
closes #3885
findChildNodes is corrently implemented, but countChildNodes doesnt transform the argument $nodeTypeConstraints NodeTypeConstraints to a string for the legacy api:
either this snipped must be included:
`php
$filter = $nodeTypeConstraints !== null ? $nodeTypeConstraints->asLegacyNodeTypeFilterString() : null;
`
or we must use $this->findChildNodes
without that, the NodeTypeConstraints is passed further down, and due to lack of typesafety it fails at the last moment:
`php
Argument 2 passed to Neos\\Utility\\Arrays::trimExplode() must be of the type string, object given, called in /tmp/neos/Development/SubContextddev/Cache/Code/Flow_Object_Classes/Neos_ContentRepository_Domain_Repository_NodeDataRepository.php on line 1127
`
Upgrade instructions
Packages:
NeosContentRepository
BUGFIX: Fusion Runtime @process with simple string
This makes it possible to use @process with a regular string.
Closes #3846
Review instructions try the fusion code from the issue.
it now works as we switched a condition (the count will only happen if it is an array and not possibly a simple value)
Packages:
NeosFusion
BUGFIX: Make import of .ico files possible
This enables the possibility of uploading .ico files.
Closes #3922
Packages:
NeosMedia
BUGFIX: Fix media commands output
Fix Neos.Media commands media:clearthumbnails, media:createthumbnails and media:removeunused to properly print a final newline.
Output before: ``` user $ ./flow media:clearthumbnails 0 [->————————–]user $
^– next prompt
`
Output after:
`
user $ ./flow media:clearthumbnails
0 [->————————–]
user $
```
Resolved: #3894
Upgrade instructions
None.
Review instructions
in bash, execute
./flow media:clearthumbnails
(by default, zsh is less affected, it recovers but shows a percent sign)
Packages:
NeosMedia
TASK: Render API docs (again)
Packages:
Media.BrowserNeos.github
TASK: Use local fonts instead of from Google
Removes the use of remote fonts from Google in favor of local font files.
Additionally node-sass was updated to allow compilation on M1 Macs.
Replaces #3873
Review instructions
Noto should still be the font used on error pages.
Packages:
Neos
TASK: Rename index to match name in database
Renames the index in the NodeData model to match the index name in migration and database.
Fixes: #3888
Packages:
NeosContentRepository
TASK: Add PHP 8.1 to build matrix
Review instructions
The build on PHP 8.1 should succeed.
Packages:
Neos.github