fix fetch args

[ashnazg]

Issue #33 . This fixes the args for oci_fetch_array. I tried to keep the same $moredata logic, based on (what I think is) original ocifetchinto return behavior of integer of rows fetched.

@schengawegga , I'll need you to test this since I don't have the right environment.

[schengawegga]

Thank @ashnazg for der PR.
I will look at it soon.

[schengawegga]

@ashnazg Thanks for your PR. I pushed a second commit, becuase at the end of the query dataset, oci_fetch_array returns a false value, to end a while loop, that is not acceptable by count(). I will doublecheck all other changed functions, on my last PHP8.2 release, if there any other functions with the same issue than oci_fetch_array. Could you do a review of my comitted change, please?

[till]

Why silence?

[schengawegga]

i think, because of all other PHP oci functions are silent, too. Maybe the DBMS should handle the error messages on its own?
@ashnazg is that right? or what do you thought by silence that function?

There is also another check, if the oci_fetch_array returns an array and not false.
i don´t have a strong opinion on call it silence or not.
@ashnazg @till what do you suggest?

[till]

@schengawegga No, that sounds plausible. Otherwise LGTM!