culqi-php icon indicating copy to clipboard operation
culqi-php copied to clipboard

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

Open gtux opened this issue 9 years ago • 6 comments

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 <[email protected]>
 * @version Dec 14, 2016
 * @copyright GT Tech E.I.R.L. <http://www.gttech.pe> (cambiarlo por su marca claro)
 */
class Culqi 
{
....
}

gtux avatar Dec 14 '16 23:12 gtux

@gtux te refieres a esto https://www.phpdoc.org/ https://www.phpdoc.org/templates

marti1125 avatar Dec 17 '16 04:12 marti1125

@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.

gtux avatar Dec 17 '16 14:12 gtux

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

marti1125 avatar Dec 20 '16 03:12 marti1125

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

marti1125 avatar Feb 06 '17 15:02 marti1125

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

gtux avatar Feb 06 '17 16:02 gtux

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

marti1125 avatar Feb 24 '17 20:02 marti1125