"New File" operator loads UI, even if explicitly told to not do so #72169

New Issue

Tristan · 2019-12-04T16:51:27+01:00

Tristan commented

2019-12-04 16:51:27 +01:00

System Information
Operating system: Linux-5.3.0-24-generic-x86_64-with-debian-buster-sid 64 Bits
Graphics card: GeForce GTX 1070 Ti/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 435.21

Blender Version
Broken: version: 2.81 (sub 16), branch: master, commit date: 2019-11-20 14:27, hash: 26bd5ebd42
Worked: (optional)

Short description of error
When executing "bpy.ops.wm.read_homefile" the parameter for "load_ui" is ignored. Extremely annoying for add-ons that need to create completely blank files.

Exact steps for others to reproduce the error
Open any .blend file.
Switch to 'Python Console'-Editor.
Enter the following:
bpy.ops.wm.read_homefile(load_ui=False)
User interface of saved default file will still be loaded. Blender should keep the UI from the previous file and only load non-UI data.

**System Information** Operating system: Linux-5.3.0-24-generic-x86_64-with-debian-buster-sid 64 Bits Graphics card: GeForce GTX 1070 Ti/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 435.21 **Blender Version** Broken: version: 2.81 (sub 16), branch: master, commit date: 2019-11-20 14:27, hash: `26bd5ebd42` Worked: (optional) **Short description of error** When executing "bpy.ops.wm.read_homefile" the parameter for "load_ui" is ignored. Extremely annoying for add-ons that need to create completely blank files. **Exact steps for others to reproduce the error** Open any .blend file. Switch to 'Python Console'-Editor. Enter the following: bpy.ops.wm.read_homefile(load_ui=False) User interface of saved default file will still be loaded. Blender should keep the UI from the previous file and only load non-UI data.

Tristan commented

2019-12-04 16:51:27 +01:00

Added subscriber: @Plyro

Robert Guetzkow commented

2019-12-04 17:17:17 +01:00

Added subscriber: @rjg

Robert Guetzkow commented

2019-12-04 17:17:17 +01:00

This may be intentionally judging by the comment in the source code (wm_homefile_read_exec() in wm_files.c ):

/* This can be used when loading of a start-up file should only change
 * the scene content but keep the blender UI as it is. */
wm_open_init_load_ui(op, true);
SET_FLAG_FROM_TEST(G.fileflags, !RNA_boolean_get(op->ptr, "load_ui"), G_FILE_NO_UI);

This may be intentionally judging by the comment in the source code ([`wm_homefile_read_exec()` in `wm_files.c` ](https://developer.blender.org/diffusion/B/browse/master/source/blender/windowmanager/intern/wm_files.c$1907)): ``` /* This can be used when loading of a start-up file should only change * the scene content but keep the blender UI as it is. */ wm_open_init_load_ui(op, true); SET_FLAG_FROM_TEST(G.fileflags, !RNA_boolean_get(op->ptr, "load_ui"), G_FILE_NO_UI); ```

Robert Guetzkow commented

2019-12-05 10:08:49 +01:00

Added subscriber: @ideasman42

Robert Guetzkow commented

2019-12-05 10:08:49 +01:00

@ideasman42 You've added this in commit blender/blender-staging@dd54b721c6. I assume this is the way you've intended it to work?

Tristan commented

2019-12-05 10:19:54 +01:00

In #72169#823711, @rjg wrote:
This may be intentionally judging by the comment in the source code (wm_homefile_read_exec() in wm_files.c ):
/* This can be used when loading of a start-up file should only change
 * the scene content but keep the blender UI as it is. */
wm_open_init_load_ui(op, true);
SET_FLAG_FROM_TEST(G.fileflags, !RNA_boolean_get(op->ptr, "load_ui"), G_FILE_NO_UI);

I read from this that it should work the way I described it. My description may not have been very clear.

bpy.ops.wm.read_homefile()

does load the homefile UI by default. Setting

load_ui=False

still loads the homefile's UI, although it shouldn't! The issue is that load_ui seems to be treated as True, even when set to False.

> In #72169#823711, @rjg wrote: > This may be intentionally judging by the comment in the source code ([`wm_homefile_read_exec()` in `wm_files.c` ](https://developer.blender.org/diffusion/B/browse/master/source/blender/windowmanager/intern/wm_files.c$1907)): > > > ``` > /* This can be used when loading of a start-up file should only change > * the scene content but keep the blender UI as it is. */ > wm_open_init_load_ui(op, true); > SET_FLAG_FROM_TEST(G.fileflags, !RNA_boolean_get(op->ptr, "load_ui"), G_FILE_NO_UI); > ``` I read from this that it **should** work the way I described it. My description may not have been very clear. ``` bpy.ops.wm.read_homefile() ``` **does** load the homefile UI by default. Setting ``` load_ui=False ``` **still** loads the homefile's UI, although it shouldn't! The issue is that **load_ui** seems to be treated as *True*, even when set to *False*.

blender-admin commented

2019-12-05 14:26:25 +01:00

This issue was referenced by 95ca3f6536

This issue was referenced by 95ca3f6536f1b756a0c87578f8d9d757ba9cd3a4

Campbell Barton commented

2019-12-05 14:45:27 +01:00

Changed status from 'Open' to: 'Resolved'

Campbell Barton closed this issue

2019-12-05 14:45:27 +01:00

Campbell Barton self-assigned this 2019-12-05 14:45:27 +01:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

"New File" operator loads UI, even if explicitly told to not do so #72169