[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-13 Thread Massimiliano L
https://bugs.kde.org/show_bug.cgi?id=488097

--- Comment #10 from Massimiliano L  ---
(In reply to Mike from comment #9)
> Yeah, please do.  I see in the log where the crash happened and that code
> has changed, though I'm not really sure if that's the cause.  Either way, I
> am unable to reproduce the crash on 6.0.5, 6.1 or master.  Fedora 40 KDE VM
> is hosed too so I'll have a look in a few days.
> 
> Meantime, any way you can test with 6.1 with Fedora?

I need to wait for Fedora to ship 6.1, it should not take long. As soon as that
happens, I will try to reproduce the bug and follow up with a new report. Thank
you again!

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-12 Thread Mike
https://bugs.kde.org/show_bug.cgi?id=488097

--- Comment #9 from Mike  ---
Yeah, please do.  I see in the log where the crash happened and that code has
changed, though I'm not really sure if that's the cause.  Either way, I am
unable to reproduce the crash on 6.0.5, 6.1 or master.  Fedora 40 KDE VM is
hosed too so I'll have a look in a few days.

Meantime, any way you can test with 6.1 with Fedora?

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-12 Thread Massimiliano L
https://bugs.kde.org/show_bug.cgi?id=488097

--- Comment #8 from Massimiliano L  ---
(In reply to Mike from comment #7)
> Git commit 90f3daa4fb814bbc9222b6b68d6ee3e896da94ae by Mike Noe.
> Committed on 11/06/2024 at 19:42.
> Pushed by noee into branch 'master'.
> 
> kcm/[core,qml]: Cleanup Manual Config/Remote printer search
> 
> qml: Add contextual help for the manual config options.  For printer host
> search, don't populate list with "local" printers.  Also, handle CUPS
> results and errors for remote printer search.
> 
> core: Don't emit remotePrintersLoaded if there is a CUPS error
> FIXED-IN: 6.2
> 
> M  +1-2src/kcm/printermanager.cpp
> M  +20   -13   src/kcm/ui/components/ManualUri.qml
> 
> https://invent.kde.org/plasma/print-manager/-/commit/
> 90f3daa4fb814bbc9222b6b68d6ee3e896da94ae

Thank you for following-up so quickly, this kind of UI improvements is very
much appreciated!

Should I open a new bug for the crashing issues or was this addressed in the
further commits?

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-11 Thread Mike
https://bugs.kde.org/show_bug.cgi?id=488097

Mike  changed:

   What|Removed |Added

   Version Fixed In||6.2
 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED
  Latest Commit||https://invent.kde.org/plas
   ||ma/print-manager/-/commit/9
   ||0f3daa4fb814bbc9222b6b68d6e
   ||e3e896da94ae

--- Comment #7 from Mike  ---
Git commit 90f3daa4fb814bbc9222b6b68d6ee3e896da94ae by Mike Noe.
Committed on 11/06/2024 at 19:42.
Pushed by noee into branch 'master'.

kcm/[core,qml]: Cleanup Manual Config/Remote printer search

qml: Add contextual help for the manual config options.  For printer host
search, don't populate list with "local" printers.  Also, handle CUPS
results and errors for remote printer search.

core: Don't emit remotePrintersLoaded if there is a CUPS error
FIXED-IN: 6.2

M  +1-2src/kcm/printermanager.cpp
M  +20   -13   src/kcm/ui/components/ManualUri.qml

https://invent.kde.org/plasma/print-manager/-/commit/90f3daa4fb814bbc9222b6b68d6ee3e896da94ae

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-11 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=488097

Nate Graham  changed:

   What|Removed |Added

   Keywords||usability
 CC||n...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-11 Thread Massimiliano L
https://bugs.kde.org/show_bug.cgi?id=488097

--- Comment #6 from Massimiliano L  ---
Created attachment 170369
  --> https://bugs.kde.org/attachment.cgi?id=170369&action=edit
Systemsettings crash seen from journalctl

Thank you for the help! Here is a log of the crash, actually two in a sequence
(triggered by the same sequence of operations).

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-06 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=488097

Bug Janitor Service  changed:

   What|Removed |Added

 Status|REPORTED|ASSIGNED
 Ever confirmed|0   |1

--- Comment #5 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/plasma/print-manager/-/merge_requests/180

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-06 Thread Massimiliano L
https://bugs.kde.org/show_bug.cgi?id=488097

--- Comment #4 from Massimiliano L  ---
(In reply to Mike from comment #3)
> What is strange, however, is that you are not forced to authenticate when
> you choose "Add Printer".  Are you running as root or some other elevated
> user?  This could also explain crashes in System Settings.

No, it's a pretty standard Fedora 40 installation. I observe the same behaviour
on my other machine, same OS. I wonder if there is some magic (polkit?) going
on behind the curtains.

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-06 Thread Mike
https://bugs.kde.org/show_bug.cgi?id=488097

--- Comment #3 from Mike  ---
FYI, there are a few UI/UX changes coming in 6.1 regarding the buttons you
mention, but yeah, it is not totally clear about the address, so we'll add some
help text and a message regarding searching results, etc. to help with the UX.

Regarding system-config-printer, yes it provides an alternative GUI, but
printer manager only uses the driver services convenience features (Dbus),
which allow for grouping physical devices and recommending "best" drivers when
configuring a printer.

What is strange, however, is that you are not forced to authenticate when you
choose "Add Printer".  Are you running as root or some other elevated user? 
This could also explain crashes in System Settings.

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-06 Thread Massimiliano L
https://bugs.kde.org/show_bug.cgi?id=488097

--- Comment #2 from Massimiliano L  ---
Thank you for the quick feedback!

(In reply to Mike from comment #1)
> When you select Add Printer, are you prompted to authenticate?

No, I am not prompted for authentication.

> Also, do you
> have the package system-config-printer installed?

I did not, now I have. But it seems to only provide an additional GUI for the
same CUPS backend?

> Can you post the output from the command: `lpinfo -v`  (You may have to
> elevate privileges)

I get the following list:
-
file cups-brf:/
network beh
network ipp
network https
network ipps
network http
direct hp
network socket
network lpd
network smb
direct hpfax
-

With or without sudo (only the order is different).

> Note: The search button on "Manual Config" asks CUPS for printers at the
> host address.  If you know the printer address, you just type it in and
> choose "Continue".  If the address is just a scheme, CUPS returns local
> printers by default, which we should probably ignore as it is bit confusing.

OK, now I better understand how this works. I think part of the confusion
arises from the fact that the "address" for CUPS can either be an individual
networked printer o a CUPS host. The scroll menu with the search result appears
only if something is returned, giving the impression that otherwise the "search
button" is doing nothing. I think right now I can only add printers through
another CUPS server and not directly as network printers, though.

Note: I find a bit confusing to have "select printer" and "continue with
address" above the address field / printer list, instead of below.

> As far as the crashing, this is something unusual.  You can turn on
> debugging for the  printer manager in KDebugSettings.  Search "print" and
> enable Full Debug for kcm and libkcups.  You can than watch the log in a
> terminal with `journalctl --user -f` to help narrow down what is crashing.

I will as soon as I have the time to look again into this! In the meantime (now
that I figured out how this window is supposed to work) I would classify this
as UI/UX improvement rather than a bug.

-- 
You are receiving this mail because:
You are watching all bug changes.

[systemsettings] [Bug 488097] Search button of "Set up a Printer Connection" wizard misbehaves

2024-06-06 Thread Mike
https://bugs.kde.org/show_bug.cgi?id=488097

--- Comment #1 from Mike  ---
When you select Add Printer, are you prompted to authenticate?  Also, do you
have the package system-config-printer installed?

Can you post the output from the command: `lpinfo -v`  (You may have to elevate
privileges)

Note: The search button on "Manual Config" asks CUPS for printers at the host
address.  If you know the printer address, you just type it in and choose
"Continue".  If the address is just a scheme, CUPS returns local printers by
default, which we should probably ignore as it is bit confusing.  

As far as the crashing, this is something unusual.  You can turn on debugging
for the  printer manager in KDebugSettings.  Search "print" and enable Full
Debug for kcm and libkcups.  You can than watch the log in a terminal with
`journalctl --user -f` to help narrow down what is crashing.

-- 
You are receiving this mail because:
You are watching all bug changes.