Check Collection exists or not. [#2939458]

Hi,

I have updated the core to 8.4.4. whenever I submit any entity which does not have a collection. I am getting the following error. I found it is throwing from the event subsriber. So, I have created a patch for this please review. Added !empty() condition before adding the token_devel route in collection.

Drupal\Core\Entity\EntityStorageException: Some mandatory parameters are missing ("workflow_transition") to generate a URL for route "entity.workflow_scheduled_transition.token_devel". in Drupal\Core\Entity\Sql\SqlContentEntityStorage->save() (line 805 of C:\xampp\htdocs\site\core\lib\Drupal\Core\Entity\Sql\SqlContentEntityStorage.php).

Comment	File	Size	Author
#2	collection_exists-2939458-2.patch	552 bytes	saranya ashokkumar
#2	8.x-1.x: PHP 7 & MySQL 5.5, D8.6 52 pass

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

25 January 2018 at 13:55

saranya purushothaman created an issue. See original summary.

Comment #2

saranya ashokkumar CreditAttribution: saranya ashokkumar at UniMity Solutions Pvt Limited commented 25 January 2018 at 13:56

Status:

Active

» Needs review

File	Size
collection_exists-2939458-2.patch	552 bytes
8.x-1.x: PHP 7 & MySQL 5.5, D8.6 52 pass

Comment #3

vani.S CreditAttribution: vani.S commented 30 March 2018 at 13:38

Status:

Needs review

» Reviewed & tested by the community

This patch worked for me. I also faced this problem.

Comment #4

MRPRAVIN CreditAttribution: MRPRAVIN at Valuebound commented 1 April 2018 at 04:49

I tested this patch working fine. Thanks for the patch.

Comment #5

Berdir

German

Switzerland

CreditAttribution: Berdir at MD Systems GmbH commented 23 May 2018 at 21:05

Status:

Reviewed & tested by the community

» Needs work

The patch has coding style issues (for example missing space after if).

I don't think that's the right fix, might work for that example because it happens to not have a collection route but that's pretty arbitrary. Will need a better check, for example having more than one argument in the path.