Feature/0.0.2.6#19
Conversation
…petición de manera repetida.
…n (lesser key better).
JuanGamezSW
requested review from
a team,
BalderramaJFP,
alxcodes,
flavioochoa,
helttonrl,
hermesjimenez,
lupitalvarado and
rbarusta
…ctionName+"Params"). Add documentation for validation services. Fix some documentation errors.
|
|
||
| #### En caso de no usar composer #### | ||
| Se puede hacer uso de las clases mediante la implementacion manual haciendo uso del archivo SWSDK.php en lugar del archivo vendor.php | ||
| Se puede hacer uso de las clases mediante la implementan manual haciendo uso del archivo SWSDK.php en lugar del archivo vendor.php |
There was a problem hiding this comment.
Esta mal redactado, debería ser implementación no se por que cambiaste esta palabra XD
| return new stampServiceCached($params); | ||
| return new StampService($params); | ||
| } | ||
| return new StampService($params); |
There was a problem hiding this comment.
Tal vez podria simplificarse asi:
if(isset($params["cached"]) && $params["cached"]){ return new stampServiceCached($params); } return new StampService($params);
| public static function StampV1($xml, $isb64 = false, $ttl = 600){ | ||
| try{ | ||
| $sello = sha1(self::getSignXml($xml)); | ||
| if(apc_add($sello, sha1($xml), $ttl)){ |
There was a problem hiding this comment.
Si no mal recuerdo, me parece que la forma de escribir el if else en php es como js
if(condicion) {
//codigo
} else {
//codigo
}
| } | ||
|
|
||
| public static function StampVersion2V1($xml, $isb64 = false, $ttl = 600){ | ||
| try{ |
There was a problem hiding this comment.
Tal vez seria bueno ser un poco mas especifico con el nombre del método, desde mi punto de vista StampVersion2V1 no me dice mucho
|
|
||
| public function __construct($params) { | ||
| parent::__construct($params); | ||
| $c = count($params); |
There was a problem hiding this comment.
normalmente las buenas practicas te indican que utilizes variables mas descriptivas para que a simple vista sepas de que trata $c no me indica que es eso
| @@ -31,11 +30,11 @@ Dar click en **Download** | |||
| * Paso 3: | |||
| Dar clic en **Composer-Setup.exe** esto abrira una ventana en su explorador para que guarde el archivo composer | |||
| composer install | ||
| ``` | ||
| De esta manera descarga las dependencias que antes escribimos dentro del require que en nuestro caso es el **SDK** | ||
| De esta manera descarga las dependencias que antes escribimos dentro del requiere que en nuestro caso es el **SDK** |
There was a problem hiding this comment.
aquí creo si es "require" porque hablas de las dependencias podrías ponerlo entre comillas tal vez para que se entienda
4 participants
New: Clase para timbrado "cached" usando la librería APCU en el servidor del cliente. Para los clientes que no lo usan no afecta, ya que el objecto "cached" se activa a través del constructor de la clase Stamp
Fix: date interval añadido en la clase "Services" para token infinito.