Skip to content

Conversation

@memurats
Copy link

Problem: Private method OC\L10N\Factory::getL10nFilesForApp() was not accessible from custom app nmctheme, causing a runtime error.

Solution: Changed the method visibility from private to public to allow controlled access from OCA\NMCTheme\L10N\FactoryDecorator.

@memurats memurats self-assigned this May 23, 2025
@memurats memurats added bug Something isn't working custom MagentaCLOUD customisation labels May 23, 2025
@memurats memurats changed the title Update Factory.php Fix visibility error by making getL10nFilesForApp() public to support custom L10N factory in nmctheme May 23, 2025
@sgyuris sgyuris added the build-ready Customization to include into build label May 23, 2025
github-actions bot pushed a commit that referenced this pull request May 23, 2025
… to support custom L10N factory in nmctheme
github-actions bot pushed a commit that referenced this pull request Jun 13, 2025
… to support custom L10N factory in nmctheme
github-actions bot pushed a commit that referenced this pull request Jun 27, 2025
… to support custom L10N factory in nmctheme
github-actions bot pushed a commit that referenced this pull request Jun 30, 2025
… to support custom L10N factory in nmctheme
@memurats memurats changed the title Fix visibility error by making getL10nFilesForApp() public to support custom L10N factory in nmctheme [FIX NC30] Fix visibility error by making getL10nFilesForApp() public to support custom L10N factory in nmctheme Jul 10, 2025
github-actions bot pushed a commit that referenced this pull request Jul 10, 2025
…pp() public to support custom L10N factory in nmctheme
github-actions bot pushed a commit that referenced this pull request Jul 10, 2025
…pp() public to support custom L10N factory in nmctheme
@memurats memurats added stable30 The branch for the PR and removed bug Something isn't working labels Jul 15, 2025
@memurats memurats changed the title [FIX NC30] Fix visibility error by making getL10nFilesForApp() public to support custom L10N factory in nmctheme [FIX NC30] Make getL10nFilesForApp() public to support custom L10N in nmctheme Jul 15, 2025
@memurats memurats changed the title [FIX NC30] Make getL10nFilesForApp() public to support custom L10N in nmctheme [FIX NC30] Make getL10nFilesForApp() method public to support custom L10N in nmctheme Jul 15, 2025
github-actions bot pushed a commit that referenced this pull request Jul 15, 2025
github-actions bot pushed a commit that referenced this pull request Jul 25, 2025
github-actions bot pushed a commit that referenced this pull request Aug 4, 2025
github-actions bot pushed a commit that referenced this pull request Aug 8, 2025
github-actions bot pushed a commit that referenced this pull request Aug 8, 2025
github-actions bot pushed a commit that referenced this pull request Aug 14, 2025
@memurats memurats added the enhancement New feature or request label Aug 25, 2025
github-actions bot pushed a commit that referenced this pull request Aug 25, 2025
github-actions bot pushed a commit that referenced this pull request Sep 12, 2025
github-actions bot pushed a commit that referenced this pull request Sep 18, 2025
@memurats memurats removed the enhancement New feature or request label Sep 29, 2025
github-actions bot pushed a commit that referenced this pull request Sep 29, 2025
github-actions bot pushed a commit that referenced this pull request Sep 29, 2025
@memurats memurats added the enhancement New feature or request label Oct 27, 2025
github-actions bot pushed a commit that referenced this pull request Oct 27, 2025
github-actions bot pushed a commit that referenced this pull request Oct 27, 2025
github-actions bot pushed a commit that referenced this pull request Oct 27, 2025
github-actions bot pushed a commit that referenced this pull request Oct 27, 2025
github-actions bot pushed a commit that referenced this pull request Oct 29, 2025
github-actions bot pushed a commit that referenced this pull request Nov 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-ready Customization to include into build custom MagentaCLOUD customisation enhancement New feature or request stable30 The branch for the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants