Problem/Motivation

Right now the TYPE column shows Custom or Contributed but does not show if the extension is a module, theme, or profile. It would be useful to know this, especially when we are working on a site we are not familiar with.

Steps to reproduce

Run report and look at the TYPE column.

Proposed resolution

Tack on the $extension->getType() to the existing type so it looks like:

  • Contributed module
  • Contributed profile
  • Contributed theme
  • Custom module
  • Custom profile
  • Custom theme

Remaining tasks

  1. Create patch
  2. Review and test patch
  3. Commit it :)

User interface changes

The extension type will show up in the report.

API changes

Data model changes

CommentFileSizeAuthor
#6 3258839-6.patch1.4 KBhmendes
#2 3258839-2.patch860 byteshmendes

Comments

Kristen Pol created an issue. See original summary.

hmendes’s picture

Status: Active » Needs review
StatusFileSize
new860 bytes

Sending a patch.

gábor hojtsy’s picture

Status: Needs review » Needs work

Thanks! This does not seem to support translation for the type name? I suspect it would look a bit less orderly to support translation, but it would be worth it IMHO.

hmendes’s picture

How should the code be? From the examples I saw in other modules people were just using
$this->t('Something @label.', ['@label' => $something])

gábor hojtsy’s picture

I think for this case, we would need to t() all the combinations t('Custom module'), etc. probably in an if/else or switch/case.

hmendes’s picture

Status: Needs work » Needs review
StatusFileSize
new1.4 KB

Something like this?
I didn't understand why we need to do this in this case... Can you explain?

gábor hojtsy’s picture

If we don't translate the type names, it would end up with the first parts translated and the second parts untranslated.

  • 64eeb1b committed on 8.x-3.x
    Issue #3258839 by hmendes, Gábor Hojtsy, Kristen Pol: Include extension...
gábor hojtsy’s picture

Status: Needs review » Fixed

Thanks! Committed and will be part of the next release :)

Status: Fixed » Closed (fixed)

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