From 18bb8a4ea558fc1a722bcd9b18c2a792ee1a63f1 Mon Sep 17 00:00:00 2001
From: alexm <alexm@verdnatura.es>
Date: Tue, 21 May 2024 13:50:45 +0200
Subject: [PATCH 1/4] fix: checking process.env.NODE_ENV

---
 back/methods/chat/sendCheckingPresence.js              | 2 +-
 back/methods/chat/sendQueued.js                        | 4 ++--
 back/methods/dms/deleteTrashFiles.js                   | 2 +-
 back/methods/docuware/upload.js                        | 2 +-
 back/methods/image/scrub.js                            | 3 +--
 back/methods/image/upload.js                           | 2 +-
 back/methods/notification/send.js                      | 2 +-
 loopback/common/methods/application/isProduction.js    | 3 +++
 loopback/common/models/application.js                  | 5 +++++
 modules/account/back/models/ldap-config.js             | 3 ++-
 modules/account/back/models/samba-config.js            | 3 ++-
 modules/client/back/methods/sms/send.js                | 2 +-
 modules/invoiceOut/back/methods/invoiceOut/download.js | 2 +-
 modules/invoiceOut/back/models/invoice-out.js          | 2 +-
 modules/mdb/back/methods/mdbVersion/upload.js          | 2 +-
 15 files changed, 24 insertions(+), 15 deletions(-)
 create mode 100644 loopback/common/methods/application/isProduction.js

diff --git a/back/methods/chat/sendCheckingPresence.js b/back/methods/chat/sendCheckingPresence.js
index 85b66e94b..12cadec04 100644
--- a/back/methods/chat/sendCheckingPresence.js
+++ b/back/methods/chat/sendCheckingPresence.js
@@ -37,7 +37,7 @@ module.exports = Self => {
         if (!recipient)
             throw new Error(`Could not send message "${message}" to worker id ${recipientId} from user ${userId}`);
 
-        if (process.env.NODE_ENV == 'test')
+        if (!Self.app.models.Application.isProduction())
             message = `[Test:Environment to user ${userId}] ` + message;
 
         const chat = await models.Chat.create({
diff --git a/back/methods/chat/sendQueued.js b/back/methods/chat/sendQueued.js
index 9a23af379..1ab4cb6e0 100644
--- a/back/methods/chat/sendQueued.js
+++ b/back/methods/chat/sendQueued.js
@@ -94,7 +94,7 @@ module.exports = Self => {
      * @return {Promise} - The request promise
      */
     Self.sendMessage = async function sendMessage(senderFk, recipient, message) {
-        if (process.env.NODE_ENV !== 'production') {
+        if (!Self.app.models.Application.isProduction()) {
             return new Promise(resolve => {
                 return resolve({
                     statusCode: 200,
@@ -149,7 +149,7 @@ module.exports = Self => {
      * @return {Promise} - The request promise
      */
     Self.getUserStatus = async function getUserStatus(username) {
-        if (process.env.NODE_ENV !== 'production') {
+        if (!Self.app.models.Application.isProduction()) {
             return new Promise(resolve => {
                 return resolve({
                     data: {
diff --git a/back/methods/dms/deleteTrashFiles.js b/back/methods/dms/deleteTrashFiles.js
index 239d654ef..77e33929a 100644
--- a/back/methods/dms/deleteTrashFiles.js
+++ b/back/methods/dms/deleteTrashFiles.js
@@ -22,7 +22,7 @@ module.exports = Self => {
         if (typeof options == 'object')
             Object.assign(myOptions, options);
 
-        if (process.env.NODE_ENV == 'test')
+        if (!Self.app.models.Application.isProduction())
             throw new UserError(`Action not allowed on the test environment`);
 
         const models = Self.app.models;
diff --git a/back/methods/docuware/upload.js b/back/methods/docuware/upload.js
index 27be72295..ae3989def 100644
--- a/back/methods/docuware/upload.js
+++ b/back/methods/docuware/upload.js
@@ -119,7 +119,7 @@ module.exports = Self => {
                 ]
             };
 
-            if (process.env.NODE_ENV != 'production')
+            if (!Self.app.models.Application.isProduction(false))
                 throw new UserError('Action not allowed on the test environment');
 
             // delete old
diff --git a/back/methods/image/scrub.js b/back/methods/image/scrub.js
index 99c6bcbf3..4af2d8d49 100644
--- a/back/methods/image/scrub.js
+++ b/back/methods/image/scrub.js
@@ -43,8 +43,7 @@ module.exports = Self => {
     Self.scrub = async function(collection, remove, limit, dryRun, skipLock) {
         const $ = Self.app.models;
 
-        const env = process.env.NODE_ENV;
-        dryRun = dryRun || (env && env !== 'production');
+        dryRun = dryRun || !Self.app.models.Application.isProduction(false);
 
         const instance = await $.ImageCollection.findOne({
             fields: ['id'],
diff --git a/back/methods/image/upload.js b/back/methods/image/upload.js
index 51da327f6..81d9759e6 100644
--- a/back/methods/image/upload.js
+++ b/back/methods/image/upload.js
@@ -41,7 +41,7 @@ module.exports = Self => {
             if (!hasWriteRole)
                 throw new UserError(`You don't have enough privileges`);
 
-            if (process.env.NODE_ENV == 'test')
+            if (!Self.app.models.Application.isProduction())
                 throw new UserError(`Action not allowed on the test environment`);
 
             // Upload file to temporary path
diff --git a/back/methods/notification/send.js b/back/methods/notification/send.js
index b2748477d..4f8f436da 100644
--- a/back/methods/notification/send.js
+++ b/back/methods/notification/send.js
@@ -70,7 +70,7 @@ module.exports = Self => {
                     const newParams = Object.assign({}, queueParams, sendParams);
                     const email = new Email(queueName, newParams);
 
-                    if (process.env.NODE_ENV != 'test')
+                    if (Self.app.models.Application.isProduction())
                         await email.send();
 
                     await queue.updateAttribute('status', statusSent);
diff --git a/loopback/common/methods/application/isProduction.js b/loopback/common/methods/application/isProduction.js
new file mode 100644
index 000000000..9479b67af
--- /dev/null
+++ b/loopback/common/methods/application/isProduction.js
@@ -0,0 +1,3 @@
+module.exports = (localAsProduction = true) => {
+    if ((!process.env.NODE_ENV && localAsProduction) || process.env.NODE_ENV == 'production') return true;
+};
diff --git a/loopback/common/models/application.js b/loopback/common/models/application.js
index 6bdc2c13a..70db17de2 100644
--- a/loopback/common/models/application.js
+++ b/loopback/common/models/application.js
@@ -1,3 +1,4 @@
+const isProductionFn = require('../methods/application/isProduction');
 
 module.exports = function(Self) {
     require('../methods/application/status')(Self);
@@ -6,4 +7,8 @@ module.exports = function(Self) {
     require('../methods/application/executeProc')(Self);
     require('../methods/application/executeFunc')(Self);
     require('../methods/application/getEnumValues')(Self);
+
+    Self.isProduction = (localAsProduction = true) => {
+        return isProductionFn(localAsProduction);
+    };
 };
diff --git a/modules/account/back/models/ldap-config.js b/modules/account/back/models/ldap-config.js
index 89f0add48..4f5c0218d 100644
--- a/modules/account/back/models/ldap-config.js
+++ b/modules/account/back/models/ldap-config.js
@@ -3,9 +3,10 @@ const app = require('vn-loopback/server/server');
 const ldap = require('../util/ldapjs-extra');
 const crypto = require('crypto');
 const nthash = require('smbhash').nthash;
+const isProduction = require('vn-loopback/common/methods/application/isProduction');
 
 module.exports = Self => {
-    const shouldSync = process.env.NODE_ENV !== 'test';
+    const shouldSync = isProduction();
 
     Self.getLinker = async function() {
         return await Self.findOne({
diff --git a/modules/account/back/models/samba-config.js b/modules/account/back/models/samba-config.js
index 927510a29..aa68af35d 100644
--- a/modules/account/back/models/samba-config.js
+++ b/modules/account/back/models/samba-config.js
@@ -1,6 +1,7 @@
 
 const ldap = require('../util/ldapjs-extra');
 const execFile = require('child_process').execFile;
+const isProduction = require('vn-loopback/common/methods/application/isProduction');
 
 /**
  * Summary of userAccountControl flags:
@@ -12,7 +13,7 @@ const UserAccountControlFlags = {
 };
 
 module.exports = Self => {
-    const shouldSync = process.env.NODE_ENV !== 'test';
+    const shouldSync = isProduction();
 
     Self.getLinker = async function() {
         return await Self.findOne({
diff --git a/modules/client/back/methods/sms/send.js b/modules/client/back/methods/sms/send.js
index 94b2b6c27..269dedacb 100644
--- a/modules/client/back/methods/sms/send.js
+++ b/modules/client/back/methods/sms/send.js
@@ -47,7 +47,7 @@ module.exports = Self => {
 
         let response;
         try {
-            if (process.env.NODE_ENV !== 'production')
+            if (!Self.app.models.Application.isProduction(false))
                 response = {result: [{status: 'ok'}]};
             else {
                 const jsonTest = {
diff --git a/modules/invoiceOut/back/methods/invoiceOut/download.js b/modules/invoiceOut/back/methods/invoiceOut/download.js
index 748e2df17..5a9d5bc51 100644
--- a/modules/invoiceOut/back/methods/invoiceOut/download.js
+++ b/modules/invoiceOut/back/methods/invoiceOut/download.js
@@ -66,7 +66,7 @@ module.exports = Self => {
             console.error(err);
         });
 
-        if (process.env.NODE_ENV == 'test') {
+        if (!Self.app.models.Application.isProduction()) {
             try {
                 await fs.access(file.path);
             } catch (error) {
diff --git a/modules/invoiceOut/back/models/invoice-out.js b/modules/invoiceOut/back/models/invoice-out.js
index e4fcc1a69..166565fe2 100644
--- a/modules/invoiceOut/back/models/invoice-out.js
+++ b/modules/invoiceOut/back/models/invoice-out.js
@@ -59,7 +59,7 @@ module.exports = Self => {
             hasPdf: true
         }, options);
 
-        if (process.env.NODE_ENV !== 'test') {
+        if (Self.app.models.Application.isProduction()) {
             await print.storage.write(buffer, {
                 type: 'invoice',
                 path: pdfFile.path,
diff --git a/modules/mdb/back/methods/mdbVersion/upload.js b/modules/mdb/back/methods/mdbVersion/upload.js
index 58fc46abb..9edaf454f 100644
--- a/modules/mdb/back/methods/mdbVersion/upload.js
+++ b/modules/mdb/back/methods/mdbVersion/upload.js
@@ -111,7 +111,7 @@ module.exports = Self => {
             const destinationFile = path.join(
                 accessContainer.client.root, accessContainer.name, appName, `${toVersion}.7z`);
 
-            if (process.env.NODE_ENV == 'test')
+            if (!Self.app.models.Application.isProduction())
                 await fs.unlink(srcFile);
             else {
                 await fs.move(srcFile, destinationFile, {
-- 
2.40.1


From d8841920665fcf07e3c0eb63a9e5e81f953bebf8 Mon Sep 17 00:00:00 2001
From: alexm <alexm@verdnatura.es>
Date: Tue, 21 May 2024 13:52:50 +0200
Subject: [PATCH 2/4] fix: checking process.env.NODE_ENV

---
 back/methods/chat/sendQueued.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/back/methods/chat/sendQueued.js b/back/methods/chat/sendQueued.js
index 1ab4cb6e0..ee1a82be2 100644
--- a/back/methods/chat/sendQueued.js
+++ b/back/methods/chat/sendQueued.js
@@ -94,7 +94,7 @@ module.exports = Self => {
      * @return {Promise} - The request promise
      */
     Self.sendMessage = async function sendMessage(senderFk, recipient, message) {
-        if (!Self.app.models.Application.isProduction()) {
+        if (!Self.app.models.Application.isProduction(false)) {
             return new Promise(resolve => {
                 return resolve({
                     statusCode: 200,
@@ -149,7 +149,7 @@ module.exports = Self => {
      * @return {Promise} - The request promise
      */
     Self.getUserStatus = async function getUserStatus(username) {
-        if (!Self.app.models.Application.isProduction()) {
+        if (!Self.app.models.Application.isProduction(false)) {
             return new Promise(resolve => {
                 return resolve({
                     data: {
-- 
2.40.1


From 1f2017c2727ce8e653e61124445735ea3436650f Mon Sep 17 00:00:00 2001
From: alexm <alexm@verdnatura.es>
Date: Tue, 21 May 2024 14:03:13 +0200
Subject: [PATCH 3/4] fix: simplify

---
 loopback/common/methods/application/isProduction.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/loopback/common/methods/application/isProduction.js b/loopback/common/methods/application/isProduction.js
index 9479b67af..151afa9cb 100644
--- a/loopback/common/methods/application/isProduction.js
+++ b/loopback/common/methods/application/isProduction.js
@@ -1,3 +1,3 @@
 module.exports = (localAsProduction = true) => {
-    if ((!process.env.NODE_ENV && localAsProduction) || process.env.NODE_ENV == 'production') return true;
+    return (!process.env.NODE_ENV && localAsProduction) || process.env.NODE_ENV == 'production';
 };
-- 
2.40.1


From ff33c926af40723cbb57ed05641deefe8c6c64f6 Mon Sep 17 00:00:00 2001
From: alexm <alexm@verdnatura.es>
Date: Tue, 21 May 2024 15:11:32 +0200
Subject: [PATCH 4/4] fix: move to boot

---
 back/methods/chat/sendCheckingPresence.js                   | 4 +++-
 back/methods/chat/sendQueued.js                             | 6 ++++--
 back/methods/dms/deleteTrashFiles.js                        | 3 ++-
 back/methods/docuware/upload.js                             | 3 ++-
 back/methods/image/scrub.js                                 | 3 ++-
 back/methods/image/upload.js                                | 3 ++-
 back/methods/notification/send.js                           | 3 ++-
 loopback/common/models/application.js                       | 6 ------
 .../methods/application => server/boot}/isProduction.js     | 0
 modules/account/back/models/ldap-config.js                  | 2 +-
 modules/account/back/models/samba-config.js                 | 2 +-
 modules/client/back/methods/sms/send.js                     | 3 ++-
 modules/invoiceOut/back/methods/invoiceOut/download.js      | 3 ++-
 modules/invoiceOut/back/models/invoice-out.js               | 3 ++-
 modules/mdb/back/methods/mdbVersion/upload.js               | 3 ++-
 15 files changed, 27 insertions(+), 20 deletions(-)
 rename loopback/{common/methods/application => server/boot}/isProduction.js (100%)

diff --git a/back/methods/chat/sendCheckingPresence.js b/back/methods/chat/sendCheckingPresence.js
index 12cadec04..7ab5d63fe 100644
--- a/back/methods/chat/sendCheckingPresence.js
+++ b/back/methods/chat/sendCheckingPresence.js
@@ -1,3 +1,5 @@
+const isProduction = require('vn-loopback/server/boot/isProduction');
+
 module.exports = Self => {
     Self.remoteMethodCtx('sendCheckingPresence', {
         description: 'Creates a message in the chat model checking the user status',
@@ -37,7 +39,7 @@ module.exports = Self => {
         if (!recipient)
             throw new Error(`Could not send message "${message}" to worker id ${recipientId} from user ${userId}`);
 
-        if (!Self.app.models.Application.isProduction())
+        if (!isProduction())
             message = `[Test:Environment to user ${userId}] ` + message;
 
         const chat = await models.Chat.create({
diff --git a/back/methods/chat/sendQueued.js b/back/methods/chat/sendQueued.js
index ee1a82be2..abda2ddc1 100644
--- a/back/methods/chat/sendQueued.js
+++ b/back/methods/chat/sendQueued.js
@@ -1,4 +1,6 @@
 const axios = require('axios');
+const isProduction = require('vn-loopback/server/boot/isProduction');
+
 module.exports = Self => {
     Self.remoteMethodCtx('sendQueued', {
         description: 'Send a RocketChat message',
@@ -94,7 +96,7 @@ module.exports = Self => {
      * @return {Promise} - The request promise
      */
     Self.sendMessage = async function sendMessage(senderFk, recipient, message) {
-        if (!Self.app.models.Application.isProduction(false)) {
+        if (!isProduction(false)) {
             return new Promise(resolve => {
                 return resolve({
                     statusCode: 200,
@@ -149,7 +151,7 @@ module.exports = Self => {
      * @return {Promise} - The request promise
      */
     Self.getUserStatus = async function getUserStatus(username) {
-        if (!Self.app.models.Application.isProduction(false)) {
+        if (!isProduction(false)) {
             return new Promise(resolve => {
                 return resolve({
                     data: {
diff --git a/back/methods/dms/deleteTrashFiles.js b/back/methods/dms/deleteTrashFiles.js
index 77e33929a..e07f93c90 100644
--- a/back/methods/dms/deleteTrashFiles.js
+++ b/back/methods/dms/deleteTrashFiles.js
@@ -1,6 +1,7 @@
 const UserError = require('vn-loopback/util/user-error');
 const fs = require('fs-extra');
 const path = require('path');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 module.exports = Self => {
     Self.remoteMethod('deleteTrashFiles', {
@@ -22,7 +23,7 @@ module.exports = Self => {
         if (typeof options == 'object')
             Object.assign(myOptions, options);
 
-        if (!Self.app.models.Application.isProduction())
+        if (!isProduction())
             throw new UserError(`Action not allowed on the test environment`);
 
         const models = Self.app.models;
diff --git a/back/methods/docuware/upload.js b/back/methods/docuware/upload.js
index ae3989def..0102911e0 100644
--- a/back/methods/docuware/upload.js
+++ b/back/methods/docuware/upload.js
@@ -1,5 +1,6 @@
 const UserError = require('vn-loopback/util/user-error');
 const axios = require('axios');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 module.exports = Self => {
     Self.remoteMethodCtx('upload', {
@@ -119,7 +120,7 @@ module.exports = Self => {
                 ]
             };
 
-            if (!Self.app.models.Application.isProduction(false))
+            if (!isProduction(false))
                 throw new UserError('Action not allowed on the test environment');
 
             // delete old
diff --git a/back/methods/image/scrub.js b/back/methods/image/scrub.js
index 4af2d8d49..3c83b3be7 100644
--- a/back/methods/image/scrub.js
+++ b/back/methods/image/scrub.js
@@ -1,6 +1,7 @@
 const fs = require('fs-extra');
 const path = require('path');
 const UserError = require('vn-loopback/util/user-error');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 module.exports = Self => {
     Self.remoteMethod('scrub', {
@@ -43,7 +44,7 @@ module.exports = Self => {
     Self.scrub = async function(collection, remove, limit, dryRun, skipLock) {
         const $ = Self.app.models;
 
-        dryRun = dryRun || !Self.app.models.Application.isProduction(false);
+        dryRun = dryRun || !isProduction(false);
 
         const instance = await $.ImageCollection.findOne({
             fields: ['id'],
diff --git a/back/methods/image/upload.js b/back/methods/image/upload.js
index 81d9759e6..b3cdfb88b 100644
--- a/back/methods/image/upload.js
+++ b/back/methods/image/upload.js
@@ -1,6 +1,7 @@
 const UserError = require('vn-loopback/util/user-error');
 const fs = require('fs/promises');
 const path = require('path');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 module.exports = Self => {
     Self.remoteMethodCtx('upload', {
@@ -41,7 +42,7 @@ module.exports = Self => {
             if (!hasWriteRole)
                 throw new UserError(`You don't have enough privileges`);
 
-            if (!Self.app.models.Application.isProduction())
+            if (!isProduction())
                 throw new UserError(`Action not allowed on the test environment`);
 
             // Upload file to temporary path
diff --git a/back/methods/notification/send.js b/back/methods/notification/send.js
index 4f8f436da..1bff7f686 100644
--- a/back/methods/notification/send.js
+++ b/back/methods/notification/send.js
@@ -1,4 +1,5 @@
 const {Email} = require('vn-print');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 module.exports = Self => {
     Self.remoteMethod('send', {
@@ -70,7 +71,7 @@ module.exports = Self => {
                     const newParams = Object.assign({}, queueParams, sendParams);
                     const email = new Email(queueName, newParams);
 
-                    if (Self.app.models.Application.isProduction())
+                    if (isProduction())
                         await email.send();
 
                     await queue.updateAttribute('status', statusSent);
diff --git a/loopback/common/models/application.js b/loopback/common/models/application.js
index 70db17de2..80c58ddc1 100644
--- a/loopback/common/models/application.js
+++ b/loopback/common/models/application.js
@@ -1,5 +1,3 @@
-const isProductionFn = require('../methods/application/isProduction');
-
 module.exports = function(Self) {
     require('../methods/application/status')(Self);
     require('../methods/application/post')(Self);
@@ -7,8 +5,4 @@ module.exports = function(Self) {
     require('../methods/application/executeProc')(Self);
     require('../methods/application/executeFunc')(Self);
     require('../methods/application/getEnumValues')(Self);
-
-    Self.isProduction = (localAsProduction = true) => {
-        return isProductionFn(localAsProduction);
-    };
 };
diff --git a/loopback/common/methods/application/isProduction.js b/loopback/server/boot/isProduction.js
similarity index 100%
rename from loopback/common/methods/application/isProduction.js
rename to loopback/server/boot/isProduction.js
diff --git a/modules/account/back/models/ldap-config.js b/modules/account/back/models/ldap-config.js
index 4f5c0218d..583ce084b 100644
--- a/modules/account/back/models/ldap-config.js
+++ b/modules/account/back/models/ldap-config.js
@@ -3,7 +3,7 @@ const app = require('vn-loopback/server/server');
 const ldap = require('../util/ldapjs-extra');
 const crypto = require('crypto');
 const nthash = require('smbhash').nthash;
-const isProduction = require('vn-loopback/common/methods/application/isProduction');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 module.exports = Self => {
     const shouldSync = isProduction();
diff --git a/modules/account/back/models/samba-config.js b/modules/account/back/models/samba-config.js
index aa68af35d..359b4b187 100644
--- a/modules/account/back/models/samba-config.js
+++ b/modules/account/back/models/samba-config.js
@@ -1,7 +1,7 @@
 
 const ldap = require('../util/ldapjs-extra');
 const execFile = require('child_process').execFile;
-const isProduction = require('vn-loopback/common/methods/application/isProduction');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 /**
  * Summary of userAccountControl flags:
diff --git a/modules/client/back/methods/sms/send.js b/modules/client/back/methods/sms/send.js
index 269dedacb..2b5674f86 100644
--- a/modules/client/back/methods/sms/send.js
+++ b/modules/client/back/methods/sms/send.js
@@ -1,5 +1,6 @@
 const got = require('got');
 const UserError = require('vn-loopback/util/user-error');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 module.exports = Self => {
     Self.remoteMethod('send', {
@@ -47,7 +48,7 @@ module.exports = Self => {
 
         let response;
         try {
-            if (!Self.app.models.Application.isProduction(false))
+            if (!isProduction(false))
                 response = {result: [{status: 'ok'}]};
             else {
                 const jsonTest = {
diff --git a/modules/invoiceOut/back/methods/invoiceOut/download.js b/modules/invoiceOut/back/methods/invoiceOut/download.js
index 5a9d5bc51..f8d42072c 100644
--- a/modules/invoiceOut/back/methods/invoiceOut/download.js
+++ b/modules/invoiceOut/back/methods/invoiceOut/download.js
@@ -1,5 +1,6 @@
 const fs = require('fs-extra');
 const path = require('path');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 module.exports = Self => {
     Self.remoteMethodCtx('download', {
@@ -66,7 +67,7 @@ module.exports = Self => {
             console.error(err);
         });
 
-        if (!Self.app.models.Application.isProduction()) {
+        if (!isProduction()) {
             try {
                 await fs.access(file.path);
             } catch (error) {
diff --git a/modules/invoiceOut/back/models/invoice-out.js b/modules/invoiceOut/back/models/invoice-out.js
index 166565fe2..b0e05b626 100644
--- a/modules/invoiceOut/back/models/invoice-out.js
+++ b/modules/invoiceOut/back/models/invoice-out.js
@@ -1,6 +1,7 @@
 const print = require('vn-print');
 const path = require('path');
 const UserError = require('vn-loopback/util/user-error');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 module.exports = Self => {
     require('../methods/invoiceOut/filter')(Self);
@@ -59,7 +60,7 @@ module.exports = Self => {
             hasPdf: true
         }, options);
 
-        if (Self.app.models.Application.isProduction()) {
+        if (isProduction()) {
             await print.storage.write(buffer, {
                 type: 'invoice',
                 path: pdfFile.path,
diff --git a/modules/mdb/back/methods/mdbVersion/upload.js b/modules/mdb/back/methods/mdbVersion/upload.js
index 9edaf454f..64de72679 100644
--- a/modules/mdb/back/methods/mdbVersion/upload.js
+++ b/modules/mdb/back/methods/mdbVersion/upload.js
@@ -1,6 +1,7 @@
 const fs = require('fs-extra');
 const path = require('path');
 const UserError = require('vn-loopback/util/user-error');
+const isProduction = require('vn-loopback/server/boot/isProduction');
 
 module.exports = Self => {
     Self.remoteMethodCtx('upload', {
@@ -111,7 +112,7 @@ module.exports = Self => {
             const destinationFile = path.join(
                 accessContainer.client.root, accessContainer.name, appName, `${toVersion}.7z`);
 
-            if (!Self.app.models.Application.isProduction())
+            if (!isProduction())
                 await fs.unlink(srcFile);
             else {
                 await fs.move(srcFile, destinationFile, {
-- 
2.40.1