Version: 1.0 (using KDE 3.3.1, SuSE) Compiler: gcc version 3.3.3 (SuSE Linux) OS: Linux (i686) release 2.6.5-7.155.29-default When using Hylafax as fax system, Kdeprintfax ignores the Paper Size setting under Page Setup. It should use the "-s" option to the sendfax command.
UNCONFIRMED (batch reassigning messed this)
Does anyone care about fixing this simple bug?
The functionality is already in kdeprintfax, but was omitted from the default HylaFax command line. Go to Settings>Configure KdeprintFax... and select the System icon. Edit the Command to use %page_s. That will supply the -s option to HylaFax. Note that kdeprintfax has a limitation in that it recognizes only A4 and Letter sizes, specifically, anything that isn't A4, it treats at letter.
> Edit the Command to use %page_s. Well, but that's not how kdeprintfax should work. Editing Command should only be used when there is a need to supply advanced options that are not available in the kdeprintfax GUI. In this case, there is a Paper Size option on the GUI, so Kdeprintfax should honor the value set there and supply it to the hylafax client.
I agree with joao@fraga.com that %page_s should be part of the default HylaFax command. I am attaching a patch to do just that.
Created attachment 15769 [details] Patch to add %page_s to the default HylaFax command
SVN commit 534411 by tibirna: Attempt to make hylafax take into account the paper size settings. Thanks to Ray Lischner. BUG:108066 M +1 -1 defcmds.h --- branches/KDE/3.5/kdebase/kdeprint/kdeprintfax/defcmds.h #534410:534411 @@ -24,7 +24,7 @@ #include <qstring.h> #define efax_default_cmd "%exe_fax %user_{NAME=\"@@\"} %dev_{DEV=@@} PAGE=%page %from_{FROM=@@} send %res %number %files" -#define hylafax_default_cmd "%exe_sendfax -n %server_h %res %time_a %enterprise_x %comment_c %email_f -d %name_{@@@}%number %files" +#define hylafax_default_cmd "%exe_sendfax -n %server_h %res %page_s %time_a %enterprise_x %comment_c %email_f -d %name_{@@@}%number %files" #define mgetty_default_cmd "%exe_faxspool %user_F %email_f %name_D %time_t %number %files" QString defaultCommand(const QString& cmd);
Closing old Resolved status bug.