freshworkstudio / transbank-web-services

DEPREACDO: Transbank Web Services SDK. Webpay OneClick, PatPass y Transacción Normal unsando SOAP
MIT License
106 stars 44 forks source link

Classmap must be protected #45

Closed techguydev closed 5 years ago

techguydev commented 5 years ago

Tengo este error:

`PHP Fatal error: Access level to Freshwork\Transbank\WebpayCaptureNullify\WebpayCaptureNullifyWebService::$classmap must be protected (as in class Freshwork\Transbank\TransbankWebService) or weaker in /Users/lmatab/Sites/compraengrupo/vendor/freshwork/transbank/src/WebpayCaptureNullify/WebpayCaptureNullifyWebService.php on line 20

In WebpayCaptureNullifyWebService.php line 20:

Access level to Freshwork\Transbank\WebpayCaptureNullify\WebpayCaptureNullifyWebService::$classmap must be protected (as in class Fr
eshwork\Transbank\TransbankWebService) or weaker `

techguydev commented 5 years ago

Solucionado:

En WebpayCaptureNullifyWebService.php debemos cambiar $classmap de la clase a protected:

Asi:

protected static $classmap = array( 'nullify' => Nullify::class, 'nullificationInput' => NullificationInput::class, 'baseBean' => BaseBean::class, 'nullifyResponse' => NullifyResponse::class, 'nullificationOutput' => NullificationOutput::class, 'capture' => Capture::class, 'captureInput' => CaptureInput::class, 'captureResponse' => CaptureResponse::class, 'captureOutput' => CaptureOutput::class );

gdespirito commented 5 years ago

Gracias. Acabo de subir el fix con el tag 1.1.6

techguydev commented 5 years ago

Gracias @gonzunigad , para todos aquellos que quieran tener exactamente la version que Gonzalo actualizó pueden hacerlo con el siguiente comando:

composer require freshwork/transbank:dev-master#59b47d0662f380b071d413519dda0511ca8792f4

gdespirito commented 5 years ago

No creo que sea necesario. Solo debes hacer composer update y se debería descargar la versión 1.1.6, que contiene el fix.

techguydev commented 5 years ago

No se porque pero con el composer update no me trajo la version actualizada, por eso comparti el comando directo al commit.

On Sun, Aug 11, 2019, 00:12 Gonzalo De Spirito notifications@github.com wrote:

No creo que sea necesario. Solo debes hacer composer update y se debería descargar la versión 1.1.6, que contiene el fix.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/freshworkstudio/transbank-web-services/issues/45?email_source=notifications&email_token=ADBBFGXKXHXVIAI3L2UXNADQD6GTVA5CNFSM4IKV6LI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4AZU7A#issuecomment-520198780, or mute the thread https://github.com/notifications/unsubscribe-auth/ADBBFGVCT7NNE6TGRIOIL33QD6GTVANCNFSM4IKV6LIQ .