Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] deleted queries still visible via SOAP api with Status=Inactive #737

Closed
2 tasks done
JoernBerkefeld opened this issue Feb 13, 2023 · 1 comment · Fixed by #738
Closed
2 tasks done

[BUG] deleted queries still visible via SOAP api with Status=Inactive #737

JoernBerkefeld opened this issue Feb 13, 2023 · 1 comment · Fixed by #738
Assignees
Labels
bug Something isn't working c/query COMPONENT
Milestone

Comments

@JoernBerkefeld
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

searching for a query by key might return a deleted entry:

        <RetrieveResponseMsg xmlns="http://exacttarget.com/wsdl/partnerAPI">
            <OverallStatus>OK</OverallStatus>
            <RequestID>5067c2d6-5099-48b3-97b3-970b75b7bb5d</RequestID>
            <Results xsi:type="QueryDefinition">
                <PartnerKey xsi:nil="true" />
                <ObjectID>91496c83-27fb-4db5-89a6-17ae6714d46b</ObjectID>
                <CustomerKey>dataView_unsubscribe</CustomerKey>
                <Status>Inactive</Status>
            </Results>
            <Results xsi:type="QueryDefinition">
                <PartnerKey xsi:nil="true" />
                <ObjectID>8dd179c5-bc45-41a0-b971-a73a77ad9540</ObjectID>
                <CustomerKey>dataView_unsubscribe</CustomerKey>
                <Status>Active</Status>
            </Results>
        </RetrieveResponseMsg>

this will impact retrieve-by-key AND delete functionality negatively

Expected Behavior

https://developer.salesforce.com/docs/marketing/marketing-cloud/guide/querydefinition.html

we need to filter by Status=Active on top of any key-filters

Steps To Reproduce

  1. Go to '...'
  2. Click on '....'
  3. Run '...'
  4. See error...

Version

4.3.3

Environment

  • OS:
  • Node:
  • npm:

Participation

  • I am willing to submit a pull request for this issue.

Additional comments

No response

@JoernBerkefeld JoernBerkefeld added bug Something isn't working c/query COMPONENT NEW not yet analyzed new issues get this label labels Feb 13, 2023
@JoernBerkefeld JoernBerkefeld added this to the 4.4.0 milestone Feb 13, 2023
@JoernBerkefeld JoernBerkefeld self-assigned this Feb 13, 2023
JoernBerkefeld added a commit that referenced this issue Feb 13, 2023
JoernBerkefeld added a commit that referenced this issue Feb 13, 2023
@github-actions
Copy link

Closed by #738.

@JoernBerkefeld JoernBerkefeld removed the NEW not yet analyzed new issues get this label label Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment