Ingobernable / kaos155

Programa de analisis de Contratación con la administración pública y relaciones societarias en España.
GNU General Public License v3.0
17 stars 7 forks source link

separar ficheros sql #23

Closed j-lml closed 7 years ago

j-lml commented 7 years ago

propuesta de separar los ficheros sql en 3 partes diferentes: CREATE_DB_SCRAP DELETE_DB_SCRAP CREATE_PROC_SCRAP

No se si afecta a algo de código, aunque entiendo que en principio no tiene que hacerlo

j-lml commented 7 years ago

Voy a intentar hacerlo siguiendo el procedimiento de git que recomiendan en GitHub a ver si cuadra:

guía de estilo

j-lml commented 7 years ago

al poner en el commit # y numero de issue automáticamente quedan relacionados

j-lml commented 7 years ago

Parece que todo correcto. Si al hacer commit se pone en el texto descriptivo almohadilla y número de incidencia, se queda relacionado automáticamente

23

Softman65 commented 7 years ago

me parece regular, mas coñazo de mantenimiento, además no pierdas de vista que la aplicación crea sola la DB con 1 solo papelote.

2017-12-01 17:14 GMT+01:00 j-lml notifications@github.com:

Parece que todo correcto. Si al hacer commit se pone en el texto descriptivo almohadilla y número de incidencia, se queda relacionado automáticamente

23 https://github.com/Ingobernable/kaos155/issues/23

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Ingobernable/kaos155/issues/23#issuecomment-348536581, or mute the thread https://github.com/notifications/unsubscribe-auth/AIBVex_j1oBwK-Lzs90N2PvLfTxcePBsks5s8CXJgaJpZM4QyROl .

Softman65 commented 7 years ago

habría que cambiar la linea 100 de https://github.com/Ingobernable/kaos155/blob/master/App/node_app/sql_common.js

donde dice:
cp.exec('mysql -u' + resp.user + ' -p' + resp.password + ' -h' + resp.host + ' < ' + app.path.normalize('sqlfiles/CREATEDB' + options.Command + '.sql'), (error, stdout, stderr) => {

deberia decir: cp.exec('mysql -u' + resp.user + ' -p' + resp.password + ' -h' + resp.host + ' < ' + app.path.normalize('sqlfiles/CREATEFULL' + options.Command + '.sql'), (error, stdout, stderr) => {