-
Notifications
You must be signed in to change notification settings - Fork 29
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
base: master
Are you sure you want to change the base?
Conversation
OpenpayApiResourceBase y OpenpayApiError necesitaban estar arriba ya que OpenpayApi y OpenpayApiAuthErrorlas 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 |
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! |
Gracias Monica, si creo que es eso! |
Gracias Monica! |
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 ! |
¡Perfecto, gracias @jose-openpay! |
Gracias @MonicaPArroyo tenia el mismo problema, tu comentario nos saca de apuros! |
Muchas gracias @MonicaPArroyo cambié el orden de los requiere y funciona perfecto. |
OpenpayApiResourceBase
yOpenpayApiError
necesitaban estar arriba ya queOpenpayApi
yOpenpayApiAuthError
las requieren para heredar de ellas.