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

Created on 8 April 2025, 17 days 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.

Merge Requests

!4Drush: modeler_api:model:export should handle case sensitivity of process id
Merged
🇺🇸United States cosmicdreams
updated 17 days 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 17 days 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

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024

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

Problem/Motivation

Merge Requests

!4Drush: modeler_api:model:export should handle case sensitivity of process idMerged

Comments & Activities

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