Luke F
Well-known member
This is brilliant, just a couple of things:
- Possibly move the new column on xf_post to installation code rather than doing the describe table every run
- For the option ids you should use a prefix to avoid conflicts on common words like 'Ignore' and 'enabled'
- Maybe make the cron run less often and with a larger batch by default
- If possible add some detection of dynamic images (especially when the new signature code is added), by server cache headers or url (by presence of query string and/or '.php') or something. It's quite common on gaming forums that you will get people putting dynamic statistics images in their signatures
- Possibly move the new column on xf_post to installation code rather than doing the describe table every run
- For the option ids you should use a prefix to avoid conflicts on common words like 'Ignore' and 'enabled'
- Maybe make the cron run less often and with a larger batch by default
- If possible add some detection of dynamic images (especially when the new signature code is added), by server cache headers or url (by presence of query string and/or '.php') or something. It's quite common on gaming forums that you will get people putting dynamic statistics images in their signatures