Danny v W
Danny v W
When the nodetranslation not exists it will trigger an error on this line https://github.com/Kunstmaan/KunstmaanBundlesCMS/blob/6.x/src/Kunstmaan/FormBundle/Controller/FormSubmissionsController.php#L154
Not all form errors are visible in an admin list. Validation for the name field is visible but the collection is not. The symfony toolbar shows the validation error for...
This method https://github.com/Kunstmaan/KunstmaanBundlesCMS/blob/6.x/src/Kunstmaan/NodeBundle/Helper/URLHelper.php#L140 can return a boolean if the media is not found. getNodeTranslation has the same issue. Error: Kunstmaan\NodeBundle\Helper\URLHelper::getMedia(): Return value must be of type array, bool returned
The validation is not triggered if you left the field blank because it returns a null value instead of empty string. https://github.com/Kunstmaan/KunstmaanBundlesCMS/blob/6.x/src/Kunstmaan/MenuBundle/Entity/BaseMenuItem.php#L352
It is possible to override the Menu Item entity (https://github.com/Kunstmaan/KunstmaanBundlesCMS/blob/6.x/src/Kunstmaan/MenuBundle/DependencyInjection/Configuration.php#L25C33-L25C45) but in the BaseMenu this is an hardcoded reference https://github.com/Kunstmaan/KunstmaanBundlesCMS/blob/6.x/src/Kunstmaan/MenuBundle/Entity/BaseMenu.php#L41 This should be set with an listener based on the...
Deleted nodes can be edited via the admin if you know the ID, does that make sense? I would expect these nodes to become unavailable.
If you delete a parent node the child nodes are not deleted, but the child nodes are not available/working anymore. The link gives a 404.
The URLHelper generate a url for a deleted node because this query (https://github.com/Kunstmaan/KunstmaanBundlesCMS/blob/6.x/src/Kunstmaan/NodeBundle/Helper/URLHelper.php#L129) does not check if the node was deleted. The url leads to an 404 page. Also fetchAssociative...
The internal _nodeTranslation parameter is added to pagination urls and should not exist in urls. Possible due changes in Symfony 5.x because Symfony 4.x does not have this issue. Possible...
Maybe is changing the translation enough instead of all code.