096 RC3 Bugs

2,204 Posts

Dimmy Reply #1, 19 years, 4 months ago

File locking does not work any more. when a manager user is editing a document then another manager user can edit the same document, before this was not posible (it should not be possible).

I get duplicated aliases even when I say that I do not want that in the system settings.

Dimmy

follow me on twitter: @dimmy01

576 Posts

sirlancelot Reply #2, 19 years, 4 months ago

Fixed in r2689 of [tt]branches/0.9.6[/tt]

2,204 Posts

Dimmy Reply #3, 19 years, 4 months ago

Nice work.

There is also a QE bug.

When you disable the linkbuttons in the QE menu the buttons are gone (thats good) but the space is still there.
before the space would be filled by the content where the button belogs to.

and QE menu does not work in ie but this fix worked for me:

follow me on twitter: @dimmy01

1,039 Posts

MadeMyDay Reply #4, 19 years, 4 months ago

Hi, just realised that the checkbox setting in the configuration for "show in menu" isn´t recognised when creating a new document. I have set it to "no", but the checkbox is checked when creating a new doc. Can someone confirm? Then I´ll put it to the tracker.

MODx deutsch | Webdesign Karlsruhe | Ressourcen für Webdesign

2,204 Posts

Dimmy Reply #5, 19 years, 4 months ago

Another thing that I miss is the option in backup to clear the logs. Is this placed somwhere els? or is this a bug?

follow me on twitter: @dimmy01

MODX Staff
12,272 Posts

Ryan Thrash Reply #6, 19 years, 4 months ago

Just click the underlined size text ... or is that no longer there?

Ryan Thrash, MODX Co-Founder
Follow me on Twitter at @rthrash or catch my occasional unofficial thoughts at thrash.me

☆ A M B ☆
1,056 Posts

pixelchutes Reply #7, 19 years, 4 months ago

I have found some issues with 096 and processing logic. Specifically, if I have a snippet call that has a &parameter=`[+placeholder+]`, the [+placeholder+] appears to get replaced/dropped prior to making it to the snippet level for processing. (eForm)

See this post: http://modxcms.com/forums/index.php/topic,14306.msg96707.html#msg96707

Can anyone else confirm on 096 RC3?

Test snippet (name: test) in 096RC3

<?php
echo "'$test1'<hr/>'$test2'";
exit;
?>

Snippet call:

[!test?test1=`test[+test1+]1`&test2=`test[+test2+]2`!]

Expected:

’test[+test1+]1’<hr/>’test[+test2+]2’

Result:

’test1’<hr/>’test2’

Test snippet (name: test) in 0.9.5.1 (rev 2220)

<?php
echo "'$test1'<hr/>'$test2'";
exit;
?>

Snippet call:

[!test?test1=`test[+test1+]1`&test2=`test[+test2+]2`!]

Expected:

’test[+test1+]1’<hr/>’test[+test2+]2’

Result:

’test[+test1+]1’<hr/>’test[+test2+]2’

0.9.5[.1] is giving the expected result... Did anything change that would cause this in 0.9.6 ?

Mike Reid - www.pixelchutes.com
MODx Ambassador / Contributor
[Module] MultiMedia Manager / [Module] SiteSearch / [Snippet] DocPassword / [Plugin] EditArea / We support FoxyCart
________________________________
Where every pixel matters.

☆ A M B ☆
1,056 Posts

pixelchutes Reply #8, 19 years, 4 months ago

UPDATE:

I have commented out the PH section from stripTags() in document parsers:



    # Remove unwanted html tags and snippet, settings and tags

    function stripTags($html, $allowed= "") {

        $t= strip_tags($html, $allowed);

        $t= preg_replace(’~\[\*(.*?)\*\]~’, "", $t); //tv

        $t= preg_replace(’~\[\[(.*?)\]\]~’, "", $t); //snippet

        $t= preg_replace(’~\[\!(.*?)\!\]~’, "", $t); //snippet

        $t= preg_replace(’~\[\((.*?)\)\]~’, "", $t); //settings

        //$t= preg_replace(’~\[\+(.*?)\+\]~’, "", $t); //placeholders

        $t= preg_replace(’~{{(.*?)}}~’, "", $t); //chunks

        return $t;

    }

However, it didn’t appear to have corrected the issue...Hmm...

Mike Reid - www.pixelchutes.com
MODx Ambassador / Contributor
[Module] MultiMedia Manager / [Module] SiteSearch / [Snippet] DocPassword / [Plugin] EditArea / We support FoxyCart
________________________________
Where every pixel matters.

MODX Staff
10,725 Posts

opengeek Reply #9, 19 years, 4 months ago

It was just a guess... next theory...

Jason Coward
Chief Architect @ MODX
http://www.jasoncoward.com | http://twitter.com/drumshaman | https://github.com/opengeek

☆ A M B ☆
1,056 Posts

pixelchutes Reply #10, 19 years, 4 months ago

Well, I can confirm is was AFTER rev.2220...

Mike Reid - www.pixelchutes.com
MODx Ambassador / Contributor
[Module] MultiMedia Manager / [Module] SiteSearch / [Snippet] DocPassword / [Plugin] EditArea / We support FoxyCart
________________________________
Where every pixel matters.