ORG REPORT:
When a user navigates to a 404 page, all of the languages available on the site seem to show up in the drop down and create bad links. I suspect this has to do with the fact that the system doesn't treat the 404 page as an entity. I'll work on a patch to temporarily mitigate the issue, but I suspect this may need a more thorough solution involving core.


NOTES:
#3057186: [PLAN] Need some kind of plugin for discovery of entity types and linking logic for non EntityInterface

related postponed:
#3001021: Language Block disapears on Pages built using Page Manager
#3039085: Hide untranslated for View Pages.

CommentFileSizeAuthor
#5 3119474-1.patch706 bytesjcourtice

Comments

jcourtice created an issue. See original summary.

skaught’s picture

Issue summary: View changes

take the block off 404 pages..

depending on how you've setup your project, generally the 404 page is not a node.. autofilter feature does require 'to be on a node'

even rather: setup a translated node groups for you 404's..

skaught’s picture

skaught’s picture

Issue summary: View changes
jcourtice’s picture

StatusFileSize
new706 bytes

Here's a patch that just exits the block creation early w/o returning the build array for the block if we get a 404 or a 403. I may be blind but I don't see a way to exclude the block on error pages via the UI. There's work in core going on but it doesn't seem to ready.

jcourtice’s picture

Status: Active » Needs review
jcourtice’s picture

skaught’s picture

(: ahh.. you're probably not blind -- i think i suggested that too offhand.

i see why/what your doing.. have not tested directly. i will look into this more. will consider this path toward next release.

i'm just not ready to act on merging here and now, even just to have it on dev line.. -- am NOT going to mark it as postponed. leaving status unchanged incase anyone else wants to review..

thanks!

skaught’s picture

nits:

  • \Drupal::request(). -- not using DI.
  • i do believe the block return will still need to return empty array.. may ironically thro some other exception..

  • SKAUGHT committed 3e4469f on 3.0.x-3119474
    Issue #3119474 by jcourtice: All languages show up in dropdown on 404...

  • SKAUGHT committed 3e4469f on 3.0.x
    Issue #3119474 by jcourtice: All languages show up in dropdown on 404...
skaught’s picture

Status: Needs review » Fixed

cleanup of DI.
merged to dev

skaught’s picture

Version: 8.x-2.3 » 3.0.x-dev
skaught’s picture

Assigned: jcourtice » Unassigned

Status: Fixed » Closed (fixed)

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