News:

SMF 2.1.4 has been released! Take it for a spin! Read more.

Main Menu

ColorizePost

Started by Bulakbol, January 15, 2008, 09:45:48 PM

Previous topic - Next topic

gbsothere

Hi,

I'm getting this error in my forum:

8: Undefined index: font_size
File: /var/www/vhosts/mywebsite.com/httpdocs/smf/Themes/default/PersonalMessage.template.php (folder sub template - eval?)
Line: 415


Line 415 is this bit of code for ColorizePost:

echo '<div class="post" style="color: '. $color. '; font-family: ', $message['member']['font_face'], ';  font-size: ', $message['member']['font_size'], ';">', $message['body'], '</div></div>





So......   font size?   The mod works fine, but the forum fills up with errors related to PMs.  Nothing to worry about?  Anything to edit?


Thanks.

My apologies, but I am taking a break from accepting PM requests for support.  If I am not currently assisting you, please do not ask as long as this notice is posted.  Thank you.

I Don't Want To Grow Old Alone


It has been proven that Steely Dan reduces the occurrence of road rage, according to an independent study.



A reminder about admin / ftp passwords etc.

Bulakbol

Quote from: gbsothere on August 01, 2009, 12:19:38 AM
echo '<div class="post" style="color: '. $color. '; font-family: ', $message['member']['font_face'], ';  font-size: ', $message['member']['font_size'], ';">', $message['body'], '</div></div>


Change that line with this to get rid of the error. Thanks for trying/using this mod.
echo '<div class="post" style="color: '. $color. '; font-family: ', $message['member']['font_face'], ';  font-size: ', !empty($message['member']['font_size']) ? $message['member']['font_size'] : '', ';">', $message['body'], '</div></div>
Please do not PM me for support · My Mods and Theme · SMF Coding Guidelines · Modifications Approval Guidelines

gbsothere

That was fast!   :)  I must have caught you working. 


We love this mod; we wouldn't give it up for anything.


Thanks so much for the fix!





My apologies, but I am taking a break from accepting PM requests for support.  If I am not currently assisting you, please do not ask as long as this notice is posted.  Thank you.

I Don't Want To Grow Old Alone


It has been proven that Steely Dan reduces the occurrence of road rage, according to an independent study.



A reminder about admin / ftp passwords etc.

gbsothere

I'm sorry, I'm afraid the exact same error is back today.   Same file, same line error. 


Is there anything else I can do? 







My apologies, but I am taking a break from accepting PM requests for support.  If I am not currently assisting you, please do not ask as long as this notice is posted.  Thank you.

I Don't Want To Grow Old Alone


It has been proven that Steely Dan reduces the occurrence of road rage, according to an independent study.



A reminder about admin / ftp passwords etc.

Bulakbol

gbsothere
Sorry but the edit I gave you should suppress the error. Make sure you edited the right file and uploaded correctly.  Or attach the file in question to your next post and I'll check it for you.
Please do not PM me for support · My Mods and Theme · SMF Coding Guidelines · Modifications Approval Guidelines

gbsothere

Hi again.   :)

Here's the file and I have SMF 1.1.10/Default (although my forum was either 1.1.8 or 1.1.9 when I installed the mod).  I did a manual install of both the mod and all of my SMF upgrades, as well, since I do have a few mods. 

I do notice on the errors that the URL action is always action=pm or a variation on that, such as index.php?action=pm;f=inbox;l=-1.


Thanks so much! 


ETA:  it was installed when the forum was 1.1.8
My apologies, but I am taking a break from accepting PM requests for support.  If I am not currently assisting you, please do not ask as long as this notice is posted.  Thank you.

I Don't Want To Grow Old Alone


It has been proven that Steely Dan reduces the occurrence of road rage, according to an independent study.



A reminder about admin / ftp passwords etc.

Bulakbol

#166
Sorry. I'll get back to you.

<edit>
Can you attach your Sources/Profile.php please. It should not give you problem like this.
Please do not PM me for support · My Mods and Theme · SMF Coding Guidelines · Modifications Approval Guidelines

gbsothere

Here it is and thanks, again, for taking the time to check this for me.   :)
My apologies, but I am taking a break from accepting PM requests for support.  If I am not currently assisting you, please do not ask as long as this notice is posted.  Thank you.

I Don't Want To Grow Old Alone


It has been proven that Steely Dan reduces the occurrence of road rage, according to an independent study.



A reminder about admin / ftp passwords etc.

Bulakbol

@gbsothere
The mod was installed properly. I cannot find the problem. I guess I have to check your Sources/Load.php now.
Please do not PM me for support · My Mods and Theme · SMF Coding Guidelines · Modifications Approval Guidelines

gbsothere

Here it is, but, if nothing becomes apparent, I'll just let it go, since the mod is performing beautifully.  (I may actually have all your mods, come to think of it.)   :)

I'm beginning to wonder if the subsequent upgrades from 1.1.8 to 1.1.9 and on to 1.1.10 might have had something to do with this, although I did run the upgrade.php. 

My apologies, but I am taking a break from accepting PM requests for support.  If I am not currently assisting you, please do not ask as long as this notice is posted.  Thank you.

I Don't Want To Grow Old Alone


It has been proven that Steely Dan reduces the occurrence of road rage, according to an independent study.



A reminder about admin / ftp passwords etc.

Bulakbol

@gbsothere
Sorry, I failed to find the problem. The mod was perfectly installed in all the files that I asked to check. How would you like to remove the font_size in PersonalMessage.template.php? Change the line I previously gave you with this to get rid of the error. I'll check the code again when I have enough time.

echo '<div class="post" style="color: '. $color. '; font-family: ', $message['member']['font_face'], ';">', $message['body'], '</div></div>
Please do not PM me for support · My Mods and Theme · SMF Coding Guidelines · Modifications Approval Guidelines

gbsothere

I'll do that on my test forum and do quite a bit of posting and personal message testing this evening, since I notice that I'm getting the same errors there as I am on my main forum.   :)


Thank you so much for taking the time to check!  We're big fans of your mods on my forums.   :)
My apologies, but I am taking a break from accepting PM requests for support.  If I am not currently assisting you, please do not ask as long as this notice is posted.  Thank you.

I Don't Want To Grow Old Alone


It has been proven that Steely Dan reduces the occurrence of road rage, according to an independent study.



A reminder about admin / ftp passwords etc.

Bulakbol

No problem. Thank you too. I'll check my codes. Maybe the upgrade affected some codes that I am not /wasn't aware of.
Please do not PM me for support · My Mods and Theme · SMF Coding Guidelines · Modifications Approval Guidelines

gbsothere

#173
Ha!  I just tried this on my test forum and sent a PM to another test account and got 8 of those errors.   :D 

I really am starting to think it's the upgrade. 

Still... the mod is performing just fine, so I'll clear out those logs and just wait, in case you discover something in the future about this.   No urgency and thank you again. 


EDIT:   well....   font_face this time, instead of font_size. 

8: Undefined index: font_face
File: /var/www/vhosts/mywebsite.com/httpdocs/smf/Themes/default/PersonalMessage.template.php (folder sub template - eval?)
Line: 415


:D



I really am okay to wait, until you have more time for this.  I know you have a lot of other things going on. 
My apologies, but I am taking a break from accepting PM requests for support.  If I am not currently assisting you, please do not ask as long as this notice is posted.  Thank you.

I Don't Want To Grow Old Alone


It has been proven that Steely Dan reduces the occurrence of road rage, according to an independent study.



A reminder about admin / ftp passwords etc.

Bulakbol

@gbsothere
Sorry. I went and check my mod but empty handed. I think you are right. I will check if it conflicts with other mods. Thanks for the info.
Please do not PM me for support · My Mods and Theme · SMF Coding Guidelines · Modifications Approval Guidelines

flamechica

Bulakbol,
I noticed you have been replying on this thread....any time to check this yet??? I would really appreciate it. Thanks!!!

Quote from: flamechica on May 15, 2009, 02:55:02 PM
I am using SMF 1.1.8 & downloaded the first version on this mod (because I did not like the drop down color choices. I wanted to give my members the option of using hex codes). How would I go about removing the part of this mod that makes it so the poster and the reader have to be using the same theme for this to work? I use the Highlight Message Backgrounds mod and made it so every theme displays the same light gray background so that all font colors are possible to read no matter what theme a member is using. So, this part of this mod is useless to me. Thanks in advance for your help. This is a great mod.

Bulakbol

@flamechica
I was thinking about making two versions after updating all my mods. It is not easy but will be done. I haven't found enough time and inclination yet. I'll let you know. :)
Please do not PM me for support · My Mods and Theme · SMF Coding Guidelines · Modifications Approval Guidelines

ralliart

Hi bulakbol, pinoy? Will this work for 2.0 RC1.2? Thanks!

Bulakbol

@ralliart
It should work kabayan. Please let me know if it doesn't work.
Please do not PM me for support · My Mods and Theme · SMF Coding Guidelines · Modifications Approval Guidelines

flamechica

Quote from: Bulakbol on August 03, 2009, 05:40:23 AM
@flamechica
I was thinking about making two versions after updating all my mods. It is not easy but will be done. I haven't found enough time and inclination yet. I'll let you know. :)

Ok, thanks. Sorry for being a pest. I've got some members that really want it and they have been pestering me.

Advertisement: