Skip to content
Snippets Groups Projects

Fix #1036

Closed Fix #1036
adrien.faveraux/vtigercrm:portalrestriction into master
Closed Adrien Faveraux requested to merge adrien.faveraux/vtigercrm:portalrestriction into master

Hello,

Fix #1036 (closed)

Merge request reports

Approval is optional

Closed by avatar (Apr 8, 2025 6:00am UTC)

Merge details

  • The changes were not merged into master.

  • Mentions issue #1036 (closed)

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
modules/Contacts/ContactsHandler.php
55 55 $adb->pquery($sql, $params);
56 56 $sendEmail = true;
57 57 }
58
59 58 //update existing portal user password
60 if ($update && $isEmailChanged) {
  • P
    Prasad @prasad ·
    Maintainer

    The (and) clause suits better here as we don't need to update on all changes. Required only when Portal is enabled and Email has changed for existing Contact.

  • Adrien Faveraux @adrien.faveraux started a thread on the diff
    • modules/Contacts/ContactsHandler.php
    55 55 $adb->pquery($sql, $params);
    56 56 $sendEmail = true;
    57 57 }
    58
    59 58 //update existing portal user password
    60 if ($update && $isEmailChanged) {
    • A
      Adrien Faveraux @adrien.faveraux ·
      Author Contributor

      $update is only trigered when Portal acces set to true. Généraly, when we do that, we attempt to customers receive mails. (the filter of alway or not alway is set on the worksflow) or you can add a button for resend a mail ;) :

      image

      or when we modify mail, the login of the cusomers is modified and will not works, send him a mail it's a good idea ;)

  • Prasad Milestone removed

    Milestone removed

  • Prasad Milestone changed to 7.2.0

    Milestone changed to 7.2.0

  • Uma Status changed to closed

    Status changed to closed

    • Please register or sign in to reply

    Copyright 2023 Vtiger. All rights reserved.