Skip to content

Comments

[6.1] {loadmoduleid} include title#46902

Open
brianteeman wants to merge 1 commit intojoomla:6.1-devfrom
brianteeman:loadmodule
Open

[6.1] {loadmoduleid} include title#46902
brianteeman wants to merge 1 commit intojoomla:6.1-devfrom
brianteeman:loadmodule

Conversation

@brianteeman
Copy link
Contributor

@brianteeman brianteeman commented Feb 17, 2026

PARTIAL Pull Request for Issue #46900 .

Summary of Changes

When you use the editor-xtd button to insert a module in an article you get the chance to select the module by title in the modal but as soon as it is inserted into the article it only displays the module id

This is a problem because when you go back to edit the article you cannot easily determine which module is being inserted. The only way to do that is to quit the article go to the module manager and find the module by id number.

This PR changes that to additionally insert the module title. This is a cosmetic change and doesnt impact existing modules that have been inserted.

Testing Instructions

Insert a module into an article and confirm that it is displayed as expected in the front end

Actual result BEFORE applying this Pull Request

{loadmoduleid 99}
where 99 is the id of the module

Expected result AFTER applying this Pull Request

{loadmoduleid 99 - Latest Posts}
where 99 is the id of the module AND Latest Posts is the title of the module

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

When you use the editor-xtd button to insert a module in an article you get the chance to select the module by title in the modal but as soon as it is inserted into the article it only displays the module id

This is a problem because when you go back to edit the article you cannot easily determine which module is being inserted. The only way to do that is to quit the article go to the module manager and find the module by id number.

This PR changes that to additionally insert the module title.
@webnet-assmann
Copy link

I have tested this item ✅ successfully on 746421e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46902.

1 similar comment
@mortezahonar
Copy link

I have tested this item ✅ successfully on 746421e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46902.


$attrs1 = $attrs;
$attrs1 .= ' data-html="{loadmoduleid ' . ((int) $item->id) . '}"';
$attrs1 .= ' data-html="{loadmoduleid ' . ((int) $item->id) . ' - ' . $this->escape($item->title) . '}"';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is better to keep the same "format" as it is for loadmodule:
{loadmodule mod_examplem,Example title}

I mean comma separated.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is for visual display only so i would not like to change it

@Fedik Fedik added the Feature label Feb 17, 2026
@mariantanase
Copy link

I have tested this item ✅ successfully on 746421e

Expected behavior: when a form is inserted, the title of that form also appears. Useful for quickly identifying the inserted module.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46902.

@exlemor
Copy link

exlemor commented Feb 21, 2026

I have tested this item ✅ successfully on 746421e

I was able to test this successfully - fresh Joomla 6.1 ß2 installation.

Thanks @brianteeman


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46902.

@alikon
Copy link
Contributor

alikon commented Feb 21, 2026

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46902.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature PR-6.1-dev RTC This Pull Request is Ready To Commit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants