Fix array expectation via cast.

[dereuromark]

Reverts #382

as I explained already to @ADmad in the comment to the original and valid change to add the typehint, the internal code expects the array, thus this revert here was invalid and a bad idea.
BC has nothing to do with this :)

See if (in_array($fieldName, $primaryKey)) { usage.

[codecov-io]

Codecov Report

Merging #383 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #383   +/-   ##
=========================================
  Coverage     92.19%   92.19%           
  Complexity      672      672           
=========================================
  Files            25       25           
  Lines          2205     2205           
=========================================
  Hits           2033     2033           
  Misses          172      172

Impacted Files	Coverage Δ	Complexity Δ
src/Shell/Task/ModelTask.php	95.02% <100%> (ø)	156 <0> (ø)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6893ffe...d6750b2. Read the comment docs.

[ADmad]

And I already explained to you why maintaining the same signature is necessary to avoid PHP from complaining about signature mismatch, in an existing userland class which extends ModelTask and has overridden this method.

Hopefully @markstory or someone else can explain the same to you again.

[dereuromark]

I think you are a bit pedantic here on the bc issue which is highly theoretical.
The actual issue remains..

[dereuromark]

I adjusted to code to ahere to the BC issue (which is really not that big of deal IMO on that require-dev plugin).
But I would prefer if this was fixed properly instead of just reverting code that was put there for a reason :)