Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cambio de orden en los require #59

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MonicaPArroyo
Copy link

OpenpayApiResourceBase y OpenpayApiError necesitaban estar arriba ya que OpenpayApi y OpenpayApiAuthError las requieren para heredar de ellas.

OpenpayApiResourceBase y OpenpayApiError necesitaban estar arriba ya que OpenpayApi y OpenpayApiAuthErrorlas requieren para heredar de ellas.
@ghost
Copy link

ghost commented Jul 23, 2022

OpenpayApiResourceBase y OpenpayApiError necesitaban estar arriba ya que OpenpayApi y OpenpayApiAuthError las requieren para heredar de ellas.

Monica, pero despues de eso lograste usar las librerias? Siempre me dice que la Clase Openpay no existe. Tuve que buscar en otras implementaciones, pero no son la ultima version.

@MonicaPArroyo
Copy link
Author

OpenpayApiResourceBase y OpenpayApiError necesitaban estar arriba ya que OpenpayApi y OpenpayApiAuthError las requieren para heredar de ellas.

Monica, pero despues de eso lograste usar las librerias? Siempre me dice que la Clase Openpay no existe. Tuve que buscar en otras implementaciones, pero no son la ultima version.

Sí, pude usarlas sin problemas. Ya no me marcó el error de que no existieran

@ghost
Copy link

ghost commented Jul 23, 2022

Monica, a mi no me marca el error inicial, pero cuando voy a usar el objeto Openpay, me dice que la clase no existe (no marca error en la carga), pero es como si no hubiera hecho la referencia del archivo php. Y encontré varios archivos que falta el cierre de la clase "}" y falta el cierre de php "?>".

Gracias! por tu ayuda!

@MonicaPArroyo
Copy link
Author

Monica, a mi no me marca el error inicial, pero cuando voy a usar el objeto Openpay, me dice que la clase no existe (no marca error en la carga), pero es como si no hubiera hecho la referencia del archivo php. Y encontré varios archivos que falta el cierre de la clase "}" y falta el cierre de php "?>".

Gracias! por tu ayuda!

¡Caray!
La falta de ?> no es problema, ya que si el archivo. Ontiene uro PHP, no es necesario marcar que finaliza.
Sobre lo primero, cuando agrego el require, en Visual code se me agregan las siguientes líneas automáticamente según voy creando las instancias de los objetos:
IMG_20220724_002311.jpg
Me llegó a suceder que la línea no se ponía en automático, pero ya con solo ponerla me reconocía la clase. No sé si eso sea lo que te está sucedie do a ti

@ghost
Copy link

ghost commented Jul 24, 2022

Gracias Monica, si creo que es eso!

@ghost
Copy link

ghost commented Jul 24, 2022

Gracias Monica!
Esta nota para quienes el mismo problema (solucion encontrada por MonicaPArroyo ). La parte que falta en la documentacion es que despues de la instruccion
require_once './vendor/autoload.php';
Deben incluir lo siguiente:
use Openpay\Data\Openpay;
use Openpay\Data\OpenpayApiTransactionError;
use Openpay\Data\OpenpayApiRequestError;
use Openpay\Data\OpenpayApiConnectionError;
use Openpay\Data\OpenpayApiAuthError;
use Openpay\Data\OpenpayApiError;

@jose-openpay
Copy link
Contributor

Buen día @MonicaPArroyo

Gracias por tu aportación a este proyecto, checaremos y ejecutaremos la integración de tus cambios a la rama master y subiremos una nueva versión de nuestra librería.

Agradecemos el tiempo y soporte brindado, quedamos pendientes a cualquier duda o comentario adicional.

Saludos, Equipo Openpay !

@MonicaPArroyo
Copy link
Author

¡Perfecto, gracias @jose-openpay!

@Gaara1996
Copy link

Gracias @MonicaPArroyo tenia el mismo problema, tu comentario nos saca de apuros!

@cesardeltadevs
Copy link

Muchas gracias @MonicaPArroyo cambié el orden de los requiere y funciona perfecto.
@jose-openpay hagan el merge para no estar batallando con este tema

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.

4 participants