Dave had a simple array[] with the TLDs in it, and Spam Karma goes through that. The problem is that Dave's list is 276 items long, which isn't too bad. The Mozilla list is over 3,000 lines long, and that includes some wildcards!
I'm no PHP expert, but am I correct in guessing that feeding PHP a 3,000+ element array to parse is going to cause issues? Perhaps we should be adding a MySQL table to the database and doing queries against that instead?
Or am I underestimating PHP?
Thoughts? Is this going to cause a problem or not?
Stephen
I agree that it's a shame and that it has worked well over the years.
However, it basically needs to be re-written from scratch to be
deployed now.
Actually, I still use SK2 on my own blogs and it does indeed mostly
work. I'd be ready to give a hand and might be able to make some
change faster, having written the thing in the first place (thoughI
haven't opened it in years and barely even remember what went were).
Many small fixes and suggestions have been made at the time, but I
don't think anybody has had the time to commit to a full update so far
(I very much understand how it is).
I don't really think SK2 needs a full-rewrite as it is... I haven't
seen any major compatibility issue with the latest version of WP so
far, despite all the breaking updates.
I suggest making a quick update with a limited scope that fixes all
urgent problems. Of course people should feel free to go on and make
deeper changes later on, but a small scope would guarantee that at
least this upgrade sees the light of day.
My top 3 changes would be:
1. Remove the phone-home/check-update call which used the old
wp-plugins.net website. The domain has been handed over to WP/Matt,
who removed all the former code and the URL is now 404. Dunno if it
would be possible to replace it by a URL on Google Code. Otherwise
just comment it.
2. Address the TLD issue. Although in practice I have never had
problems so far, it is indeed an issue.
I am not convinced that supporting the new ICANN "commercial brand"
tld is really a great idea, in that I can smell it becoming a strong
source of spam URLs in the future (and I can't see many reason a
non-SEO humans would have to sign with a .BRAND URL). However,
technically, they will be legal URLs and should supported.
This means replacing the last part of the regex by a blank star, which
means in turn the potential for a lot more false positives (since it
will essentially be matching any two words separated by a dot and no
spacing). I am in favour of making this an optional setting: 1. Legacy
mode *or* 2. Support generic TLD... and people can choose what fit
their blog type best.
3. It'd be nice to fix the nonce used in WP URLs, so that email links
for moderation can be clicked and work. Back when I added nonce
support, WP code was utterly broken and there was no way to make it
work. I am sure they fixed it by now and it shouldn't be too hard to
fix it on SK2 side.
Does anybody see any other small-scale change that should be added to the list?
I'll have a look at the code later today and get started on 1. and
2... If anybody feel up for a look at WP doc regarding nonce, I'd
certainly appreciate the help.
Cheers!
> --
> You received this message because you are subscribed to the Google Groups
> "SK2 GPL Dev" group.
> To view this discussion on the web visit
> https://groups.google.com/d/msg/sk2-gpl-dev/-/yDDkrhOiRwsJ.
> To post to this group, send email to sk2-g...@googlegroups.com.
> To unsubscribe from this group, send email to
> sk2-gpl-dev...@googlegroups.com.
> For more options, visit this group at
> http://groups.google.com/group/sk2-gpl-dev?hl=en.
>
I've been making minor commits all along, including fairly recently. Certainly more recent than a couple years. I don't consider the plugin dead.
Just FYI.
In terms of updates, the first thing I think we need to do is fix up the Comment Page/Dashboard integration that was begun and never quite finished in Trunk. The accordion effect that results from the open/close on rollover is terrible! Preferably it should just show the Karma score, and on *click*, open up the details.
Regards,
Stephen Rider