Certainly a small issue that can wait, but this module IMO clutters the variables table. When querying the table with LIKE 'pathauto_%' there are 63 rows in the version I am using, could this data not just be serialized and stored as a single row?

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Closed (won't fix)

It could be stored as a single row, but according to http://www.drupaltoughlove.com/review/printer-friendly-pages that´s a bad idea (see the part about using #tree TRUE).

Generally speaking, I am not sure of a reason to keep the variables table "clean". If there is a performance problem or some other reason then sure, but as far as I know that is not a problem with 63 entries or even many more.

tjholowaychuk’s picture

tjholowaychuk commented

True, it would not work with system_settings_form(). Certainly your call! its a great module I just feel bad plastering the variables table with entries since the entire table is scanned, I suppose the caching makes that pretty irrelevant though