Prestashop 1.7 release #31

edenriquez · 2017-08-08T19:05:21Z

Prestashop1.6 to 1.7 adaptation:

This adaptation have the same features as https://github.com/conekta/conektaprestashop

Card + MSI
Oxxo
Spei
Webhook validation
Admin order display
Additional to it we refactor the code from conektaprestashop.php and distribute:
SQL Queries into model (Database.php)
redirection of payment method of current buy to Controller (validation.php)
build order parameters into model (Config.php)

remove unusefull methods

* Change structure OXXO and SPEI - PS 1.7 * Add config_es - PS 1.7

model implementation

functions over config.php and correct flow in pay methods

* Change structure OXXO and SPEI - PS 1.7 * Add config_es - PS 1.7 * Add model database

* Change structure OXXO and SPEI - PS 1.7 * Add config_es - PS 1.7 * Add model database * Add template payment return * Add model payment card

Feature/card 1.7

* Tokenize card * Refactor - Prestashop Validation report

* Tokenize card * Refactor - Prestashop Validation report * Correction branch

* Tokenize card * Refactor - Prestashop Validation report * Correction branch * Correction branch final

* Correction blank spaces and totals * Correction discount lines * Correction security validator

webhook validation

webhook + pruebas

* unused variable * translation issues

* fixes for bugs with paymenth methods * anoher fix * Change vars OXXO and SPEI' * Feature/card 1.7 (#21) * unused variable * translation issues * translation issues * msi

* Add message error in card method and fix static methods * monthly installments

* Add message error in card method and fix static methods * Standards validator prestashop

edenriquez · 2017-08-08T19:54:27Z

config.xml

@@ -2,7 +2,7 @@
 <module>
 	<name>conekta_prestashop</name>
 	<displayName><![CDATA[Conekta Prestashop]]></displayName>
-	<version><![CDATA[2.0.3]]></version>
+	<version><![CDATA[1.0.0]]></version>


it is our current version of prestashop 1.7 , was changed from 2.0.3 because this file was taked from prestashop 1.6 repo

edenriquez · 2017-08-08T19:54:59Z

admin_controller.php

@@ -1,139 +0,0 @@
-<?php


unused form/file

edenriquez · 2017-08-08T19:55:33Z

controllers/front/admin_controller.php

@@ -1,139 +0,0 @@
-<?php


unused form/file

edenriquez · 2017-08-08T20:29:02Z

lib/conekta-php/lib/Conekta.php

@@ -26,7 +26,6 @@
 require_once dirname(__FILE__).'/Conekta/Subscription.php';


from here start conekta-php library upgrade into plugin structure, this changes was reviewed in conekta/conekta-php#84

ends at mails/en/conektabanorte.html

RamsesCarbajal · 2017-08-14T15:50:56Z

conekta_prestashop.php

+
+
+        if (empty($url)) {
+            $url = _PS_BASE_URL_ . __PS_BASE_URI__ . "modules/conektaprestashop/webhook/notification.php";


creo que debe de llevar un _ modules/conektaprestashop/webhook/notification.php
conekta_prestashop

fruizg0302 · 2017-08-10T18:09:01Z

lib/conekta-php/lib/Conekta/Address.php

+	var $city    = "";
+	var $state   = "";
+	var $zip     = "";
+	var $country = "";

 	public function __get($property)


Te sobra un tab aquí

Estos tabs se arreglaron en el pull request https://github.com/conekta/conekta-php/pull/90/files solo hace falta hacer un pull de esa rama de la libreria aqui en prestashop

@eduardoconekta enterado

fruizg0302 · 2017-08-14T16:43:45Z

conekta_prestashop.php

+                        );
+                    }
+
+                    $webhook = \Conekta\Webhook::create(array_merge(array(


Esta variable ya no se utiliza más adelante, me parece que la podríamos quitar.

Event signature authentication added

Eduardo Enriquez and others added 30 commits July 19, 2017 13:07

remove unusefull methods

b35766d

Merge pull request #4 from conekta/feature/card-1.7

52299a1

remove unusefull methods

Feature/oxxo spei ps1.7 (#5)

31609de

* Change structure OXXO and SPEI - PS 1.7 * Add config_es - PS 1.7

model implementation

39fad43

Merge pull request #6 from conekta/feature/card-1.7

27b1b33

model implementation

functions over config.php and correct flow in pay methods

5d81404

Merge pull request #7 from conekta/feature/card-1.7

a1c7fd3

functions over config.php and correct flow in pay methods

model implementation

9c15bf4

functions over config.php and correct flow in pay methods

b80f231

Feature/oxxo spei ps1.7 (#8)

28ebc7a

* Change structure OXXO and SPEI - PS 1.7 * Add config_es - PS 1.7 * Add model database

Feature/oxxo spei ps1.7 (#9)

a9f75c0

* Change structure OXXO and SPEI - PS 1.7 * Add config_es - PS 1.7 * Add model database * Add template payment return * Add model payment card

last commit

8c8de7a

updating functions and registers

19e244d

Merge pull request #10 from conekta/feature/card-1.7

c7f456a

Feature/card 1.7

Tokenize card (#11)

c3e0471

Feature/fix validation ps (#12)

0a08ce0

* Tokenize card * Refactor - Prestashop Validation report

Feature/fix validation ps (#13)

653d7c5

* Tokenize card * Refactor - Prestashop Validation report * Correction branch

Feature/fix validation ps (#14)

eb83ecb

* Tokenize card * Refactor - Prestashop Validation report * Correction branch * Correction branch final

Correction blank spaces and totals (#15)

c1af3ae

Feature/prestashop validator (#16)

1ca519f

* Correction blank spaces and totals * Correction discount lines * Correction security validator

webhook validation

9763bcc

Merge pull request #17 from conekta/feature/card-1.7

d1013c1

webhook validation

webhook + pruebas

c63ac2d

Merge pull request #18 from conekta/feature/card-1.7

0a9db4b

webhook + pruebas

Feature/card 1.7 (#19)

588c63f

* unused variable * translation issues

Support (#22)

7227e88

* fixes for bugs with paymenth methods * anoher fix * Change vars OXXO and SPEI' * Feature/card 1.7 (#21) * unused variable * translation issues * translation issues * msi

Add message error in card method and fix static methods (#29)

4eb1b51

Monthlyinstallments (#28)

fcb14eb

* Add message error in card method and fix static methods * monthly installments

array convention on getJumps function

ae56b3b

Feature/ps message fixes (#30)

bea7ccc

* Add message error in card method and fix static methods * Standards validator prestashop

Eduardo Enriquez added 2 commits August 2, 2017 17:07

Plugin librery upgrade + header recognition

e4176f6

header recognition

53ab0f4

edenriquez requested review from julsdelatierra, RamsesCarbajal, andreasantillana and fruizg0302 August 8, 2017 19:52

edenriquez commented Aug 8, 2017

View reviewed changes

admin_controller.php

@@ -1,139 +0,0 @@

<?php

Copy link

Contributor Author

edenriquez Aug 8, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unused form/file

edenriquez commented Aug 8, 2017

View reviewed changes

controllers/front/admin_controller.php

@@ -1,139 +0,0 @@

<?php

Copy link

Contributor Author

edenriquez Aug 8, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unused form/file

edenriquez commented Aug 8, 2017

View reviewed changes

edenriquez removed the request for review from andreasantillana August 8, 2017 20:39

RamsesCarbajal suggested changes Aug 14, 2017

View reviewed changes

fruizg0302 suggested changes Aug 14, 2017

View reviewed changes

Changes requested in webhook var and underscore plugin name

cce2801

RamsesCarbajal approved these changes Aug 15, 2017

View reviewed changes

fruizg0302 approved these changes Aug 15, 2017

View reviewed changes

edenriquez merged commit 8775187 into master Aug 17, 2017

LucianoPSerfe pushed a commit to LucianoPSerfe/conekta_prestashop that referenced this pull request Jan 15, 2021

Merge pull request conekta#31 from conekta/feature/rsa-signature

310891f

Event signature authentication added

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Prestashop 1.7 release #31

Prestashop 1.7 release #31

edenriquez commented Aug 8, 2017 •

edited

Loading

edenriquez Aug 8, 2017

edenriquez Aug 8, 2017

edenriquez Aug 8, 2017

edenriquez Aug 8, 2017 •

edited

Loading

edenriquez Aug 8, 2017

RamsesCarbajal Aug 14, 2017

fruizg0302 Aug 10, 2017

edenriquez Aug 15, 2017

fruizg0302 Aug 15, 2017

fruizg0302 Aug 14, 2017

		@@ -26,7 +26,6 @@
		require_once dirname(__FILE__).'/Conekta/Subscription.php';



		if (empty($url)) {
		$url = _PS_BASE_URL_ . __PS_BASE_URI__ . "modules/conektaprestashop/webhook/notification.php";

Prestashop 1.7 release #31

Prestashop 1.7 release #31

Conversation

edenriquez commented Aug 8, 2017 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

edenriquez Aug 8, 2017 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

edenriquez commented Aug 8, 2017 •

edited

Loading

edenriquez Aug 8, 2017 •

edited

Loading