Closed (fixed)
Project:
Upgrade Status
Version:
8.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
9 Jul 2021 at 12:40 UTC
Updated:
4 Feb 2022 at 11:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
michaellenahan commentedHere is the patch.
Comment #3
michaellenahan commentedComment #4
michaellenahan commentedComment #7
hotwebmatter commentedGreat idea. Patch is simple Form API code, clearly appropriate, and works well. Make it so.
P.S.: Here's the screenshot:

Comment #8
hotwebmatter commentedComment #9
kristen polThis would be helpful for me but I'd also *really* like a link to the project page. If you click on the project name, it gives you the upgrade status details. I'm wondering if this new column should be combined with a link off to the project page to handle this case as well. Thoughts?
Comment #10
gábor hojtsyThere is an abudance of columns and you already need a sizable screen to fit it all :D So I am a bit wary to add more. If we can come up with creative ways to present this differently or some existing data differently so that it fits better that would be great.
Comment #11
kristen polTrue :) What about tacking it onto the end of the first column, e.g.
MEDIA ENTITY DOCUMENT (media_entity_document)?Comment #12
michaellenahan commentedComment #13
hmendes commentedHello, adding a patch with the suggestion from #11 and a screenshot.
Comment #14
hmendes commentedI haven't updated the title and IS to wait to see if this approach is accepted first.
If you want i can change the MR as well.
Comment #15
kristen polThanks for updating the patch. A couple things:
Nitpick: How about combining these into one line? e.g.
Nitpick: To be consistent with other code, it would be good to change the double quotes to single quotes like shown here:
Comment #16
kristen polI think it looks good :)
Comment #17
kristen polActually, I see there is a
$extension->getName()that could be used instead of getting the name yourself, so how about just using:or, alternatively, adding the
$extension->getName()when$extension->info['name']gets set instead.Comment #18
kristen polComment #19
hmendes commentedChanging the patch to get the
$extension->getName()instead of$extension->getPathname()as suggested in #17.Looks cleaner now :)
Comment #20
gábor hojtsyThanks! Re linking to the project page @kristen_pol, the drupal.org issues column already links to a filtered issue queue for the project. The project page itself should be one click from there. I know that is not a direct link but there is only so much space in this table :D
Comment #21
gábor hojtsyComment #23
gábor hojtsyFix race condition problem :D