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

Order: Client referral list showing wrong results

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.0.2
    • Component/s: None
    • Labels:
      None

      Description

      To reproduce:

      • Create a client
      • Sign the client up as an affiliate
      • Access the order form as an unauthenticated user following the affiliates referral link
      • Signup and place an order
      • Sign the new user up as an affiliate
      • View the client's referral list in the client interface and see that it shows the referrals from the first client not the second

      To resolve update order/controllers/referrals.php line 37

              if (!($affiliate = $this->OrderAffiliates->getByClientId($this->client->id))) {
      

      With

              if (!($affiliate = $this->OrderAffiliates->getByClientId($this->Session->read('blesta_client_id')))) {
      

      It seems that order/controllers/payouts.php also uses $this->client which may not be set. These also should be replaced by using $this->Session->read('blesta_client_id') or the client object fetched with it.

        Activity

        jonathan Jonathan Reissmueller created issue -
        jonathan Jonathan Reissmueller made changes -
        Field Original Value New Value
        Rank Ranked higher
        jonathan Jonathan Reissmueller made changes -
        Sprint 5.0.2 Sprint 1 [ 128 ]
        jonathan Jonathan Reissmueller made changes -
        Rank Ranked higher
        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 14 minutes [ 840 ]
        Worklog Id 14708 [ 14708 ]
        abdy Abdy Franco made changes -
        Status In Progress [ 3 ] In Review [ 5 ]
        Resolution Fixed [ 1 ]
        abdy Abdy Franco made changes -
        Status In Review [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            abdy Abdy Franco
            Reporter:
            jonathan Jonathan Reissmueller
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:
              Fix Release Date:
              26/Jan/21

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0 minutes
              0m
              Logged:
              Time Spent - 14 minutes
              14m

                Agile