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]: heldForReview flag in config does not work with delete_without_reviewing in moderator mode #935

Closed
1 of 2 tasks
Practicalbutterfly5 opened this issue Sep 29, 2022 · 2 comments
Assignees
Labels
Bug / Error Something isn't working Feature Added Feature has been added

Comments

@Practicalbutterfly5
Copy link

Practicalbutterfly5 commented Sep 29, 2022

Duplicate Issues

  • There are no existing posts relating to my problem
  • There are existing posts relating to my problem, but the solution given, doesn't work for me.

What happened?

Edit: I have ironed out the cause of this issue. It's in the following code. Here the script checks if userisnotowner=true and sets deletion mode to report without checking if moderator mode is true.

I am trying to set up a fully automated hold for review system for another channel. I am the moderator on that channel.
But using delete_without_reviewing = True with removal_type = heldForReview does not work for me in moderator mode and instead of holding comments for review it reports them


+use_this_config = True
+your_channel_id = removed
+skip_confirm_video = True
+moderator_mode = True
+auto_close = True
+scan_mode = RecentVideos
+channel_to_scan = removed
+recent_videos_amount = 3
+filter_mode = AutoSmart
+detect_link_spam = False
+delete_without_reviewing = True
+remove_all_author_comments = False
+removal_type = heldForReview
+whitelist_excluded = False
+enable_logging = True

Release version

YT-Spammer-Purge-2.17.0-Beta2

Steps to reproduce

  1. Use the config options mentioned earlier.

What platform are you seeing this problem on?

Linux (can also be ChromeOS)

Relevant log output

.
.
.

============================ Match Samples: One comment per matched-comment author ============================
 1.  [x53]  telegram me @geek...: Congratulations among my lucky winners ��..                                       




==================== (See log file for channel IDs of matched authors above) ====================

 NOTE:  Check that all comments listed above are indeed spam.
 > If you see missed spam or false positives, you can submit a filter suggestion here: TJoe.io/filter-feedback

Updating log file, please wait...
                                          
 Finishing Log File...
                               


Reporting Comments... - Progress: [0 / 53] (In Groups of 50)
Reporting Comments... - Progress: [50 / 53] (In Groups of 50)
Reporting Comments... - Progress: [53 / 53] (In Groups of 50)
Comments Reported! If no error messages were displayed, then everything was successful.

Program Complete.
Auto-close enabled in config. Exiting in 5 seconds...
@Practicalbutterfly5 Practicalbutterfly5 added the Bug / Error Something isn't working label Sep 29, 2022
@Practicalbutterfly5 Practicalbutterfly5 changed the title [Bug]: heldForReview flag in config does not work with autodelete in moderator mode [Bug]: heldForReview flag in config does not work with automatic heldForReview in moderator mode Sep 29, 2022
@Practicalbutterfly5 Practicalbutterfly5 changed the title [Bug]: heldForReview flag in config does not work with automatic heldForReview in moderator mode [Bug]: heldForReview flag in config does not work with delete_without_reviewing in moderator mode Sep 29, 2022
@ThioJoe
Copy link
Owner

ThioJoe commented Dec 19, 2022

Got a chance to look at this and I'm not sure if I had put that there on purpose or not. But even if it was, it should still have some kind of message or warning. I'll have to review that to make sure if automated that it can only be run in certain modes. Unfortunately it might take a bit because that's the part of the program with some of the most complicated logic.

@LaloCo
Copy link

LaloCo commented Apr 5, 2023

Yeah, I need this too. Forking to try @Practicalbutterfly5's cause of the issue.

@LaloCo LaloCo mentioned this issue Apr 5, 2023
8 tasks
@ThioJoe ThioJoe closed this as completed Jan 1, 2024
@ThioJoe ThioJoe added the Feature Added Feature has been added label Jan 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug / Error Something isn't working Feature Added Feature has been added
Projects
None yet
Development

No branches or pull requests

3 participants