Getting the following error when adding a feed in the aggregator:
FastCGI sent in stderr: "PHP message: Uncaught PHP Exception Drupal\Component\Plugin\Exception\PluginNotFoundException: "The "FeedTitle" plugin does not exist." at /home/user/html/drupal/core/lib/Drupal/Component/Plugin/Discovery/DiscoveryTrait.php line 57" while reading response header from upstream, client: 127.0.0.1, server: dev.drupal.me, request: "POST /aggregator/sources/add HTTP/1.1", upstream: "fastcgi://unix:/var/run/php-fpm/php-fpm.sock:", host: "dev.drupal.me", referrer: "http://dev.drupal.me/aggregator/sources/add"
Using nginx with the perusio's config.

CommentFileSizeAuthor
#11 2419005.11.patch1.7 KBalexpott
#11 8-11-interdiff.txt1.65 KBalexpott
#8 interdiff.txt1.15 KBdawehner
#8 2419005-8.patch2.79 KBdawehner
#5 2419005-4.patch2.79 KBdawehner
#4 2419005-4.patch54.41 KBdawehner
#4 2419005-fail.patch835 bytesdawehner
#3 2419005-3.patch53.14 KBdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented
Status: Needs work » Active

i was able to reproduce it, but i dont get why testbot doesnt fail...there are certainly tests for this :/

ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented

ah, i see..i get this right after enabling the module..if i clear cache, it works..its probably why testbot doesnt have those fails

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Active » Needs review
FileSize
2419005-3.patch53.14 KB

This is one way to solve it.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
FileSize
2419005-fail.patch835 bytes
2419005-4.patch54.41 KB

And this time with a test.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
FileSize
2419005-4.patch2.79 KB

.. this time with a proper patch.

The last submitted patch, 4: 2419005-fail.patch, failed testing.

ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented

thanks for the patch!

+++ b/core/modules/system/src/Tests/Entity/EntityValidationTest.php
@@ -79,6 +79,19 @@ protected function createTestEntity($entity_type) {
+    $get_cached_discoviers = function () {

nit: typo, probably discoveries

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
FileSize
2419005-8.patch2.79 KB
interdiff.txt1.15 KB

HAHA, this looked super weird, but I could not figure out why.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan commented
Status: Needs review » Reviewed & tested by the community

Assuming green.
Should we be documenting somewhere that there is some black magic in plugin.manager.{x} naming that results in the plugin cache being cleared on module install?

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Needs work

We can do this without the service name change (and therefore API change) and the magic. Just tag the service with 

    tags:
      - { name: plugin_manager_cache_clear }
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Needs work » Needs review
FileSize
8-11-interdiff.txt1.65 KB
2419005.11.patch1.7 KB

Like so.

ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented
Status: Needs review » Reviewed & tested by the community

good to know where the magic is hidden :P Thanks!

  • catch committed d9a318b on 8.0.x 
    Issue #2419005 by dawehner, alexpott: Aggregator: Uncaught PHP Exception...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.