Bdtjtk

I've used this code to help delete mass messages, but now for some reason it's not working.

Delete all messages from specific channel:

Traceback (most recent call last):

  File "C:UserstestOneDriveDocumentsmass.py", line 33, in <module>

delete_all(auth_token, channel_id, username1, username2, get_all_messages(auth_token, channel_id))



  File "C:UserstestOneDriveDocumentsmass.py", line 16, in get_all_messages

prev = prev + messages



TypeError: can only concatenate list (not "dict") to list

Code:

import json, requests, sys

print ("Delete all messages from specific channel")



username1 = "test"

username2 = "test#0001"

auth_token = "RWYFHyrtYRY.RYYR_jqj114452E"

channel_id = "345634345364"

delete_from_all_users = "False"



def get_all_messages(auth, id, last="", prev=):

    if not last:

        messages = json.loads(requests.get("http://canary.discordapp.com/api/v6/channels/" + id + "/messages", headers={"authorization": auth}, params={"limit": 100}).content)

    else:

        messages = json.loads(requests.get("http://canary.discordapp.com/api/v6/channels/" + id + "/messages", headers={"authorization": auth}, params={"before" : last, "limit" : 100}).content)



    prev = prev + messages



    if len(messages) < 100:

        print ("Got to end of channel at " + str(len(prev)) + " messages")

        return prev

    else:

        oldest = sorted(messages, key=lambda x: x["timestamp"], reverse=True)[-1]

        return get_all_messages(auth, id, last=oldest["id"], prev=prev)



def delete_all(auth, id, user1, user2, messages):

    print ("Trying to delete all messages in " + id + " from username " + user1)

    for message in messages:

        # print(message["author"]["username"])

        if (message["author"]["username"] == user1):

            requests.delete("http://canary.discordapp.com/api/v6/channels/" + id + "/messages/" + message["id"],headers={"authorization": auth})

    print ("All messages were deleted")



delete_all(auth_token, channel_id, username1, username2, get_all_messages(auth_token, channel_id))

I'm not sure what changed. The script worked. I ran it again a month later, got this error.

If you want to keep the old behavior of having a flat list when the content received is itself a list, but also be able to append non-list received messages as individual items, consider:

if isinstance(messages, list):

    prev.extend(messages)

else:  # messages is not really a list!

    prev.append(messages)

Otherwise, if you just want to append the newly-received batch of messages as a single item to your list no matter what, that would look like:

prev.append(messages)

You can only use + with a list on the left-hand side if the thing on the right-hand side is also a list. The error message indicates that not to be the case here (the thing on the right-hand side, in the responses you're getting from the server today, is a dict and not a list).

messages should be a list, in your case it's a dictionary concatenation should be list + list

If you need to append an int or any other single element, you could always convert it into a single element list using or directly use append.

prev = prev + [messages]

OR

prev.append(messages)