Skip to content

Strip 'Seed' suffix from seed names in all command output #984

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
markstory merged 3 commits into from
Jan 6, 2026
Merged

Strip 'Seed' suffix from seed names in all command output #984

markstory merged 3 commits into from
Jan 6, 2026
+53 −37

Conversation

@dereuromark
Copy link
Member

@dereuromark dereuromark commented Jan 2, 2026 •
edited
Loading

Summary

  • Strip 'Seed' suffix from seed class names and add ' seed' suffix for clarity
  • Status command now shows Users seed instead of UsersSeed
  • Reset command output now shows Reset: Users seed instead of Reset: UsersSeed
  • Execution output now shows == Users seed: seeding instead of == UsersSeed: seeding

Before:

  pending    ControllersActionsSeed
  executed   UsersSeed              (2026-01-02 19:40:50)

 == UsersSeed: already executed

After:

  pending    ControllersActions seed
  executed   Users seed              (2026-01-02 19:40:50)

 == Users seed: already executed

This makes the output clearer by showing human-readable names with "seed" suffix to indicate what type of entity is being displayed.

@dereuromark
Strip 'Seed' suffix from seed names in all command output
0619791 
Makes seed name display consistent across all commands by removing the
'Seed' suffix. The run command confirmation was already doing this, but
status, reset, and execution output still showed the full class name.
@dereuromark dereuromark requested a review from markstory January 2, 2026 19:49
@dereuromark
Add ' seed' suffix to display names for clarity
2f706f9 
Changes output from "Users" to "Users seed" to make it clearer
that seeds are being displayed, not other entities.
markstory
markstory reviewed Jan 5, 2026
View reviewed changes
@dereuromark
Refactor seed display name logic to utility method
228f248 
Move the seed name suffix stripping logic to Util::getSeedDisplayName()
to avoid code duplication across SeedCommand, SeedResetCommand,
SeedStatusCommand, and Manager.
@dereuromark dereuromark requested a review from markstory January 5, 2026 22:56
@markstory markstory merged commit 1c06cf8 into 5.x Jan 6, 2026
14 checks passed
@markstory markstory deleted the fix-seed-name-display-consistency branch January 6, 2026 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LordSimal LordSimal LordSimal approved these changes

@markstory markstory Awaiting requested review from markstory

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

5.x (CakePHP 5.x, builtin-backend)

Development

Successfully merging this pull request may close these issues.

4 participants

@dereuromark @markstory @LordSimal