Some improves

[leonnleite]

PHP 7.1+ dependency

[CarsonF]

Thanks for your work on this @leonnleite. It's been on my list to fix up.
I'll give it a good look over in a week or two.

We are currently implementing HTTPlug in our other libraries currently. Would you mind using that here instead of guzzle directly?

[leonnleite]

Nice, I will do that. ;)

(I need do some unit tests)

so... what you think about php 7.1 dependency?

[CarsonF]

so... what you think about php 7.1 dependency?

👍 I'm all for it.

I just pushed a feature/refactor branch up with a lot of work I had done almost a year ago. Obviously things have changed since then, but feel free to pull concepts/code from it. I liked how the QueryCompiler and BasicClient (other than that name) were shaping up. Results, exceptions, and authentication still needed work.

[pereorga]

Hi,

Sorry for hijacking this thread.

Is there a stable branch of this project? What would be the recommended branch to use?

I see that the master branch has a dependency of a specific version of guzzle (3.9.2), which does not make much sense, if this library is meant to be reusable.

[mikegillis677]

The purpose of this library was to target older versions of PHP (and hence Guzzle) during the time period when people were trying to transition from 5.6 or older to 7.0. That's basically the situation my company found itself in when this library was written.

I believe that there are several Salesforce SDK implementations you can find on https://packagist.org that would work just as well.

If it turns out that there's a market for an improved version of this library that targets PHP 7.1 and Guzzle 6, I'd have to see what I could do.