Resources

[Ozzy47]

I think it would be a good idea if resources going forward would be required to list the following on the overview page.

1) How many queries it adds to the pages.
2) List of changes any XF tables.
3) List all tables added.
4) Report if it uses any direct DB queries instead of the finder.

Possibly more but I think that covers the major concerns.

 

[Dad.]

Genius

 

[Chris D]

1) How many queries it a adds to the pages.

Query count overall is a fairly pointless metric.

It’s important that queries aren’t executed unnecessarily (n+1 type queries are avoidable, for example) but at the same time we’d welcome a higher query count if it meant avoiding performance or scalability issues. Several quick queries are better than one slow query.

2) List of changes any XF tables.

Generally unnecessary these days. The schema manager alleviates most issues arising from duplicate column names on our side - these used to commonly block upgrades. Even so, good naming and good housekeeping and best practice mean that changing core tables should be fairly insignificant.

3) List all tables added.

Same as above really.

4) Report if it uses any direct DB queries instead of the finder.

Not really any useful information here. Direct DB queries are generally unavoidable and/or just simply acceptable in some cases, though their use should generally be justified. Without reasonable justification that would be against the resource standards and should be reported to the author as a bug in the first instance.