4975-mdbVersion-last-method #1245

Merged
guillermo merged 12 commits from 4975-mdbVersion-last-method into dev 2023-01-16 11:40:23 +00:00
6 changed files with 125 additions and 16 deletions

View File

@ -252,6 +252,9 @@
"Receipt's bank was not found": "No se encontró el banco del recibo", "Receipt's bank was not found": "No se encontró el banco del recibo",
"This receipt was not compensated": "Este recibo no ha sido compensado", "This receipt was not compensated": "Este recibo no ha sido compensado",
"Client's email was not found": "No se encontró el email del cliente", "Client's email was not found": "No se encontró el email del cliente",
"App name does not exist": "El nombre de aplicación no es válido",
"Try again": "Vuelve a intentarlo",
"Aplicación bloqueada por el usuario 9": "Aplicación bloqueada por el usuario 9",
"Failed to upload file": "Error al subir archivo", "Failed to upload file": "Error al subir archivo",
"The DOCUWARE PDF document does not exists": "The DOCUWARE PDF document does not exists" "The DOCUWARE PDF document does not exists": "The DOCUWARE PDF document does not exists"
} }

View File

@ -0,0 +1,46 @@
const UserError = require('vn-loopback/util/user-error');
module.exports = Self => {
Self.remoteMethodCtx('last', {
Review

Si este método devuelve la última versión, quizás sería mejor nombrarlo last-version?

Si este método devuelve la última versión, quizás sería mejor nombrarlo last-version?
Review

Como está dentro de mdbVersion, ¿ya se intuye que obtienes la última version no?

Sabiendo esto, ¿hago el cambio de nombre de last → last-version?

Como está dentro de **mdbVersion**, ¿ya se intuye que obtienes la última version no? Sabiendo esto, ¿hago el cambio de nombre de **last → last-version**?
Review

Ahora que lo comentas, si tiene sentido.

Ahora que lo comentas, si tiene sentido.
description: 'Gets the latest version of a access file',
Outdated
Review

Cambiar la descripción

Cambiar la descripción
accepts: [
{
arg: 'appName',
type: 'string',
required: true,
description: 'The app name'
}
],
returns: {
type: 'number',
root: true
},
http: {
path: `/:appName/last`,
verb: 'GET'
}
});
Self.last = async(ctx, appName) => {
const models = Self.app.models;
const versions = await models.MdbVersion.find({
where: {app: appName},
fields: ['version']
});
if (!versions.length)
throw new UserError('App name does not exist');
let maxNumber = 0;
for (let mdb of versions) {
if (mdb.version > maxNumber)
maxNumber = mdb.version;
}
let response = {
version: maxNumber
};
return response;
};
};

View File

@ -11,20 +11,22 @@ module.exports = Self => {
type: 'string', type: 'string',
required: true, required: true,
description: 'The app name' description: 'The app name'
}, }, {
{ arg: 'toVersion',
arg: 'newVersion',
type: 'number', type: 'number',
required: true, required: true,
description: `The new version number` description: `The new version number`
}, }, {
{
arg: 'branch', arg: 'branch',
type: 'string', type: 'string',
required: true, required: true,
description: `The branch name` description: `The branch name`
}, }, {
{ arg: 'fromVersion',
type: 'string',
required: true,
description: `The old version number`
}, {
arg: 'unlock', arg: 'unlock',
type: 'boolean', type: 'boolean',
required: false, required: false,
@ -41,16 +43,13 @@ module.exports = Self => {
} }
}); });
Self.upload = async(ctx, appName, newVersion, branch, unlock, options) => { Self.upload = async(ctx, appName, toVersion, branch, fromVersion, unlock, options) => {
const models = Self.app.models; const models = Self.app.models;
const userId = ctx.req.accessToken.userId;
const myOptions = {}; const myOptions = {};
const $t = ctx.req.__; // $translate const $t = ctx.req.__; // $translate
const TempContainer = models.TempContainer; const TempContainer = models.TempContainer;
const AccessContainer = models.AccessContainer; const AccessContainer = models.AccessContainer;
const fileOptions = {}; const fileOptions = {};
let tx; let tx;
if (typeof options == 'object') if (typeof options == 'object')
@ -63,6 +62,7 @@ module.exports = Self => {
let srcFile; let srcFile;
try { try {
const userId = ctx.req.accessToken.userId;
const mdbApp = await models.MdbApp.findById(appName, null, myOptions); const mdbApp = await models.MdbApp.findById(appName, null, myOptions);
if (mdbApp && mdbApp.locked && mdbApp.userFk != userId) { if (mdbApp && mdbApp.locked && mdbApp.userFk != userId) {
@ -71,6 +71,19 @@ module.exports = Self => {
})); }));
} }
const existBranch = await models.MdbBranch.findOne({
where: {name: branch}
}, myOptions);
if (!existBranch)
throw new UserError('Not exist this branch');
let lastMethod = await Self.last(ctx, appName, myOptions);
lastMethod.version++;
if (lastMethod.version != toVersion)
throw new UserError('Try again');
const tempContainer = await TempContainer.container('access'); const tempContainer = await TempContainer.container('access');
const uploaded = await TempContainer.upload(tempContainer.name, ctx.req, ctx.result, fileOptions); const uploaded = await TempContainer.upload(tempContainer.name, ctx.req, ctx.result, fileOptions);
const files = Object.values(uploaded.files).map(file => { const files = Object.values(uploaded.files).map(file => {
@ -83,7 +96,7 @@ module.exports = Self => {
const accessContainer = await AccessContainer.container('.archive'); const accessContainer = await AccessContainer.container('.archive');
const destinationFile = path.join( const destinationFile = path.join(
accessContainer.client.root, accessContainer.name, appName, `${newVersion}.7z`); accessContainer.client.root, accessContainer.name, appName, `${toVersion}.7z`);
if (process.env.NODE_ENV == 'test') if (process.env.NODE_ENV == 'test')
await fs.unlink(srcFile); await fs.unlink(srcFile);
@ -104,7 +117,7 @@ module.exports = Self => {
await fs.mkdir(branchPath, {recursive: true}); await fs.mkdir(branchPath, {recursive: true});
const destinationBranch = path.join(branchPath, `${appName}.7z`); const destinationBranch = path.join(branchPath, `${appName}.7z`);
const destinationRelative = `../../.archive/${appName}/${newVersion}.7z`; const destinationRelative = `../../.archive/${appName}/${toVersion}.7z`;
try { try {
await fs.unlink(destinationBranch); await fs.unlink(destinationBranch);
} catch (e) {} } catch (e) {}
@ -112,7 +125,7 @@ module.exports = Self => {
if (branch == 'master') { if (branch == 'master') {
const destinationRoot = path.join(accessContainer.client.root, `${appName}.7z`); const destinationRoot = path.join(accessContainer.client.root, `${appName}.7z`);
const rootRelative = `./.archive/${appName}/${newVersion}.7z`; const rootRelative = `./.archive/${appName}/${toVersion}.7z`;
try { try {
await fs.unlink(destinationRoot); await fs.unlink(destinationRoot);
} catch (e) {} } catch (e) {}
@ -120,10 +133,18 @@ module.exports = Self => {
} }
} }
await models.MdbVersionTree.create({
app: appName,
version: toVersion,
branchFk: branch,
fromVersion,
userFk: userId
}, myOptions);
await models.MdbVersion.upsert({ await models.MdbVersion.upsert({
app: appName, app: appName,
branchFk: branch, branchFk: branch,
version: newVersion version: toVersion
}, myOptions); }, myOptions);
if (unlock) await models.MdbApp.unlock(ctx, appName, myOptions); if (unlock) await models.MdbApp.unlock(ctx, appName, myOptions);
@ -133,7 +154,7 @@ module.exports = Self => {
if (tx) await tx.rollback(); if (tx) await tx.rollback();
if (fs.existsSync(srcFile)) if (fs.existsSync(srcFile))
await fs.unlink(srcFile); fs.unlink(srcFile);
throw e; throw e;
} }

View File

@ -8,6 +8,9 @@
"MdbVersion": { "MdbVersion": {
"dataSource": "vn" "dataSource": "vn"
}, },
"MdbVersionTree": {
"dataSource": "vn"
},
"AccessContainer": { "AccessContainer": {
"dataSource": "accessStorage" "dataSource": "accessStorage"
} }

View File

@ -1,3 +1,4 @@
module.exports = Self => { module.exports = Self => {
require('../methods/mdbVersion/upload')(Self); require('../methods/mdbVersion/upload')(Self);
require('../methods/mdbVersion/last')(Self);
}; };

View File

@ -0,0 +1,35 @@
{
"name": "MdbVersionTree",
"base": "VnModel",
"options": {
"mysql": {
"table": "mdbVersionTree"
}
},
"properties": {
"app": {
"type": "string",
"description": "The app name",
"id": true
},
"version": {
"type": "number"
},
"branchFk": {
"type": "string"
},
"fromVersion": {
"type": "number"
},
"userFk": {
"type": "number"
}
},
"relations": {
"branch": {
"type": "belongsTo",
"model": "MdbBranch",
"foreignKey": "branchFk"
}
}
}