nkhuw
February 13, 2021, 4:19pm
13
Leo:
It's not the quoting that's causing the problem.
The problem there is that they are explicitly asking for the "open" verb, which will run Explorer. (From memory, if we changed what that did in the registry, it'd break double-clicking folders within Explorer, or make them open in Opus, which is undesirable. It'd also make turning off Explorer Replacement a lot more error prone, especially on a system with more than one third-party file manager, since the original verb could get lost in multiple renames.)
They should instead set the second argument to NULL, which makes ShellExecute use the default verb. If Explorer Replacement is turned on, we make "openindopus" the default verb, but that shouldn't be used explicitly either. Just set the verb to NULL and let the system take care of it.
Thanks... this is all a bit beyond my comprehension, but I will add this information and a link to this discussion to that Electron bug report. THanks!
1 Like