Skip to content

docs: Port "Plugin config helper" service page to 7.2#581

Open
promptless-for-oss wants to merge 1 commit into
mautic:7.2from
Promptless:promptless/docs-issue-375-plugin-config-helper-7.2
Open

docs: Port "Plugin config helper" service page to 7.2#581
promptless-for-oss wants to merge 1 commit into
mautic:7.2from
Promptless:promptless/docs-issue-375-plugin-config-helper-7.2

Conversation

@promptless-for-oss

Copy link
Copy Markdown
Contributor

Open this suggestion in Promptless to view citations and reasoning process

Ports the legacy Plugin config helper guide into docs/plugin_services/plugin_config_helper.rst, converting the Markdown source to RST. Documents injecting BundleHelper and using getBundleConfig() to read a Plugin's Config/config.php array. Content sourced from 7.1 porting issue #375 (parent #374); targets base branch 7.2.

Trigger Events


Tip: Attach PDFs in Slack messages to Promptless—it can even extract images from them 📎

Ports the legacy Plugin config helper guide into
docs/plugin_services/plugin_config_helper.rst, converting the Markdown
source to RST and documenting BundleHelper injection and getBundleConfig().
Resolves docs issue mautic#375.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

(Line 41)

Verified the getBundleConfig($bundleName, $configKey = '', $includePlugins = false) signature and behavior against current BundleHelper.php on the 7.x branch: it returns the full bundle config when $configKey is omitted and throws when the bundle or key doesn't exist. Also confirmed getMauticBundles() and getPluginBundles() exist, updating the 10-year-old legacy $this->get('mautic.helper.bundle') snippet to modern dependency injection.

Source: https://github.com/mautic/mautic/blob/7.x/app/bundles/CoreBundle/Helper/BundleHelper.php

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant