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 an attachment (id=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.