Improve views integration

Created on 2 February 2025, 22 days ago

Problem/Motivation

When I opened #3211814: Views integration for authmap table , I didn't know enough about some things, which caused issues (as you know).

I made it better. Unlike that one, this feels 'finished' to me now.

There are no current urgent problems, but in this MR, the "Delete link"

  • does not guess alias names anymore
  • so it can also be added to other views. (I tested it in a User view, and in an extended node view that linked to two different users. Works fine, on the correct user/authmap entry.)
  • and, does not need an invisible "authname" field added to the view anymore (I've deleted it from the included view config).

Also: the URL to the delete screen adds a ?destination=CURRENT-PAGE, so you get redirected to the proper view afterwards. (The return URL is now incorrect if you try to delete an entry through the view-edit screen - but that is also the case for e.g. Core's node-operations links.)

Also: I'm confident that we made the right choice re. not showing the provider name in the default view... but I added some text to the README about it.

Remaining tasks

Review and merge, if you trust that I tested enough.

User interface changes

None (except the return URL is now incorrect if you try to delete an entry through the view-edit screen...)

📌 Task
Status

Active

Version

2.0

Component

Code

Created by

🇳🇱Netherlands roderik Amsterdam,NL / Budapest,HU

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024