Adding text to additional images individually

[gjh42]

The $file variable holds the filename only; $base_image has the subfolder (if any) plus filename.This code will handle any file (as long as it doesn't have spaces), and put the text above or below the image (depending on the value of a constant ADDIMG_TEXT_TOP which should be set in the definitions file):

    // additional image titles
    $badchars = array('/','\\','.','-',' ');//not allowed in constant names
    // if any other characters could be used in filepath, add them to $badchars
    $add_const_name = 'ADDIMG_' . strtoupper(str_replace($badchars,'_',$base_image));
    if (defined($add_const_name)) {//add text above or below images
      $link = (ADDIMG_TEXT_TOP)? '<div class="addImgTitle">' . constant($add_const_name) . '</div>' . $link: $link . '<div class="addImgTitle">' . constant($add_const_name) . '</div>';
    }
    // /additional image titles
    // List Box array generation: 

Add to the definition file:

define ('ADDIMG_TEXT_TOP','true'};//change to false to put text below images 

The $badchars setup is intended to convert spaces, but apparently is unable to do so. It will handle / \ . - characters.

[palomapottery]

Just because Microsoft lets users make file/folder names with spaces doesn't mean it's a good idea...
It is a universal computer best practice to use only letters, numbers, _, -, . and perhaps one or two other characters in filenames. Keeping letters lower case is also a good idea, as some applications are case sensitive and forgetting what you capitalized can cause unexpected problems.

Thank you for your suggestions and info on naming images and filenames. Much appreciated! I normally try not to use spaces. From now on I will be sure to follow that rule more rigorously. Thanks!

[gjh42]

A bit more study revealed that $base_image has "images/" plus the subfolder (if any) plus filename. This requires a bit more processing, as shown:

    $add_const_name = 'ADDIMG_' . strtoupper(str_replace($badchars,'_',ereg_replace('(^images/)','',$base_image)));

    $add_const_name = 'ADDIMG' . strtoupper(ereg_replace('(^images/)[/\\.-]','_',$base_image)));//test if this works 
    echo $base_image . $add_const_name; 

The first version just adds another function to strip "images/" from the front of the string; the second version rolls up all of the replacement into one function.

Please try the second version so I can get more verification that it works.

PS -
I really like your site (I'm a potter too.) I wish you would put the "powered by Zen Cart" link back into the footer so you can add it to the showcase as an example of what can really be done with the platform.

[gjh42]

After local testing, here is the final version of the additional image titles code:

    // additional image titles
    $add_img_const = 'ADDIMG_' . strtoupper(ereg_replace('[/\\.-]','_',substr_replace($base_image,'',0,7)));// if any other characters could be used in filepath, add them to list [  ]
    if (defined($add_img_const)) {//add text above or below images
      $link = (ADDIMG_TEXT_TOP)? '<h4 class="addImgTitle">' . constant($add_img_const) . '</h4>' . $link: $link . '<h4 class="addImgTitle">' . constant($add_img_const) . '</h4>';
    }
    // /additional image titles 

I would still appreciate a field tester before submitting the mod.

[gjh42]

Additional Image Titles is now available in Free Addons.

See the support thread for questions and further help.

[lat9]

Wow! This installed like a dream. Thank you for the contribution!

[524cw]

Just because Microsoft lets users make file/folder names with spaces doesn't mean it's a good idea...
It is a universal computer best practice to use only letters, numbers, _, -, . and perhaps one or two other characters in filenames. Keeping letters lower case is also a good idea, as some applications are case sensitive and forgetting what you capitalized can cause unexpected problems.

They should teach this in public schools. Along with "why you can't make digital pictures bigger".