problem with constructor

[eluser]Unknown[/eluser]
Learn? I think un-forget would be better ! Basically I'm adding to someone elses code and was rushing (too much?). I'd forgotten all about that - I'll do some revision and redo the code and see if there's any improvement!