oaeproject/Hilary

Avoid too many return statements within this function.

Open

    return createUser();

Avoid too many return statements within this function.

Confirmed

    return response.status(400).send('You need to accept the Terms and Conditions');

Similar blocks of code found in 7 locations. Consider refactoring.

Open

OAE.tenantRouter.on('get', '/api/user/:userId/memberships', (request, response) => {
  const limit = OaeUtil.getNumberParam(request.query.limit, 10, 1, 25);
  PrincipalsAPI.getMembershipsLibrary(
    request.ctx,
    request.params.userId,

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

OAE.tenantRouter.on('post', '/api/user/:userId/email/verify', (request, response) => {
  PrincipalsAPI.verifyEmail(request.ctx, request.params.userId, request.body.token, (error, user) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

OAE.tenantRouter.on('post', '/api/user/:userId/termsAndConditions', (request, response) => {
  PrincipalsAPI.acceptTermsAndConditions(request.ctx, request.params.userId, (error, user) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

const _handleDeleteUser = function (request, response) {
  PrincipalsAPI.deleteUser(request.ctx, request.params.userId, (error) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

const _handleGetFullProfile = function (request, response) {
  PrincipalsAPI.getFullUserProfile(request.ctx, request.params.userId, (error, user) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

const _getUsersForTenant = function (request, response) {
  PrincipalsAPI.getAllUsersForTenant(request.ctx, request.params.tenantAlias, (error, users) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

const _handleRestoreUser = function (request, response) {
  PrincipalsAPI.restoreUser(request.ctx, request.params.userId, (error) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

const _handleResendEmailToken = function (request, response) {
  PrincipalsAPI.resendEmailToken(request.ctx, request.params.userId, (error) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

const _deleteEmailToken = function (request, response) {
  PrincipalsAPI.deleteEmailToken(request.ctx, request.params.userId, (error) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

const _handleUpdateUser = function (request, response) {
  PrincipalsAPI.updateUser(request.ctx, request.params.userId, request.body, (error, user) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

const _handleGetMe = function (request, response) {
  PrincipalsAPI.getMe(request.ctx, (error, meData) => {
    if (error) {
      return response.status(error.code).send(error.msg);
    }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  const recaptchaData = {
    remoteip: request.connection.remoteAddress,
    challenge: request.body.recaptchaChallenge,
    response: request.body.recaptchaResponse
  };

• Read up
  Read up