Results 1 to 4 of 4
  1. #1
    Join Date
    Sep 2009
    Location
    Stuart, FL
    Posts
    12,488
    Plugin Contributions
    88

    Default Has there ever been a LARGE_IMAGE_WIDTH (and _HEIGHT) constant?

    The title pretty much says/asks it all. I'm looking at the changes to the main-image and additional-images' handling for IH4 and Zen Colorbox, both of which are quite convinced that such a constant exists; it's not there inspecting the database via phpMyAdmin.

  2. #2
    Join Date
    Jul 2012
    Posts
    16,733
    Plugin Contributions
    17

    Default Re: Has there ever been a LARGE_IMAGE_WIDTH (and _HEIGHT) constant?

    Quote Originally Posted by lat9 View Post
    The title pretty much says/asks it all. I'm looking at the changes to the main-image and additional-images' handling for IH4 and Zen Colorbox, both of which are quite convinced that such a constant exists; it's not there inspecting the database via phpMyAdmin.
    So, ZC 1.5.1 and ZC 1.5.4 (assuming those in between as well) include LARGE_IMAGE_WIDTH in the admin/includes/modules/update_product.php file below the comment that the information is being assigned to support future image handler code...

    Even in that file, $large_width and $large_height (previously assigned to the LARGE_IMAGE_WIDTH and height constants) are overwritten with values returned from getimagesize($src), so the constant is not even used in that file.

    Not sure if the thought is that an image handler (perhaps not that used by IH4 or Zen Colorbox) would assign the LARGE_IMAGE_WIDTH (and height) during processing, but haven't seen anything that assigns a value to either LARGE_IMAGE_WIDTH or LARGE_IMAGE_HEIGHT.

    In fact IH4 it appears that the code almost needs there not to be a defined value in order to return the original size of the image (not to resize the image). Makes me wonder about system response (using IH4 for example) if a value *is* provided... Will/could it increase the image(s) size if the size constant was assigned? Would it fail?

    Overall yeah a good question, what I have seen as far as usage of the constant though is: If there were a LARGE_IMAGE_WIDTH/LARGE_IMAGE_HEIGHT constant assigned, when using the handle_image function, then that value should be sent to the handler in that area of code because one would expect to see the image at LARGE_IMAGE_WIDTH and LARGE_IMAGE_HEIGHT. As if a remnant of properly using a constant that isn't assigned awaiting the "time" when it is assigned and the system "properly" responds.
    ZC Installation/Maintenance Support <- Site
    Contribution for contributions welcome...

  3. #3
    Join Date
    Jan 2004
    Posts
    66,373
    Blog Entries
    7
    Plugin Contributions
    274

    Default Re: Has there ever been a LARGE_IMAGE_WIDTH (and _HEIGHT) constant?

    Quote Originally Posted by lat9 View Post
    The title pretty much says/asks it all. I'm looking at the changes to the main-image and additional-images' handling for IH4 and Zen Colorbox, both of which are quite convinced that such a constant exists; it's not there inspecting the database via phpMyAdmin.
    I'm pretty sure there hasn't been before v160.
    But in v160 the code does expect it, kinda ... although I think it needs some dependencies fixed (like constants defined): https://github.com/zencart/zencart/b...images.php#L89

    But ... what's the background behind your question? Are you wanting to add them to IH?
    .

    Zen Cart - putting the dream of business ownership within reach of anyone!
    Donate to: DrByte directly or to the Zen Cart team as a whole

    Remember: Any code suggestions you see here are merely suggestions. You assume full responsibility for your use of any such suggestions, including any impact ANY alterations you make to your site may have on your PCI compliance.
    Furthermore, any advice you see here about PCI matters is merely an opinion, and should not be relied upon as "official". Official PCI information should be obtained from the PCI Security Council directly or from one of their authorized Assessors.

  4. #4
    Join Date
    Sep 2009
    Location
    Stuart, FL
    Posts
    12,488
    Plugin Contributions
    88

    Default Re: Has there ever been a LARGE_IMAGE_WIDTH (and _HEIGHT) constant?

    Quote Originally Posted by DrByte View Post
    I'm pretty sure there hasn't been before v160.
    But in v160 the code does expect it, kinda ... although I think it needs some dependencies fixed (like constants defined): https://github.com/zencart/zencart/b...images.php#L89

    But ... what's the background behind your question? Are you wanting to add them to IH?
    The background is that I was looking to change the additional_images.php module for a client who's running ZC 1.5.4, noticed the use of those constants and then wondered where they were defined.

 

 

Similar Threads

  1. Is there a plugin to show how many times a product has been viewed?
    By Zadris in forum All Other Contributions/Addons
    Replies: 2
    Last Post: 4 Jun 2012, 09:57 PM
  2. There has been an error processing your credit card. secpay
    By Husky570 in forum Addon Payment Modules
    Replies: 1
    Last Post: 7 Jun 2010, 10:54 AM
  3. There has been an error processing your credit card.
    By kevinatbrough in forum Built-in Shipping and Payment Modules
    Replies: 6
    Last Post: 1 Oct 2007, 10:18 PM
  4. There has been an error processing your credit card
    By zb21hay in forum Built-in Shipping and Payment Modules
    Replies: 2
    Last Post: 21 Oct 2006, 04:04 AM

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  
disjunctive-egg
Zen-Cart, Internet Selling Services, Klamath Falls, OR