Thread Details
  • State Suggested Answer
  • Replies 2 replies
  • Answers 1 answer
  • Subscribers 353 subscribers
  • Views 681 views
  • Users 0 members are here
Forum Actions
Related

upload package failed - "Code attempted dynamically-named method call"

Matthew Rees-George

I've just tried to update a package on Sugar 12.2 and hit this error on upload:

custom/src/cloudtech/ctsync/handler/connection/ConnectionAbstract.php
Code attempted dynamically-named method call on line 141
Code attempted dynamically-named method call on line 162
Code attempted dynamically-named method call on line 183
Code attempted dynamically-named method call on line 217

I tested uploading on sugarondemand versions 12.0.1 and 12.1.0 and they also exhibit this behaviour.
A local version of 12.0.1 does not complain about the dynamic methods.

I was unaware that this was functionality going to be black-listed, and I'm pretty sure that it was working on 12.x until recently.

There doesn't seem to be any mention in sugarclub.sugarcrm.com/.../sugar-12-0-q2-2022-customization-guide or the 12.1 and 12.2 guides.

Was it a security patch?

Parents Reply Children
  • 0 in reply to Rafael Fernandes

    Thanks Rafael. Yes you are correct, dynamic methods were removed a while ago.

    Somehow the scanner was missing the calls in this particular package until recently. I'm very sure about that because we have been loading the package into sugarondemand up until a month ago without issue.
    Perhaps the method of invocation...

    $this->{__FUNCTION__ . 'Impl'}($module)

    ...was missed by the scanner until recently as it did not use a $variable.

    Anyway, it was easy to change to static method invocation.