- Issue created by @pameeela
- π¦πΊAustralia pameeela
Looks like this was introduced by π Fatal when entity type doesn't exist (anymore) Active but this should be handled since node is installed?
- First commit to issue fork.
- πΊπΈUnited States phenaproxima Massachusetts
I started on this, but guess what...Trash breaks the moderated_content view, it turns out, with the following backtrace:
The website encountered an unexpected error. Try again later.<br><br><em class="placeholder">TypeError</em>: strpos(): Argument #1 ($haystack) must be of type string, Drupal\Core\Database\Query\Condition given in <em class="placeholder">strpos()</em> (line <em class="placeholder">141</em> of <em class="placeholder">modules/contrib/trash/src/ViewsQueryAlter.php</em>). Drupal\trash\ViewsQueryAlter->alterQueryForEntityType() (Line: 101) Drupal\trash\ViewsQueryAlter->alterQuery() (Line: 169) trash_views_query_alter() call_user_func_array() (Line: 355) Drupal\Core\Extension\ModuleHandler->Drupal\Core\Extension\{closure}() (Line: 307) Drupal\Core\Extension\ModuleHandler->invokeAllWith() (Line: 354) Drupal\Core\Extension\ModuleHandler->invokeAll() (Line: 1473) Drupal\views\Plugin\views\query\Sql->alter() (Line: 1376) Drupal\views\ViewExecutable->build() (Line: 393) Drupal\views\Plugin\views\display\PathPluginBase->execute() (Line: 198) Drupal\views\Plugin\views\display\Page->execute() (Line: 1690) Drupal\views\ViewExecutable->executeDisplay() (Line: 81) Drupal\views\Element\View::preRenderViewElement() call_user_func_array() (Line: 107) Drupal\Core\Render\Renderer->doTrustedCallback() (Line: 825) Drupal\Core\Render\Renderer->doCallback() (Line: 387) Drupal\Core\Render\Renderer->doRender() (Line: 203) Drupal\Core\Render\Renderer->render() (Line: 238) Drupal\Core\Render\MainContent\HtmlRenderer->Drupal\Core\Render\MainContent\{closure}() (Line: 593) Drupal\Core\Render\Renderer->executeInRenderContext() (Line: 231) Drupal\Core\Render\MainContent\HtmlRenderer->prepare() (Line: 128) Drupal\Core\Render\MainContent\HtmlRenderer->renderResponse() (Line: 90) Drupal\Core\EventSubscriber\MainContentViewSubscriber->onViewRenderArray() (Line: 246) Symfony\Component\EventDispatcher\EventDispatcher::Symfony\Component\EventDispatcher\{closure}() (Line: 206) Symfony\Component\EventDispatcher\EventDispatcher->callListeners() (Line: 56) Symfony\Component\EventDispatcher\EventDispatcher->dispatch() (Line: 188) Symfony\Component\HttpKernel\HttpKernel->handleRaw() (Line: 76) Symfony\Component\HttpKernel\HttpKernel->handle() (Line: 37) Drupal\Core\Test\StackMiddleware\TestWaitTerminateMiddleware->handle() (Line: 53) Drupal\Core\StackMiddleware\Session->handle() (Line: 48) Drupal\Core\StackMiddleware\KernelPreHandle->handle() (Line: 28) Drupal\Core\StackMiddleware\ContentLength->handle() (Line: 116) Drupal\page_cache\StackMiddleware\PageCache->pass() (Line: 90) Drupal\page_cache\StackMiddleware\PageCache->handle() (Line: 48) Drupal\Core\StackMiddleware\ReverseProxyMiddleware->handle() (Line: 51) Drupal\Core\StackMiddleware\NegotiationMiddleware->handle() (Line: 36) Drupal\Core\StackMiddleware\AjaxPageState->handle() (Line: 51) Drupal\Core\StackMiddleware\StackedHttpKernel->handle() (Line: 709) Drupal\Core\DrupalKernel->handle() (Line: 19)
At least the tests catch this, but it means that we probably have to fix a bug in Trash before this can be committed.
- πΊπΈUnited States phenaproxima Massachusetts
Turns out that bug is fixed in π TypeError: strpos(): Argument #1 ($haystack) must be of type string in line 141 of ViewsQueryAlter.php Active , I guess it just needs to be released?
- πΊπΈUnited States phenaproxima Massachusetts
No longer blocked; @amateescu rolled a new patch release of Trash which includes the necessary fix.
-
phenaproxima β
committed d86bf2e8 on 1.x
Issue #3508799 by phenaproxima, pameeela: Trash link is a 404 on install...
-
phenaproxima β
committed d86bf2e8 on 1.x
-
phenaproxima β
committed 0ff5bf72 on 1.0.x
Issue #3508799 by phenaproxima, pameeela: Trash link is a 404 on install...
-
phenaproxima β
committed 0ff5bf72 on 1.0.x
- πΊπΈUnited States phenaproxima Massachusetts
Gave this a quick manual test and proved that /admin/content/trash is no longer a 404 in an install of Drupal CMS that has no content types apart from the basic page. So, merged into 1.x and cherry-picked to 1.0.x!
Automatically closed - issue fixed for 2 weeks with no activity.