Page 2 of 3

Re: Discuss: MODX 1.2.3 Released

Posted: Thu Oct 08, 2009 8:18 pm
by igorw
Sure. Instead of <![CDATA you just have to use <![CDATA

Re: Discuss: MODX 1.2.3 Released

Posted: Thu Oct 08, 2009 9:13 pm
by Helter Skelter
Thank you for the quick reply! I assume that the replacement will function the same in the overall_header because it is an html file? Seems to work fine atm.

Re: Discuss: MODX 1.2.3 Released

Posted: Thu Oct 08, 2009 11:35 pm
by ToonArmy
Helter Skelter wrote:Thank you for the quick reply! I assume that the replacement will function the same in the overall_header because it is an html file? Seems to work fine atm.
It shouldn't be in the file literally, you should copy the rendered CDATA without the HTML escaping.

Re: Discuss: MODX 1.2.3 Released

Posted: Sun Oct 11, 2009 8:48 am
by bx67212
there is any way to create module via modx?

and there is any way to setconfig() via modx?

:)

Re: Discuss: MODX 1.2.3 Released

Posted: Sun Oct 11, 2009 1:17 pm
by A_Jelly_Doughnut
No, there is not a way to do either of those functions.

Re: Discuss: MODX 1.2.3 Released

Posted: Sun Oct 11, 2009 1:27 pm
by igorw
You should be looking at UMIL instead: http://www.phpbb.com/mods/umil/

Re: Discuss: MODX 1.2.3 Released

Posted: Fri Nov 06, 2009 4:02 am
by leviatan21
It's possible to warn users not use a regular text editor while install a MOD by hand ?
look what's happens today in one of my MOD : RSS Feed 2.0
The user completely lose a file.

And this is a very common problem

Thanks !

Re: Discuss: MODX 1.2.3 Released

Posted: Fri Nov 06, 2009 4:11 am
by tumba25
You can put that in the Author notes if you want.

Re: Discuss: MODX 1.2.3 Released

Posted: Fri Nov 06, 2009 4:26 am
by leviatan21
tumba25 wrote:You can put that in the Author notes if you want.
Of course I can do it from now, I just want to contribute...
You should know this is a very common problem, mostly for newbie 8-)

IMHO the user is right, this should be a common note for all.
I was thinking add a note in the Disclaimer & Other Notes

Re: Discuss: MODX 1.2.3 Released

Posted: Sat Nov 07, 2009 9:14 pm
by Pony99CA
leviatan21 wrote:
tumba25 wrote:You can put that in the Author notes if you want.
Of course I can do it from now, I just want to contribute...
You should know this is a very common problem, mostly for newbie 8-)

IMHO the user is right, this should be a common note for all.
I was thinking add a note in the Disclaimer & Other Notes
That certainly sounds reasonable to me. I know I messed something up once trying to use Windows Notepad because of the requirements for editing text files (UTF-8 with no BOM or something most newbies -- or Windows users -- wouldn't understand).

Steve

Re: Discuss: MODX 1.2.3 Released

Posted: Fri Nov 13, 2009 2:56 pm
by leviatan21
I need to ask this one more time, here because I see some discrepancies :
According the sample-30-123.xml file, in the link-group tag, extra .xml files for templates should go directly in the same folder as the main install.xml
If I do not understand it in the wrong way, extra .xml template files should go in the contrib folder for AutoMOD ?

Thanks

Re: Discuss: MODX 1.2.3 Released

Posted: Fri Nov 13, 2009 5:58 pm
by igorw
Our policy states:
All changes to the phpBB core files should be placed in the install.xml file. Changes for non-English languages and non-prosilver templates should not be placed in this file. These should have a separate XML file, but should be linked to from the main install file using the link tag.

(...)

Optional install files or instructions should be placed within the contrib directory. Additional required instructions should be placed in the root directory or in a directory called named xml/.
So you can put it in either the root, in contrib/ or in xml/.

Re: Discuss: MODX 1.2.3 Released

Posted: Fri Nov 13, 2009 6:07 pm
by leviatan21
eviL<3 wrote:So you can put it in either the root, in contrib/ or in xml/.
Thanks :D

Re: Discuss: MODX 1.2.3 Released

Posted: Sat Nov 14, 2009 1:55 pm
by Sajaki
would you consider adding conditional statements in MODX ?

currently this find action will fail if the string isn't found. but i think it would be nice to be able to add an if-else construct to this.

Code: Select all

<find><![CDATA[//@define('EMED_MYMOD', true);]]></find>
for example :

Code: Select all

<edit>
<condition>
<find><![CDATA[//@define('EMED_MYMOD', true);]]></find>
<then>
 <action>action type</action>
</then>
<else>
 <action>another action type</action>
</else>
</condition>
</edit>

Re: Discuss: MODX 1.2.3 Released

Posted: Sat Nov 14, 2009 2:02 pm
by igorw
It's an interesting thought but it would make MODX extremely complex and wouldn't give enough benefit. MODX is designed as a diff system. Logic really doesn't belong into there.