Drush: modeler_api:model:export should handle case sensitivity of process id

Open on Drupal.org →

Created on 8 April 2025, about 2 months ago

Problem/Motivation

When testing the drush command i discovered I could copy and paste the process id from the model into the drush command. The only reason why it failed was that it was expecting a lower case machine name. We can handle this for the user by lowercaseing the incoming id.

🐛 Bug report

Status

Active

Version

1.0

Component

Code

Created by

🇺🇸United States cosmicdreams Minneapolis/St. Paul

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Merge Requests

!4Drush: modeler_api:model:export should handle case sensitivity of process id
Merged
🇺🇸United States cosmicdreams
updated about 2 months ago

Comments & Activities

Issue created by @cosmicdreams
Merge request !43517852: Drush: modeler_api:model:export should handle case sensitivity of process id → (Merged) created by cosmicdreams
Comment about 2 months ago →
🇩🇪Germany jurgenhaas Gottmadingen
Good point. I've got 2 suggestions:

Let's use mb_strtolower instead of strtolower

Let's apply the same logic to other arguments in the Drush commands as well

Comment 28 days ago →

jurgenhaas → committed f0443e41 on 1.0.x

Issue #3517852 by cosmicdreams, jurgenhaas: Drush: modeler_api:model:...

Comment 28 days ago →
🇩🇪Germany jurgenhaas Gottmadingen
Just realized that the MR was set to automatic merge, which is not how we want to do things. So, the change from the MR went in without further review.

I've now added the suggestions from #3 and committed those changes directly.
Comment 14 days ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024