Feature/edit company details

src/api/middleware/company.js

@@ -110,15 +110,18 @@ export const isNotDisabled = (owner) => async (req, res, next) => {
 };
 
 export const canManageAccountSettings = (companyId) => async (req, res, next) => {
-    const company = await (new CompanyService()).findById(companyId, true);
-
-    // only god or the same company can change account settings
-    if (!req.hasAdminPrivileges && company._id.toString() !== req.targetOwner) {
-        return next(new APIError(
-            HTTPStatus.FORBIDDEN,
-            ErrorTypes.FORBIDDEN,
-            ValidationReasons.INSUFFICIENT_PERMISSIONS_COMPANY_SETTINGS
-        ));
+    try {
+        const company = await (new CompanyService()).findById(companyId, true);
+        if (!req.hasAdminPrivileges && company._id.toString() !== req.targetOwner) {
+            return next(new APIError(
+                HTTPStatus.FORBIDDEN,
+                ErrorTypes.FORBIDDEN,
+                ValidationReasons.INSUFFICIENT_PERMISSIONS_COMPANY_SETTINGS
+            ));
+        }
+        return next();
+    } catch (err) {
+        console.error(err);
+        return next(err);
     }
-    return next();
 };

src/api/middleware/validators/company.js

@@ -75,7 +75,6 @@ export const deleteCompany = useExpressValidators([
     existingCompanyParamValidator,
 ]);
 
-
 export const getOffers = useExpressValidators([
     existingCompanyParamValidator,
 ]);
@@ -106,3 +105,28 @@ export const setDefaultValuesConcurrent = (req, res, next) => {
     }
     return next();
 };
+
+export const edit = useExpressValidators([
+    existingCompanyParamValidator,
+    body("name", ValidationReasons.DEFAULT)
+        .optional()
+        .isString().withMessage(ValidationReasons.STRING).bail()
+        .isLength({ min: CompanyConstants.companyName.min_length })
+        .withMessage(ValidationReasons.TOO_SHORT(CompanyConstants.companyName.min_length))
+        .isLength({ max: CompanyConstants.companyName.max_length })
+        .withMessage(ValidationReasons.TOO_LONG(CompanyConstants.companyName.max_length)),
+    body("bio", ValidationReasons.DEFAULT)
+        .optional()
+        .isString().withMessage(ValidationReasons.STRING).bail()
+        .isLength({ max: CompanyConstants.bio.max_length })
+        .withMessage(ValidationReasons.TOO_LONG(CompanyConstants.bio.max_length)),
+    body("contacts", ValidationReasons.DEFAULT)
+        .optional()
+        .customSanitizer(ensureArray)
+        .isArray({ min: CompanyConstants.contacts.min_length, max: CompanyConstants.contacts.max_length })
+        .withMessage(ValidationReasons.ARRAY_SIZE(CompanyConstants.contacts.min_length, CompanyConstants.contacts.max_length)),
+    body("logo", ValidationReasons.DEFAULT)
+        .optional()
+        .isString().withMessage(ValidationReasons.STRING).bail()
+        .isURL().withMessage(ValidationReasons.URL),
+]);

src/api/routes/company.js

@@ -202,4 +202,33 @@ export default (app) => {
                 return next(err);
             }
         });
+
+
+    /**
+     * Edit company details.
+     * Company or admin can edit.
+     */
+
+    router.put("/:companyId/edit",
+        or([
+            authMiddleware.isCompany,
+            authMiddleware.isAdmin,
+            authMiddleware.isGod
+        ], { status_code: HTTPStatus.UNAUTHORIZED, error_code: ErrorTypes.FORBIDDEN, msg: ValidationReasons.INSUFFICIENT_PERMISSIONS }),
+        validators.edit,
+        (req, res, next) => companyMiddleware.canManageAccountSettings(req.params.companyId)(req, res, next),
+        (req, res, next) => companyMiddleware.isNotBlocked(req.params.companyId)(req, res, next),
+        (req, res, next) => companyMiddleware.isNotDisabled(req.params.companyId)(req, res, next),
+        async (req, res, next) => {
+            try {
+                const companyService = new CompanyService();
+                const offerService = new OfferService();
+                const company = await companyService.changeAttributes(req.params.companyId, req.body);
+                await offerService.updateAllOffersByCompanyId(company);
+                return res.json(company);
+            } catch (err) {
+                return next(err);
+            }
+        }
+    );
 };

src/services/company.js

@@ -1,8 +1,9 @@
-import { COMPANY_BLOCKED_NOTIFICATION,
+import {
+    COMPANY_BLOCKED_NOTIFICATION,
     COMPANY_UNBLOCKED_NOTIFICATION,
     COMPANY_DISABLED_NOTIFICATION,
     COMPANY_ENABLED_NOTIFICATION,
-    COMPANY_DELETED_NOTIFICATION
+    COMPANY_DELETED_NOTIFICATION,
 } from "../email-templates/companyManagement.js";
 import EmailService from "../lib/emailService.js";
 import Account from "../models/Account.js";
@@ -60,14 +61,14 @@ class CompanyService {
         if (!showHidden) query.withoutBlocked().withoutDisabled();
         if (!showAdminReason) query.hideAdminReason();
         const company = await query;
-        return  company;
+        return company;
     }
 
     /**
      * @param {@param} companyId Id of the company
      */
     async block(companyId, adminReason) {
-        const company =  await Company.findByIdAndUpdate(
+        const company = await Company.findByIdAndUpdate(
             companyId,
             {
                 isBlocked: true,
@@ -96,12 +97,19 @@ class CompanyService {
      * @param {*} company_id id of the company
      * @param {*} attributes object containing the attributes to change in company
      */
-    async changeAttributes(company_id, attributes) {
-        const company = await Company.findOneAndUpdate(
-            { _id: company_id },
-            attributes,
-            { new: true });
-        return company;
+    async changeAttributes(companyId, companyDetails) {
+        try {
+            const company = await Company.findOneAndUpdate(
+                { _id: companyId },
+                companyDetails,
+                { new: true }
+            );
+
+            return company;
+        } catch (err) {
+            console.error(err);
+            throw err;
+        }
     }
 
     /**
@@ -188,7 +196,6 @@ class CompanyService {
             throw err;
         }
     }
-
 }
 
 export default CompanyService;

src/services/offer.js

@@ -468,6 +468,15 @@ class OfferService {
         await Offer.deleteMany({ owner: companyId });
     }
 
+    async updateAllOffersByCompanyId(company) {
+        const changes = {
+            ownerName: company.name,
+            ownerLogo: company.logo,
+            contacts: company.contacts,
+        };
+        const offers = await Offer.updateMany({ owner: company._id }, changes, { new: true });
+        return offers;
+    }
 }
 
 export default OfferService;