AfipSDK / afip.php

Libreria para usar los Web Services de AFIP
https://afipsdk.com/
MIT License
268 stars 118 forks source link

afip.php/src/Class/ElectronicBilling.php #23

Closed gonzaloalonsod closed 5 years ago

gonzaloalonsod commented 5 years ago

Linea 287: public function FormatDate($date) { return date_format(DateTime::CreateFromFormat('Ymd', $date.''), 'Y-m-d'); }

Pasar de un string a un datetime y devuelta a string, creo que seria mejor devolver el valor original del web service como dice la documentacion: CAEFchVto String(8)

AfipSDK commented 5 years ago

La funcion sirve para quienes necesiten pasar de la fecha que devuelve el web service yyyymmdd a una mas legible yyyy-mm-dd (con guines)

gonzaloalonsod commented 5 years ago

Entiendo pero es muy específica la conversión y como te digo es pasar a objeto y devuelta a string. Yo devolvería como viene y que cada uno haga como mejor le parezca luego

AfipSDK commented 5 years ago

Si lo queres como viene basta con no usar ese metodo

El El lun, 8 oct. 2018 a las 22:35, Gonzalo Alonso notifications@github.com escribió:

Entiendo pero es muy específica la conversión y como te digo es pasar a objeto y devuelta a string. Yo devolvería como viene y que cada uno haga como mejor le parezca luego

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/AfipSDK/afip.php/issues/23#issuecomment-428030667, or mute the thread https://github.com/notifications/unsubscribe-auth/ApZ56FXDNt9GT2r-RUXeJfKJwI_uFQCeks5ui_1hgaJpZM4XNyt8 .

gonzaloalonsod commented 5 years ago

Claro. public function CreateVoucher($data, true) Eso seria pasando la opcion true como segundo parametro?

AfipSDK commented 5 years ago

Este metodo no se usa, es para aquellos que quieran convertir la fecha que viene por defecto en los web services (yyyymmdd) a una con los guiones yyyy-mm-dd, no se usa automáticamente dentro de la libreria, por ningun metodo, si no lo necesitas simplemente no escribis ningun cosigo que lo incluya

El El mar, 9 oct. 2018 a las 08:58, Gonzalo Alonso notifications@github.com escribió:

Closed #23 https://github.com/AfipSDK/afip.php/issues/23.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/AfipSDK/afip.php/issues/23#event-1892603887, or mute the thread https://github.com/notifications/unsubscribe-auth/ApZ56L54e_txB0x6zJ7cfzWyYGJYtwMtks5ujI9ZgaJpZM4XNyt8 .

gonzaloalonsod commented 5 years ago

https://github.com/AfipSDK/afip.php/blob/master/src/Class/ElectronicBilling.php Linea 100 lo utiliza

AfipSDK commented 5 years ago

Tenes razon!! Jajaja (perdon uno se olvida de cosas) Es porque la idea del SDK es facilitar las cosas desde el principio, y una fecha mas legible es mas utilizada que una que tangan que transformar uno mismo si o si

gonzaloalonsod commented 5 years ago

jaja claro bueno quizas utilizo el parametro true para que devuelta todos los valores y lo parseo. Gracias