search

rediris-es / simplesamlphp-module-idpinstaller

A SimpleSAMLphp module for pre-installing an IdP
GNU Lesser General Public License v2.1
0 stars 0 forks source link

Mantener comentarios en config/config.php y config/authsources.php #4

Open lonoak opened 6 years ago

lonoak commented 6 years ago

Tras ejecutar el instalador de IdP de referencia, los ficheros resultantes config/config.php y config/authsources.php quedan sin comentarios, y sería conveniente que mantuviesen los comentarios de la versión más reciente de los config-templates/config.php y config-templates/authsources.php.

La manera apropiada de hacer esto pienso que sería recorriendo el array de la plantilla mediante el uso de [token_get_all](http://es.php.net/manual/en/function.token-get-all.php), y preservando los nodos de comentarios.

El volcado a fichero se hace por aquí.

adriangmemorandum commented 6 years ago

Acabo de subir los ultimos cambios a la rama de comentarios. Estos son los ficheros generados:

authsources.txt config.txt

Aparte de revisar y corregir el codigo de mi compañero, he mejorado algunas cosas y creado varias funciones en el archivo lib/functions.php, que serán las que se utilizarán desde los distintos puntos en los que se modifica el contenido de estos dos ficheros de configuración.

lonoak commented 6 years ago

Tienen buena pinta...