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

[NFR] Documentar su clase, hints para los IDE's #16

Open
gtux opened this issue Dec 14, 2016 · 6 comments
Open

[NFR] Documentar su clase, hints para los IDE's #16

gtux opened this issue Dec 14, 2016 · 6 comments
Labels

Comments

@gtux
Copy link

gtux commented Dec 14, 2016

Los que usamos su librería oficial estaríamos agradecidos si ayudan al IDE para que genere automáticamente el descubrimiento de las clases y métodos internos, una forma podría ser agregar lo que el estándar PHPDoc menciona, y así tendríamos:

/**
 * Clase Culqi
 *
 * @property-read Cargos Cargos
 * @property-read Suscripciones Suscripciones
 * @property-read Devoluciones Devoluciones
 * @property-read Planes Planes
 *
 * @author Grover Manuel Campos Ancajima <grover@gttech.pe>
 * @version Dec 14, 2016
 * @copyright GT Tech E.I.R.L. <http://www.gttech.pe> (cambiarlo por su marca claro)
 */
class Culqi 
{
....
}
@marti1125
Copy link
Contributor

@gtux
Copy link
Author

gtux commented Dec 17, 2016

@marti1125 Sí, pero específicamente al @property-read, que es útil sobretodo en clases que tienen propiedades dinámicas, en este caso Culqi no tiene como atributos las clases Cargos o Suscripciones, sino que se establecen en el constructor, por lo que los IDE's no tienen conocimiento de esta variable y menos de su tipo, así que no te lo proponen al momento de escribir el código.

Al documentar la clase con @property-read el IDE asume que de alguna manera existe ese atributo y es de tal tipo así que te lo ofrece en el autocompletado de código.

@marti1125
Copy link
Contributor

@gtux listo lo reviso mañana 👍 algun otro feedback que tuvieras?

@marti1125
Copy link
Contributor

@gtux hola!! aun lo sigues necesitando.... =/ aun el la lib para la version de la nueva api se sigue modificando

@gtux
Copy link
Author

gtux commented Feb 6, 2017

No, no esperé a que hicieran la actualización, yo mismo lo arreglé en mi copia.

@marti1125
Copy link
Contributor

@gtux la version 1.3.* de culqi-php ya tiene docs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants