Bug 419498

Summary:	gwenview is can't load current directory with %U
Product:	[Applications] gwenview	Reporter:	ferweer <gyrlgith>
Component:	general	Assignee:	Gwenview Bugs <gwenview-bugs-null>
Status:	RESOLVED WORKSFORME
Severity:	normal	CC:	nate
Priority:	NOR
Version First Reported In:	19.12.3
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description ferweer 2020-04-01 14:29:55 UTC

SUMMARY
hi, sorry strange english.
%U command is open current directory to open path with application.
but gwenview %U command on terminal get error.
that is say Loading '%U' failed could not open file path.

but strange action.
gwenview is can open load directory.
but first view say error.
if i use cursor key action, can view file in current directory = open directory.

why this is? why first view get error?

STEPS TO REPRODUCE
1. some directory in open terminal
2. open terminal in some directory.
3. command. gwenview %U

OBSERVED RESULT
gwenview can open current directory. but gwenview first view get error.
but you can view current directory file in gwenview with cursor action.

EXPECTED RESULT
gwenview need interaction %U.

Operating System: Kubuntu 19.10
KDE Plasma Version: 5.18.3
KDE Frameworks Version: 5.67.0
Qt Version: 5.12.4
Kernel Version: 5.3.0-45-lowlatency

ADDITIONAL INFORMATION
maintenancer to busy? in opensources.

Comment 1 Nate Graham 2020-04-15 00:49:02 UTC

Are you actually running `gwenview %U`? Or giving it a real path, like `gwenview /home/` ?

Comment 2 ferweer 2020-04-17 11:29:00 UTC

(In reply to Nate Graham from comment #1)
> Are you actually running `gwenview %U`? Or giving it a real path, like
> `gwenview /home/` ?

thanks, i use shell script > .sh
this sh in code hear

#!/bin/bash
gwenview %U&

run this shell script example in ./documents
you can get error. but can see file view.

Comment 3 Nate Graham 2020-04-17 15:17:56 UTC

That's not going to work. :) %U is a special symbol in .desktop files that gets replaced with a space-separated list of URLs. For your purpose, you need to pass the list of URLs yourself, or in the script.

Comment 4 ferweer 2020-05-05 20:05:41 UTC

(In reply to Nate Graham from comment #3)
> That's not going to work. :) %U is a special symbol in .desktop files that
> gets replaced with a space-separated list of URLs. For your purpose, you
> need to pass the list of URLs yourself, or in the script.

thanks. i changed script.
gwenview ./* &