Uploaded image for project: 'Blesta Core'
  1. Blesta Core
  2. CORE-5301

Contact email verification banner not removed after verification

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 5.10.0
    • Fix Version/s: 5.11.0-b3
    • Component/s: Client Interface
    • Labels:
      None

      Description

      To reproduce:

      1. Enable email verfication
      2. Add a contact (may or may not happen with main contact)
      3. Complete email verification
      4. Note the banner still appears

      Possible fix:

      Based on what I saw, looks like logic is wrong in app/controllers/client_contacts.php on line 250

      I think that should be not operator:

      if ((!$email_verification = $this->EmailVerifications->getByContactId($contact->id))) {

        Activity

        admin Paul Phillips created issue -
        admin Paul Phillips made changes -
        Field Original Value New Value
        Rank Ranked higher
        admin Paul Phillips made changes -
        Sprint 5.11.0 Sprint 7 [ 207 ]
        admin Paul Phillips made changes -
        Rank Ranked higher
        jonathan Jonathan Reissmueller made changes -
        Sprint 5.11.0 Sprint 7 [ 207 ] 5.11.0 Sprint 7, 5.12.0 Sprint 1 [ 207, 208 ]
        jonathan Jonathan Reissmueller made changes -
        Rank Ranked higher
        abdy Abdy Franco made changes -
        Assignee Abdy Franco [ abdy ]
        abdy Abdy Franco made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        abdy Abdy Franco made changes -
        Remaining Estimate 0 minutes [ 0 ]
        Time Spent 1 hour, 43 minutes [ 6180 ]
        Worklog Id 17516 [ 17516 ]
        abdy Abdy Franco made changes -
        Status In Progress [ 3 ] In Review [ 5 ]
        Resolution Fixed [ 1 ]
        jonathan Jonathan Reissmueller made changes -
        Status In Review [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            abdy Abdy Franco
            Reporter:
            admin Paul Phillips
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0 minutes
              0m
              Logged:
              Time Spent - 1 hour, 43 minutes
              1h 43m

                Agile