3.3: final preparation and performance

Quote from: rthrash at Oct 19, 2005, 01:33 PM

1) Can we programatically get the email address of the Manager user ID 0. I would like to use it as the default for the contact form.

Yes. Look in my NewsFeed code as it does this when no default username is set. If you need I can pull this code out for you.

Quote from: sottwell at Oct 19, 2005, 01:39 PM

Why not use [(emailsender)]?

I think that’s the perfect solution. Maybe all the tags could be added to the bottom of http://modxcms.com/adding-tags.html and also in a separate section of the Developer’s guide.

It’s covered both in Content Editor’s Guide - MODx Tags, and in Designer’s Guide, Adding MODx Tags. I specifically mention [(emailsender)] in both documents.

And now that you point it out, I see that! Do you think a DL of tags/uses would be helpful at the end of the docs?

Quote from: rthrash at Oct 19, 2005, 01:33 PM

2) Can someone help me get to the bottom of the backslash issue found in the blog section. Look at the topmost Safari entry for an example. When displayed as a standalone article it has no backslashes. When displayed as a summary, it does.

Figured it out!
’s aren’t stored in the db escaped. They are stored normally as in here’s.

The parser must be filtering them out for pages but not for snipets

This also appears in the manager with pagetitles appearing with /’s in the tree but ’s in the edit form.

This brings up the fundamental issue of whether you want ’s escaped in sql. If so there is some major work to be done.

I have solved the problem in NewsPublisher and issued a patch which is attached to this post. It also adds a redirect to the document created function I wrote a while back but forgot about.

I think we’re really close to ready on the new "default" content. Very very minor stuff, in fact.

• Work out backslash issue in NewsPublisher snippet (which it appears Mark has worked out (and thanks Susan for the great email suggestion!)
• Delete the miscellaneous "blog testing" posts comments and pages
• Work out why the "enter your comments below" stuff is coming across in the Newslisting ... it shouldn’t probably do that ...
• Thorough testing of all content, and of QuickEditor in IE

It would be nice to have a way to count the number of comments and return that as a seperate part of the Read More ... line of the NewsListing summaries page. Ideas there?

As usual... review the latest at Pixelhick.com.

Hi Mark,

Thanks for the update. This version will remove magic quotes if they are used. Magic quotes can sometimes interfare with these type of applications.

Please test and verify that all is ok

Ryan,

Can you finalize the name of the images used inside the template and send them to me?

I think we should change assets/templates/TicTac-Wuhan to assets/templates/default

I think we need to meta tags instead of [[GetKeywords]]

Quote from: xwisdom at Oct 21, 2005, 03:39 AM

Ryan,

Can you finalize the name of the images used inside the template and send them to me?

OK if I just commit them to the current trunk?

I think we should change assets/templates/TicTac-Wuhan to assets/templates/default

Agreed and done and will likewise commit.

I think we need to meta tags instead of [[GetKeywords]]

The use of the Meta content on a per page basis is awkward, particularly if you’re working on a page after the fact. This is another one of those things taht should probably be moved to the template level, with per-page overrides. Regardless, it’s done.

Images committed now, and other files no longer needed removed.