Suitecrm: Change AOS Product Images to use an entry point

Created on 13 Mar 2015 · 9Comments · Source: salesagility/SuiteCRM

Product images currently links directly to the image in the upload folder. This only works if SuiteCRM is configured poorly and the upload folder is accessible (i.e. the .htaccess files are being ignored). We should change the product image code to refer to the image via the download entry point.

Important Fix Proposed Bug

Source

JimMackin

All 9 comments

has this been fixed? It looks to me like the code _is_ referring to the image via the download entry point, but still getting 403 error and image won't show. (Suite 7.7.4)

ghost on 28 Sep 2016

@EZS-JD , "Fix Proposed" label means a pull request has been submit and need to be reviewed before be pulled in.

shogunpol on 29 Sep 2016

still not fixed...

aryeharmon on 25 Oct 2017

😕3

@samus-aran @Dillon-Brown i think will great if we will see that will in new relase 7.10.12

When we will have assigned milestones to tickets, we as community memebers can better expect what will come in next relases :)

What do you think about it?

Mausino on 19 Dec 2018

Hey @Mausino This has been merged indicated by the label "Resolved Next Release" Which was part of the PR Party :tada:
Regards to setting up milestones we are doing some housekeeping from the event and will be assigning all the issues that have this label "Resolved Next Release" to the 7.10.12 milestone :+1:

samus-aran on 19 Dec 2018

👍1

@samus-aran

will be assigning all the issues that have this label "Resolved Next Release" to the 7.10.12 milestone 👍

Hi, should all "Resolved Next Relase" get the 7.10.12 milestone ? Only we wont forget on them.

Mausino on 27 Dec 2018

Hey @Dillon-Brown @samus-aran from this issue thread it looks like this issue was fixed in 7.10.12, however the issue still appears to be present in 7.11.5. Is the fix going to be merged into 7.11.x too?

Many thanks,
Andy

babington-andy on 10 Jul 2019

Hi @babington-andy, I've created a duplicate PR for this here: #7539. It will need to be re-assessed and code-reviewed before getting merged into the next 7.10.x and 7.11.x release.