search

CiviMRF / CMRF_Abstract_Core

Abstract implementation for the the CMRF Core. Should be included into the concrete module implementation
GNU Affero General Public License v3.0
1 stars 4 forks source link

sequential is not an option but is treated as one. #5

Closed artfulrobot closed 6 years ago

artfulrobot commented 6 years ago

The CiviCRM API's sequential option is wrongly moved into a requests's options sub-array. This is then ignored as that's not the right place for it. sequential should be supplied as part of the main params array, not in the options part.

The result is that results are indexed by their IDs instead of a sequence.

jaapjansma commented 6 years ago

Thanks. I have merged your PR.