DateTimePlugin needs rationalisation and slight fix

So I installed DateTimePlugin this morning and noticed the following things:

Firstly, the installer complained about Foswiki::Plugins::DateTimePlugin does not define $Foswiki::Plugins::DateTimePlugin::VERSION My little understanding of "real" perl would lead to to think that the VERSION variable is mis-declared, perhaps using my instead of our as in other plugins...

Secondly, and more frustrating for me: Is {DateTimePlugin}{TimezoneOffset} the "best" way to go to get server time default output from this plugin? (I assume that is what it intends to do.) Surely using the {DisplayTimeValues} exposed through the localisation part of Configure( lib/LocalSite.cfg) would be a better way. It could be used in the Foswiki::Time::formatTime call about line 263 of DateTimeplugin.pm. Its doesn't seem to be a major coding issue - I can almost do it myself!!!... (I guess my question is: does this soundlike the way to do it.)

I only ask becasue I would tire really quickly of using %DATETIME{date="28 Nov 2024 - 08:38"}% all the time.

In any case, my reading of the current code is that {DateTimePlugin}{TimezoneOffset} is broken anyway in that is doesn't actually change anything as the lines at about 175 where $inc is set, should be about 6 lines down after the incHours is actually set based on {DateTimePlugin}{TimezoneOffset}