| Subject: | POD for write_url() should switch order of arguments $string and $format |
Currently, the documentation for write_url() lists the arguments as
"write_url($row, $col, $url, $format, $string, $tip)". This would seem
to imply that if I wanted to give a label for the link ($string), but
not a format ($format), I could do "write_url($row, $col, $url, undef,
$string)". But that does not work. Although there is a provision in the
code that allows the two arguments to be switched, that only works if
the $format is defined.
Instead, the documentation should be changed to be consistent with the
write_url() function in Spreadsheet::WriteExcel, "write_url($row, $col,
$url, $label, $format)", which correctly indicates that the label should
be given *before* the format. This works whether or not a $format is given.
Indeed, when you look at the source code of
Spreadsheet::WriteExcelXML::Worksheet, the argument parsing initially
parses argument 3 as $string and argument 4 as $format; only swapping
them if argument 3 is a Format object (which won't work for undef).