The contents of %{...} are expected to be a reference and instead you have a glob.

Two changes I would make to the above code

• Presumably FTP->new is returning an object reference, not a list, so my($this) should probably be simply my $this.
• Assuming $this is an object reference and the object is implemented as a hash, %{*$this} should be simply %{$this} or even just %$this. See perlreftut for further information.

However, it is generally not good programming practice to break the opacity of an object reference. I would look more carefully at the docs for the FTP module and rewrite the code so that the hash assignment is unnecessary.

Best, beth