Uploaded image for project: 'AdMax'
  1. AdMax
  2. ADMAX-2355

problem with getting distribution by accountID

    Details

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

      DistributionService on jboss5

      Description

      Table used to get the distributions from the getDistributionByAccountAndChannelId method in DistributionService on jboss5 does not seem to be mapped with the accountID from tsacommon.accountDistributionMap on dev-db5.

      When I change the status of distributionID=131 for accountID=319 using the updateAccountDistributions method (which in turn updates the tsacommon.accountDistributionMap table), the changed status is reflected in a call to getDistributionByAccountAndChannelId where accountID=319.

      If I then change the status of the same distribution (id=131) but for accountID=11572 this time, the changed status is NOT reflected in a call to getDistributionByAccountAndChannelId where accountID=11572, and instead shows the same status as for accountID=319.

      Note that all changes are reflected as expected in the tsacommon.accountDistributionMap table.

      This is erroneous and leads me to believe that either (A) a distribution's status should be changed without associating it with an accountID (that is, take out the accountID parameter needed in updateAccountDistributions) – please note that only the distributionID, accountID and status are reflected in the tsacommon.accountDistributionMap table. Or (B) the table that is used to retrieve the distributions from the getDistributionByAccountAndChannelId method call must get updated with the accountID from the tsacommon.accountDistributionMap table.

        Attachments

          Activity

            People

            • Assignee:
              dcostantino Dave Costantino (Inactive)
              Reporter:
              caitlyn Caitlyn Duggan (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: