Force.com-Toolkit-for-PHP icon indicating copy to clipboard operation
Force.com-Toolkit-for-PHP copied to clipboard

Numerous code and doc issues

Open aikar opened this issue 10 years ago • 0 comments

Hello, I finally got around to updating our toolkit version as I got tired of the issues with the PHPDoc's of the old version I was using and wanted to get us up to date.

I see many of the issues have been fixed, but there are still glaring issues with the code that any good IDE identifies immediately!

There are a few methods like partner client sendSingleEmail that use uninitialized variables like so:

$arg->messages = $messages;

$arg was never created... And due to the questionable behavior of PHP, the code likely 'works', but is still wrong.

Many doc blocks also miss $ like so:

  • @param array sObjects Array of sObjects

So to clean up the code, please open the code base in an IDE and fix the issues inspections are alerting on where possible (Sadly the use of relying on SoapClient for Class generation is going to result in many unknown classes)

But overall glad to see the docs mostly right now.

aikar avatar Oct 16 '15 19:10 aikar