Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

admin, product view, concatenated paths / double slashes in url / so file not found error #220

Closed
torvista opened this issue Feb 23, 2021 · 4 comments

Comments

@torvista
Copy link
Contributor

torvista commented Feb 23, 2021

After removing the silence operators.
On each page refresh, this warning due to the concatenating of constants.

PHP Warning: getimagesize(D:/FullPathToShop/public_html/tienda-158//tienda-158/bmz_cache/ventureshield/generic_full_kitjpg.image.80x80.jpg): Failed to open stream: No such file or directory in D:\FullPathToShop\public_html\tienda-158\admin\image_handler.php on line 571.

In my PR'ed file this is

            case 'layout_info':
                [$width, $height] = getimagesize(DIR_FS_CATALOG . $selected_image_file);
                $heading[] = [
                    'text' => '<strong>' . TEXT_INFO_IMAGE_INFO . '</strong>'
                ];

As $selected_image_file traces back to the class....I'll step further back.

@lat9
Copy link
Collaborator

lat9 commented Feb 24, 2021

I'm having a hard time replicating this. What action is being performed when the warning is logged?

@drbyte
Copy link

drbyte commented Feb 24, 2021

Given that the path is repeated, is it a DIR_WS_ZZZZZ or other define issue?

@lat9
Copy link
Collaborator

lat9 commented Feb 24, 2021

I'm having a hard time replicating this. What action is being performed when the warning is logged?

Note to self: It helps to make edits to the directory that you're testing on.

@lat9
Copy link
Collaborator

lat9 commented Feb 24, 2021

Given that the path is repeated, is it a DIR_WS_ZZZZZ or other define issue?

I've replicated it now; it's the fact that the $selected_image_file is prepended with DIR_WS_CATALOG.

@lat9 lat9 added the bug label Feb 25, 2021
@lat9 lat9 added this to the v5.1.10 milestone Feb 25, 2021
lat9 added a commit to lat9/zen_Image-Handler that referenced this issue Feb 25, 2021
lat9 added a commit that referenced this issue Feb 25, 2021
#220: Correcting missing file on admin sidebox display
@lat9 lat9 modified the milestones: v5.1.10, v5.1.11 Feb 25, 2021
@lat9 lat9 closed this as completed Mar 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants