[PR #265] [MERGED] PayflexCPV4 refactoring and fixes #278

New issue

Closed

opened 2026-03-03 15:44:17 +03:00 by kerem · 0 comments

kerem commented 2026-03-03 15:44:17 +03:00

Owner

Copy link

📋 Pull Request Information

Original PR: https://github.com/mewebstudio/pos/pull/265
Author: @nuryagdym
Created: 2/18/2025
Status: ✅ Merged
Merged: 2/18/2025
Merged by: @nuryagdym

Base: master ← Head: payflexcpv4-refactoring

---

📝 Commits (6)

• f439d6f PayFlexCPV4Pos - fix response decode exception
• 02a250f PayFlexCPV4Pos - remove unsupported transactions
• 0c4ae6f PayFlexCPV4Pos - fix 3d payment status request
• ff3e9fe PayFlexCPV4Pos - change gateway endpoints config
• d275b15 PayFlexCPV4Pos - use http_build_query instead of form_params
• 9bbc5df PayFlexCPV4Pos - coverage tests

📊 Changes

16 files changed (+422 additions, -559 deletions)

View changed files

📝 README.md (+1 -1)
📝 config/pos_production.php (+2 -3)
📝 config/pos_test.php (+2 -3)
➖ examples/payflex-cp-v4/regular/cancel.php (+0 -3)
➖ examples/payflex-cp-v4/regular/form.php (+0 -3)
➖ examples/payflex-cp-v4/regular/post-auth.php (+0 -3)
➖ examples/payflex-cp-v4/regular/refund.php (+0 -3)
📝 phpstan-baseline.neon (+12 -6)
📝 src/DataMapper/RequestDataMapper/PayFlexCPV4PosRequestDataMapper.php (+11 -106)
📝 src/DataMapper/ResponseDataMapper/PayFlexCPV4PosResponseDataMapper.php (+10 -68)
📝 src/Gateways/PayFlexCPV4Pos.php (+128 -61)
📝 src/Serializer/PayFlexCPV4PosSerializer.php (+3 -28)
📝 tests/Unit/DataMapper/RequestDataMapper/PayFlexCPV4PosRequestDataMapperTest.php (+86 -11)
📝 tests/Unit/DataMapper/ResponseDataMapper/PayFlexCPV4PosResponseDataMapperTest.php (+63 -0)
📝 tests/Unit/Gateways/PayFlexCPV4PosTest.php (+101 -199)
📝 tests/Unit/Serializer/PayFlexCPV4PosSerializerTest.php (+3 -61)

📄 Description

• bug fix: Banka API'i Accept header'i gondermeyice JSON response donmeye baslamis, bu da decode ederken exection'a sebep oluyordu. (issue #264)
• bug fix: isteklerde MerchantId yerine HostMerchantId gonderilecek sekilde guncellendi. (issue #264)
• breaking change: configdeki gateway_endpoints URL'ler tekrardan duzenlendi. Bu gateway henuz 100% calismadigindan bu breaking change sorun cikarmamasi lazim.
• clean up: PayFlexV4'den kopyalarken bazi PayflexCPV4 tarafindan deskteklenmeyen islemler vardi. Bu eski kodlar kaldirildi.
• refactoring: istek gonderirken form_params kullanmak yerine serializerda http_build_query kullanilacak sekilde degistirildi.
• tests: test eksigi olan yerlere yeni testler eklendi

---

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/mewebstudio/pos/pull/265 **Author:** [@nuryagdym](https://github.com/nuryagdym) **Created:** 2/18/2025 **Status:** ✅ Merged **Merged:** 2/18/2025 **Merged by:** [@nuryagdym](https://github.com/nuryagdym) **Base:** `master` ← **Head:** `payflexcpv4-refactoring` --- ### 📝 Commits (6) - [`f439d6f`](https://github.com/mewebstudio/pos/commit/f439d6f78a4d94b23cd81dbabac9ec6ac1db8c92) PayFlexCPV4Pos - fix response decode exception - [`02a250f`](https://github.com/mewebstudio/pos/commit/02a250f4ccea36b1cba2f95fe74a93e8bf897a52) PayFlexCPV4Pos - remove unsupported transactions - [`0c4ae6f`](https://github.com/mewebstudio/pos/commit/0c4ae6f0fbbcb36d403cbde9be1f38d6dbab9a14) PayFlexCPV4Pos - fix 3d payment status request - [`ff3e9fe`](https://github.com/mewebstudio/pos/commit/ff3e9fec045adfc967f6b2284872be406abab176) PayFlexCPV4Pos - change gateway endpoints config - [`d275b15`](https://github.com/mewebstudio/pos/commit/d275b15b6b211244545e2f72d5a6a53f9f54daac) PayFlexCPV4Pos - use http_build_query instead of form_params - [`9bbc5df`](https://github.com/mewebstudio/pos/commit/9bbc5df81fedff36f1910eb89a0ef85c2b1ba194) PayFlexCPV4Pos - coverage tests ### 📊 Changes **16 files changed** (+422 additions, -559 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+1 -1) 📝 `config/pos_production.php` (+2 -3) 📝 `config/pos_test.php` (+2 -3) ➖ `examples/payflex-cp-v4/regular/cancel.php` (+0 -3) ➖ `examples/payflex-cp-v4/regular/form.php` (+0 -3) ➖ `examples/payflex-cp-v4/regular/post-auth.php` (+0 -3) ➖ `examples/payflex-cp-v4/regular/refund.php` (+0 -3) 📝 `phpstan-baseline.neon` (+12 -6) 📝 `src/DataMapper/RequestDataMapper/PayFlexCPV4PosRequestDataMapper.php` (+11 -106) 📝 `src/DataMapper/ResponseDataMapper/PayFlexCPV4PosResponseDataMapper.php` (+10 -68) 📝 `src/Gateways/PayFlexCPV4Pos.php` (+128 -61) 📝 `src/Serializer/PayFlexCPV4PosSerializer.php` (+3 -28) 📝 `tests/Unit/DataMapper/RequestDataMapper/PayFlexCPV4PosRequestDataMapperTest.php` (+86 -11) 📝 `tests/Unit/DataMapper/ResponseDataMapper/PayFlexCPV4PosResponseDataMapperTest.php` (+63 -0) 📝 `tests/Unit/Gateways/PayFlexCPV4PosTest.php` (+101 -199) 📝 `tests/Unit/Serializer/PayFlexCPV4PosSerializerTest.php` (+3 -61) </details> ### 📄 Description - **bug fix**: Banka API'i `Accept` header'i gondermeyice JSON response donmeye baslamis, bu da decode ederken exection'a sebep oluyordu. (issue #264) - **bug fix**: isteklerde `MerchantId` yerine `HostMerchantId` gonderilecek sekilde guncellendi. (issue #264) - **breaking change**: configdeki `gateway_endpoints` URL'ler tekrardan duzenlendi. Bu gateway henuz 100% calismadigindan bu breaking change sorun cikarmamasi lazim. - **clean up:** PayFlexV4'den kopyalarken bazi PayflexCPV4 tarafindan deskteklenmeyen islemler vardi. Bu eski kodlar kaldirildi. - **refactoring:** istek gonderirken `form_params` kullanmak yerine serializerda `http_build_query` kullanilacak sekilde degistirildi. - **tests:** test eksigi olan yerlere yeni testler eklendi --- _{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

kerem 2026-03-03 15:44:17 +03:00

• closed this issue
• added the
  pull-request
  label

kerem referenced this issue 2026-03-03 15:44:18 +03:00

[PR #278] [MERGED] Issue 274 ziraat katilim payfor support #282

Sign in to join this conversation.

No Branch/Tag specified

Branches Tags

master

v2

v2-introduce-response-object

v1.8.0

issue-289-fix-kuveytpos-soap-calls

fix/phpstan-and-php8.5

v1.6

task/pacgage-upgrade

master-old-pkg

1.8.0

1.7.1

1.7.0

1.6.0

1.5.0

1.4.2

1.4.1

1.4.0

1.3.0

1.2.0

1.1.1

1.1.0

1.0.1

1.0.0

0.16.1

0.16.0

0.15.1

0.15.0

0.14.0

0.13.2

0.13.1

0.13.0

0.12.2

0.12.1

0.12.0

0.11.0

0.10.5

0.10.4

0.10.3

0.10.2

0.10.1

0.10.0

0.9.1

0.9.0

0.8.2

0.8.1

0.8.0

0.7.4

0.7.3

0.7.2

0.7.1

0.7.0

0.6.1

0.6.0

0.5.2

0.5.1

0.5.0

0.4.17

0.4.16

0.4.15

0.4.14

0.4.13

0.4.12

0.4.11

0.4.10

0.4.9

0.4.8

0.4.7

0.4.6

0.4.5

0.4.4

0.4.3

0.4.2

0.4.1

0.4.0

0.3.8

0.3.7

0.3.6

0.3.5

0.3.4

0.3.3

0.3.2

0.3.1

0.2.0

0.1.8

0.3.0

0.1.7

0.1.6

0.1.5

0.1.2

0.1.1

0.1.0

Labels

Clear labels   

PayFlexV4

  

TROY

  

akbankpos

  

bug

  

duplicate

  

enhancement

  

garanti

  

help wanted

  

isbank

  

kuveyt

  

payten

  

posnet

  

pull-request

Mirrored from GitHub Pull Request

  

vakifbank

  

ziraat

No labels

PayFlexV4

TROY

akbankpos

bug

duplicate

enhancement

garanti

help wanted

isbank

kuveyt

payten

posnet

pull-request

vakifbank

ziraat

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/pos#278

No description provided.