Problem/Motivation

There should be a check if the $node is an instance of Node on the downloadAllFiles().

Steps to reproduce

Create a Url::fromRoute('download_all_files.download_path', ['nid' => NODE_ID, 'field_name' => FIELD_NAME]) where NODE_ID is not valid node id.

I’ve created a patch that solves the problem by checking the node’s validity.

CommentFileSizeAuthor
checks_node_validity.patch1.29 KBkarma86

Comments

karma86 created an issue. See original summary.

larisse’s picture

Status: Active » Needs review
jeroent’s picture

Status: Needs review » Closed (duplicate)

I'm closing this as a duplicate of #3197043: Custom entity types.

Once that issue is fixed, this check is no longer necessary because of the changes in the routing.yml file.