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

Add Ban Plugin Support to Admin Comments Manager #742

Closed
eSilverStrike opened this issue Jan 19, 2017 · 3 comments
Closed

Add Ban Plugin Support to Admin Comments Manager #742

eSilverStrike opened this issue Jan 19, 2017 · 3 comments
Assignees
Labels
Feature Issues that describe new features. low priority
Milestone

Comments

@eSilverStrike
Copy link
Member

Use Ban API to add ban IP support to Geeklogs Comment Manager.

See forum plugin for code example.

Obviously still keep SPAM-X ban IP support but maybe make an API for it (in SPAM-X) so other plugins can add/delete IP's to SPAM-X Ban IP table.

@eSilverStrike eSilverStrike added Feature Issues that describe new features. low priority labels Jan 19, 2017
@eSilverStrike eSilverStrike added this to the 2.2.0 milestone Jan 19, 2017
@mystralkk mystralkk self-assigned this Nov 11, 2017
@mystralkk
Copy link
Member

Implemented with change set 7d7b939. IP addresses banned by the Spam-X plugin are now shown in red. I added the following functions to spamx/functions.inc:

  • SPAMX_isIPBanned($ipToCheck)
  • SPAMX_getBannedIPs()
  • SPAMX_registerBannedIPs($ips)
  • SPAMX_deleteBannedIPs($ips)

@eSilverStrike
Copy link
Member Author

I'm going to open this just so I can add BAN plugin support as well. This way the user can ban the user from posting via spam-x or ban the user outright from visiting the site via the ban plugin

@eSilverStrike eSilverStrike reopened this Nov 11, 2017
@eSilverStrike eSilverStrike self-assigned this Nov 11, 2017
eSilverStrike added a commit that referenced this issue Nov 14, 2017
Added Ban plugin support to Comment Manager Feature #742.

If User banned then name will be in red.

Fix for banning ips of comment submissions. Fix for if Spam-X plugin disabled or not installed.
eSilverStrike added a commit that referenced this issue Nov 14, 2017
Removed ban sql used for testing and rolled back accidental error.log update. Fix for feature #742
@eSilverStrike
Copy link
Member Author

User names also now are in red if user is banned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Issues that describe new features. low priority
Projects
None yet
Development

No branches or pull requests

2 participants