Skip to content

ObjectGroup: Rename ItemGroup to ObjectGroup - #12256

Merged
thojou merged 1 commit into
ILIAS-eLearning:trunkfrom
matheuszych:feature/12/rename_item_group_to_object_group
Oct 28, 2026
Merged

thojou merged 1 commit into
ILIAS-eLearning:trunkfrom
matheuszych:feature/12/rename_item_group_to_object_group

Conversation

@matheuszych

Copy link
Copy Markdown
Contributor

The repository object type itgr is renamed to obgr. Classes, DIC, database tables/columns, language keys, import/export schemas, page XML, and cross-component type strings now use ObjectGroup. Historical itgr imports, goto targets, page XML, and icons remain dual-supported.

/cc @thojou @oliversamoila

@thojou thojou added improvement php Pull requests that update Php code labels Oct 23, 2026
@oliversamoila

Copy link
Copy Markdown
Contributor

To put this into a larger context: this change to the naming convention was requested and approved by the JourFixe as part of the feature ‘Add New Object Action for Item Groups’.

Thank you very much.
Kind regards,
@oliversamoila

@thojou

thojou commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Hi @fneumann, @lscharmer, @alex40724, @smeyer-ilias, @iszmais, @chfsx , @kergomard, @mjansenDatabay, @thibsy, @oliversamoila

Please have a look at the changes in your respective components. Most of it is straightforward renaming of identifiers, variables, classes, and methods. Since the old name "ItemGroup" is relatively generic, we kindly ask you to double-check that we haven't accidentally modified any "ItemGroup" code that is unrelated to the renamed ObjectGroup component and may have slipped through our internal review process.

@fneumann, @lscharmer: Administration
@alex40724: COPage
@alex40724, @smeyer-ilias: Container, Course, Repository
@iszmais, @lscharmer, @chfsx: Dashboard
@smeyer-ilias: Export, Session
@kergomard: ILIASObject
@mjansenDatabay, @thibsy, @chfsx: Init
@thibsy, @oliversamoila: UI

If there are no objections, we assume that we will be able to incorporate this in good time before 'Coding Completed'.
As always, we will take care of follow-ups related to these changes that negatively affect your components, even after the merge.

Best regards,
@thojou

@chfsx

chfsx commented Oct 23, 2026

Copy link
Copy Markdown
Member

Hi @thojou

Dashboard and Init look good from my side, no objections. Thanks!

Best,
@chfsx

@iszmais iszmais left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for the changes inside the dashboard!

LGMT 👍

@fneumann fneumann left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change in Administration is fine, thanks!

@oliversamoila oliversamoila left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Changes in UI looks good.
Thanks!

@thojou

thojou commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

Dear colleagues,

if there are no further objections, I'll merge this PR tomorrow.
As mentioned, I will handle any feedback regarding this change even after the merge.

Kind regards,
@thojou

@kergomard kergomard left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you very much @matheuszych and @thojou for this improvement and sorry I left you waiting, somehow this completely slipped my attention.

The changes in ILIASObject lgtm.

Best,
@kergomard

The repository object type `itgr` is renamed to `obgr`. Classes, DIC,
database tables/columns, language keys, import/export schemas, page XML,
and cross-component type strings now use ObjectGroup. Historical `itgr`
imports, goto targets, page XML, and icons remain dual-supported.
@thojou
thojou force-pushed the feature/12/rename_item_group_to_object_group branch from 590538e to dd09e60 Compare October 28, 2026 14:54
@thojou

thojou commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Hey all,

Thanks everyone for your feedback. I assume we're good to go now.

As announced, I'll go ahead and merge these changes.

Kind regards,
@thojou

@thojou
thojou merged commit baee651 into ILIAS-eLearning:trunk Oct 28, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants