minor glitches in entry-parsing

[matsch]

hi,

i found 2 minor glitches for entries:

1. if you have a "--" in your entries titel the rdf-tags after the article will fuck up and a "-->" will be shown on the website after your article.
example:

Code: Select all

        <!--
        <rdf:RDF xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
                 xmlns:trackback="http://madskills.com/public/xml/rss/module/trackback/"
                 xmlns:dc="http://purl.org/dc/elements/1.1/">
        <rdf:Description
                 rdf:about="..."
                 trackback:ping="..."
                 dc:title="Test -->
        
        -->

2. emoticons at the very beginning of an entry will not be converted (it took me a while to figure what the problem was!).

not really worth mentioning, but...

best regards,
matsch

[matsch]

sorry for this extra post.

1. the used title was "Test -- Test".

matsch

[garvinhicking]

I cannot verify your "--" issue! Is there no ">" sign in your test? Do you have a URL to see this in action? Which event plugins do you have installed?

About the second problem, this is due to the regular expression that tries to not convert smilies that are part of a sentence, and sadly they can't be matche dto the beginning of an entry. We found that it's highly unusual that smilies are put at the beginning of an entry and thus did not bother our heads and making the regexps more complex

Best regards,
Garvin

[matsch]

Hi Garvin,

The bug with the "--" in the title still applies in version 1.0 of s9y. I guess I'm the only one who likes to use -- in titles

The RDF tags fuck up even on a fresh install with the new default template.

Here is an examlpe:
http://matsch.binaervarianz.de/blog/ind ... orbei.html
Another example (with the fresh install) I send you by PM.

Here my installed Event Plugins (on the fresh one):

Code: Select all

Markup: Serendipity, version: 1.1
Markup: Emoticate, version: 1.2
Markup: NL2BR, version: 1.3
Browser Compatibility, version: 1.0
Spam Protector, version: 1.51
Spartacus, version: 2.7
Announce entries, version: 1.02
Autosave entries, version: 0.2.1
Tagging of entries, version: 2.50
Fix common XHTML errors, version: 1.4
[Layout-Plugin: Printerfriendly version], version: 1.0
[Layout-Plugin: Quote-Markup], version: 1.0
Livesearch, version: 1.3
Extended properties for entries, version: 1.8

Hope this helps!

Thank and best regards,
Matthias

[matsch]

Hi,

I just checked. It doesn't seem to be valid XHTML 1.0:

http://validator.w3.org/check?verbose=1 ... -Test.html

What to do?

Thanks,
matsch

btw., I also filed a bug for firefox - maybe too fast

[garvinhicking]

Hi!

The "--" bug got fixed. It requires a template change to work, though.

In your RDF section you need to use:

Code: Select all

dc:title="{$entry.title_rdf|@default:$entry.title}"

instead of

Code: Select all

dc:title="{$entry.title}"

Best regards,
Garvin.

[matsch]

Ok.

Thanks. I will have a look into it later...

I knew just after posting I was to fast filing this bug report...
https://bugzilla.mozilla.org/show_bug.cgi?id=342494

Too lazy late at night...

Anyway. Thanks again,
matsch

[matsch]

The "--" bug got fixed.

Maybe in the CVS version you mean? Didn't do the trick for me (v1.0).

Waiting for v1.1!

Sorry for bothering!

Thanks,
matsch

[garvinhicking]

Hi!

The fix works in 1.0. What does your entries.tpl look like, did you include the patch that I told you to insert?

Of course s9y can't fix bugs in your own templates, but it has been fixed in all Spartacus + bundled templates.

Regards,
Garvin