STL importer turns underscores into spaces. #88532

New Issue

Will · 2021-05-24T20:10:25+02:00

Will commented

2021-05-24 20:10:25 +02:00

System Information
Operating system: Linux
Graphics card: Intel

Blender Version
Broken: 2.92

Short description of error

The bundled .STL importer turns underscores in the filename into spaces in the object name.

E.G. Some_File.stl becomes bpy.data.objects["Some File"].

Both Blender and common filesystems support both underscores and spaces in the name.

It's not clear to me what tangible benefit could come from this transformation, and it adds an extra step for me to reverse it I import an object and want to re-export it with the same name.

Exact steps for others to reproduce the error

Import an .STL file with an underscore in its name.

**System Information** Operating system: Linux Graphics card: Intel **Blender Version** Broken: 2.92 **Short description of error** The bundled `.STL` importer turns underscores in the filename into spaces in the object name. E.G. `Some_File.stl` becomes `bpy.data.objects["Some File"]`. Both Blender and common filesystems support both underscores and spaces in the name. It's not clear to me what tangible benefit could come from this transformation, and it adds an extra step for me to reverse it I import an object and want to re-export it with the same name. **Exact steps for others to reproduce the error** Import an `.STL` file with an underscore in its name.

Will commented

2021-05-24 20:10:25 +02:00

Added subscriber: @WCN

Philipp Oeser commented

2021-05-25 09:49:43 +02:00

Added subscribers: @MikhailRachinskiy, @lichtwerk

Philipp Oeser commented

2021-05-25 09:49:43 +02:00

Changed status from 'Needs Triage' to: 'Needs Developer To Reproduce'

Philipp Oeser commented

2021-05-25 09:49:43 +02:00

The Addon uses bpy.path.display_name
https://docs.blender.org/api/master/bpy.path.html#bpy.path.display_name

Personally, I find it a bit obscure as well, display_name_from_filepath seems like the better choice.

But in the end, this seems to be working as intended, will try to get a second opinion.

CC @MikhailRachinskiy

The Addon uses `bpy.path.display_name` https://docs.blender.org/api/master/bpy.path.html#bpy.path.display_name Personally, I find it a bit obscure as well, `display_name_from_filepath` seems like the better choice. But in the end, this seems to be working as intended, will try to get a second opinion. CC @MikhailRachinskiy

Mikhail Rachinskiy commented

2021-05-25 17:45:05 +02:00

It does seem unreasonable, I would rather keep file name intact if possible.

@lichtwerk I can handle this issue if your hands full.

It does seem unreasonable, I would rather keep file name intact if possible. @lichtwerk I can handle this issue if your hands full.

Philipp Oeser commented

2021-05-25 18:00:56 +02:00

Changed status from 'Needs Developer To Reproduce' to: 'Confirmed'

Philipp Oeser commented

2021-05-25 18:00:56 +02:00

In #88532#1165959, @MikhailRachinskiy wrote:
@lichtwerk I can handle this issue if your hands full.

Please do, thx!
(we might want to mention this in the release logs since I can imagine there are people out there that have adapted to this behavior)

> In #88532#1165959, @MikhailRachinskiy wrote: > @lichtwerk I can handle this issue if your hands full. Please do, thx! (we might want to mention this in the release logs since I can imagine there are people out there that have adapted to this behavior)

blender-admin commented

2021-05-26 06:13:39 +02:00

This issue was referenced by d55fa2e02a

This issue was referenced by d55fa2e02a195275807a967989d13b99ee1bdc52

Mikhail Rachinskiy self-assigned this 2021-05-26 06:28:06 +02:00

Mikhail Rachinskiy commented

2021-05-26 06:28:06 +02:00

Changed status from 'Confirmed' to: 'Resolved'

Mikhail Rachinskiy closed this issue

2021-05-26 06:28:06 +02:00

Mikhail Rachinskiy commented

2021-05-26 06:28:06 +02:00