Skip to content

Improve rename script with Sylius mode and bug fixes#482

Merged
TheMilek merged 2 commits into
Sylius:2.1from
Rafikooo:improve-rename-script
Jan 9, 2026
Merged

Improve rename script with Sylius mode and bug fixes#482
TheMilek merged 2 commits into
Sylius:2.1from
Rafikooo:improve-rename-script

Conversation

@Rafikooo

@Rafikooo Rafikooo commented Jan 9, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Add --sylius flag for Sylius official plugin naming convention (Sylius\{Name}Plugin instead of Acme\Sylius{Name}Plugin)
  • Add --auto-detect flag to detect plugin name from directory name
  • Fix config_key bug: was including _plugin suffix, now correctly generates acme_search instead of acme_search_plugin

Usage examples

Community plugin (default):

php bin/rename-plugin.php --company=Acme --feature=Search
# Result: Acme\SearchPlugin\AcmeSearchPlugin, config: acme_search

Sylius official plugin:

php bin/rename-plugin.php --sylius --feature=Adyen
# Result: Sylius\AdyenPlugin\SyliusAdyenPlugin, config: sylius_adyen

Auto-detect from directory:

# In directory named "MailerLitePlugin":
php bin/rename-plugin.php --sylius --auto-detect
# Detects: feature=MailerLite

- Add --sylius flag for Sylius official plugin naming convention
  (Sylius\{Name}Plugin instead of Acme\Sylius{Name}Plugin)
- Add --auto-detect flag to detect plugin name from directory name
- Fix config_key bug: was including "_plugin" suffix, now correctly
  generates "acme_search" instead of "acme_search_plugin"
- COMPANY, SYLIUS=1, DESCRIPTION, FEATURE env vars for CI/automation
- Auto-detect feature name from directory (e.g., SearchPlugin → Search)
- Remove --auto-detect flag (now default behavior)
@Rafikooo
Rafikooo force-pushed the improve-rename-script branch from d109a95 to ccbdcd6 Compare January 9, 2026 13:41
@TheMilek
TheMilek merged commit 2c9d141 into Sylius:2.1 Jan 9, 2026
1 check passed
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.

2 participants