Topic: Share this topic SMF Mod

[Branko.]

Sharethis::ShareThisWho(), is part of Share This by Suki... 

Code: [Select]

<file name="$sourcedir/Who.php">
<operation>
<search position="replace"><![CDATA[ 'mods' => array(]]></search>
<add><![CDATA[ 'mods' => array(
Sharethis::ShareThisWho(),]]></add>
</operation>
</file>
$txt['subjectcopy'] is part of ChangeAllSubjects by runic

Code: [Select]

/file>
<file name="$sourcedir/Who.php">
<operation>
<search position="replace" whitespace="loose"><![CDATA[ 'mods' => array(]]></search>
<add><![CDATA[ 'mods' => array( $txt['subjectcopy'],]]></add>
</operation>
</file>
All is related with copyright

[Arantor]

That's where I was going I just didn't recognise which mod had subjectcopy in it...

[Kimmie]

Ok so the error is coming from the Change All Subjects mod? I had that one, and decided to uninstall it and go with the "Edit All Message Titles" mod. It looks like the uninstall left this code even though I didn't get any test fails when I uninstalled it. I should just be able to remove this line right?

 $txt['subjectcopy']


Asking before hand as I don't want to mess anything up. This is the parse code for it. I need to also remove the , at the end right?

[Branko.]

The reverse action
Find

Code: [Select]

'mods' => array( $txt['subjectcopy'],
replace with

Code: [Select]

'mods' => array(

[Kimmie]

The reverse action
Find

Code: [Select]

'mods' => array( $txt['subjectcopy'],
replace with

Code: [Select]

'mods' => array(

That doesn't work anymore because there are two other mod codes in between that. I just wasn't sure about the comma at the end

Code: [Select]

*/
'mods' => array(
SaShop_Who(),
Sharethis::ShareThisWho(), $txt['subjectcopy'],
),

[Arantor]

The comma at the end is fine, you just need to ensure you don't have extra commas with nothing in them.

This would be bad:

Code: [Select]

*/
'mods' => array(
SaShop_Who(),
Sharethis::ShareThisWho(), ,
),
But if you're removing the mod, this would be correct:

Code: [Select]

*/
'mods' => array(
SaShop_Who(),
Sharethis::ShareThisWho(),
),

[Kimmie]

Thanks Arantor. That is what I needed to know