Item13318: Shipped topics are missing attachment metadata, and TOPICINFO

Some of the Icon sets don't have any attachment meta in their topics.

-- GeorgeClark - 21 Mar 2015

Also several topic didn't contain TOPICINFO headers.

-- GeorgeClark - 21 Mar 2015

Expanding this to cover any default topics. Finding more with out-of-sync metadata

-- GeorgeClark - 21 Mar 2015

MichaelDaum... could you please review. Thanks.

• DefaultUserRegistration/validate.js doesn't have compressed and gzipped forms in the manifest.
• UserRegistration/js.js also missing compressed / gzipped forms

I think that js.js is not used, and validate.js probably ought to be moved to the JavascriptFiles topic.

-- GeorgeClark - 23 Mar 2015

Yes, adding proper TOPICINFO is a Good Thing To Do. This wasn't the case previously as we worked under the impression that some day TOPICINFO should be removed from the TXT file. This actually is a very very old discussion that didn't lead to anywhere up to now. During the course of 1.2.0 it then turned out that not having any TOPICINFO forces the store code into a slow code path that severely impacts overall performance of the system. So any topic not having proper TOPICINFO is considered "dirty" and as such in need of a forced save to repair the topic. Which means: any topic without a TOPICINFO that is about to be saved by a user will actually be saved twice: (1) a repairment save to bring back proper TOPICINFO and (2) the actual save done by the user. This is of course a bad thing to happen and should remain an exception.

So shipping all topics of the distro including TOPICINFO is absolutely the right thign to do. As well as is the case for any other extension being built. There are still a lot of extension that ship "dirty" TXT files. But that's probably worth another task.

Wrt piling up js files at System.JavascriptFiles: Let's better keep those js files close to where they are actually, unless they are of more general use somewhere else. The mentioned js files are probably closely related to the registration app. So better keep them there in that case.

-- MichaelDaum - 23 Mar 2015

Commenting is disabled while not logged in