PHP Weather 2?

[Martin G. <gim...@gi...>]

Hi everybody

As those of you who follow the phpweather-checkins mailing list will
know, I've been adding and deleting files from CVS today. I've also
made a number of small corrections.

My plan is to finally make a release of the code in CVS. I've been
using it on Gimpster.com for some months now without problems.

I'm very happy with most of the code, although I've discovered that
the text output can be improved. The 'problem' is that many of the
strings are put together like this:

  $output = $strings['foo'] . $var_a .
            $strings['bar'] . $var_b .
            $strings['baz'] . $var_c;

instead of this

  $output = sprintf($strings['foobarbaz'], $var_a, $var_b, $var_c);

The advantage of the last method is, that the translator can make
$var_c come before $var_a in the final string. Changing this requires
updates of the translations, but it shouldn't be that hard to do since
it's mostly a matter of moving and combining existing strings.

There's also currently three open bug reports, but they're not too
serious. And if we wait until all bugs are fixed, then we'll never be
able to release anything.

So, please update from CVS and see if you can spot any problems. Mail
back tomorrow with your comments.

-- 
Martin Geisler                       My GnuPG Key: 0xF7F6B57B

See http://gimpster.com/ and http://phpweather.net/ for:
PHP Weather => Shows the current weather on your webpage and
PHP Shell   => A telnet-connection (almost :-) in a PHP page.