Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default options not retrieved correctly #1063

Open
metabiswadeep opened this issue Sep 27, 2024 · 2 comments
Open

Default options not retrieved correctly #1063

metabiswadeep opened this issue Sep 27, 2024 · 2 comments
Assignees
Labels
investigating Investigating the issue waiting for reporter There are data requested from the reporter

Comments

@metabiswadeep
Copy link

Describe the bug
While working on fixing default options not being retrieved correctly by CPDB, I found that the CUPS is not returning the values of the default options correctly. It can be seen that default options are retrieved by https://github.com/OpenPrinting/cpdb-backend-cups/blob/master/src/backend_helper.c#L688 and it seems that the wrong default value is retrieved here, atleast for the case of resolution for a dummy printer instance of default resolution 1200dpi. It goes on to https://github.com/OpenPrinting/cpdb-backend-cups/blob/master/src/backend_helper.c#L701 which then goes to https://github.com/OpenPrinting/cpdb-backend-cups/blob/master/src/backend_helper.c#L1955 then to https://github.com/OpenPrinting/cpdb-backend-cups/blob/master/src/backend_helper.c#L1967 which gives 300 as the output though it should give 1200. It seems that the CUPS calls being made here are not returning the correct default values.

Expected behavior
The correct default value should get returned from CUPS.

System Information:

  • Ubuntu 23.10
  • CUPS version 2.4.6
@michaelrsweet michaelrsweet self-assigned this Oct 9, 2024
@michaelrsweet
Copy link
Member

Have you tested with the most recent version of CUPS? 2.4.6 is getting a little old and there have been several bug fixes WRT defaults since then...

@michaelrsweet michaelrsweet added investigating Investigating the issue waiting for reporter There are data requested from the reporter labels Oct 9, 2024
@metabiswadeep
Copy link
Author

Hey, sorry for the late reply. Yes I tried on the master branch and it still does not return the correct default options.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigating Investigating the issue waiting for reporter There are data requested from the reporter
Projects
None yet
Development

No branches or pull requests

2 participants