Skip to content

CMS-1384: cleanup service.php, add byCoordinate action - #107

Open
leenwood wants to merge 2 commits into
mainfrom
CMS-1384
Open

CMS-1384: cleanup service.php, add byCoordinate action#107
leenwood wants to merge 2 commits into
mainfrom
CMS-1384

Conversation

@leenwood

Copy link
Copy Markdown
Member
  • add byCoordinate delivery-points lookup
  • bump widget version to 3.5.0
  • replace custom http_response_code() polyfill with native function
  • remove unreachable debug-mode branches

- add byCoordinate delivery-points lookup
- bump widget version to 3.5.0
- replace custom http_response_code() polyfill with native function
- remove unreachable debug-mode branches
@leenwood leenwood self-assigned this Aug 26, 2026
- propagate real upstream HTTP status code from CDEK API to frontend response (was hardcoded to 200 regardless of upstream errors)
- fix dead curl error check (was testing $result instead of $response, substr(false, ...) never equals false)
- remove exit() in default branch of http_response_code() that silently fell back to 200 OK and broke JSON output for unmapped status codes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant