I'm a little concerned about the template sharing and possible issues related to it, but I suppose I'll have those concerns answered when the release is available. Everything else looks promising. Can't wait to see the rest of the feature list :D
While I understand that there is nothing wrong with the script, you do not require that I call the compile functions using do_update. Wouldn't it be more uniform to have both functions working similar? Especially in this case, where the update queries are a bit tedious to write out....
I jsut got around to playing with the MI again today and was installing it on 2.1.7. I ran across a few changes to the import_xml function. The most concerning is the following code found on line 474 of /sources/lib/admin_functions.php
Sorry to have this reopened Brandon, but I disagree (for situations just like this ) that wild cards should not be used for banned names. Also, for some reason wildcards we left in. If you put ma*tt (as suggested in the bug report above) you could match anything just about. "maps to the burger patty" would match ma*tt for example. What does that have to do with ma*tt?
Perhaps another method of defining wildcards needs to be used (ie [*]), but wildcards should not be disabled. Personally I believe that * should be removed from user names all together. It's quite annoying.
Don't you think that capitalizing the first word in a no-caps topic is a little outside of the boundry of the ACP option? I mean I set it to stop shouting, not stop...whispering? Might be a big pain in the butt to check before running: