that would work if the screenshots folder was in the images folder but its in the main directory

Ulminia wrote:addon['dir'] doesn't work.. tried it at home too, it has to be hard-coded for images

Code: Select all

$imgdir = '/components/com_wowroster/addons/RosterGallery/screenshots/';
$imgdirthumb = '/components/com_wowroster/addons/RosterGallery/screenshots/thumbs/';


It is now possible to see the thumbnails of the uploaded images.

It is still not possible to:
1. Display pending images that wait for approval in the RosterGallery configuration menu
2. Display already approved screenshots in the RosterGallery configuration menu
3. Open (download) images when clicking on a thumbnail in the gallery


--> As a result hard-coding line 17 & 18 in index.php fixes only the thumbnail gallery. I guess there are other positions in the index.php or in the admin/index.php that also cause problems (see problems 1 to 3)...

Ulminia wrote:that would work if the screenshots folder was in the images folder but its in the main directory

Why is it in the main directory?
There should be no reason why it cannot be in the addon directory

it is in the addon directory its /addons/RosterGallery/screenshots/..
im getten ready to release the next verson for the 2.0 final roster using some complacated dir structurs to make sure it displays right lol
and my solution

Code: Select all

$imagex = str_replace( 'img/', '', $roster->config['interface_url'] );
$imgdir = $imagex.$addon['url'].'screenshots/';
$imgdirthumb = $imagex.$addon['url'].'screenshots/thumbs/';

Try this

Code: Select all

$imgdir = $addon['url_path'].'screenshots/';
$imgdirthumb = $addon['url_path'].'screenshots/thumbs/';

yup that would work

zanix wrote:Try this

Code: Select all

$imgdir = $addon['url_path'].'screenshots/';
$imgdirthumb = $addon['url_path'].'screenshots/thumbs/';

It is now possible to see the thumbnails of the uploaded images.

It is still not possible to:
1. Display pending images that wait for approval in the RosterGallery configuration menu
2. Display already approved screenshots in the RosterGallery configuration menu
3. Open (download) images when clicking on a thumbnail in the gallery


--> As a result line 17 & 18 in index.php fixes only the thumbnail gallery. I guess there are other positions in the index.php or in the admin/index.php that also cause problems (see problems 1 to 3)...

yes im aware of them not working in the admin section was mor concerned with the main page working

open body_apimage.tpl in the inc folder of RosterGallery and change

$apimgdir = 'addons' . DIR_SEP . 'RosterGallery' . DIR_SEP . 'screenshots' . DIR_SEP . 'thumbs' . DIR_SEP;
to
$apimgdir = $addon['url_path'] . DIR_SEP . 'screenshots' . DIR_SEP . 'thumbs' . DIR_SEP;

same thing with body_uapimage.tpl

$upimgdir = $addon['url_path'] . DIR_SEP . 'screenshots' . DIR_SEP . 'thumbs' . DIR_SEP;

fixes the admin section as well

Wow, you are really fast Ulminia .

Ulminia wrote:yes im aware of them not working in the admin section was mor concerned with the main page working

Yes I know but as I said, the main page is still not working after having modified index.php line 17&18. The thumbnails show up correctly now, but when moving the mouse over a thumbnail, the target URL of the image looks strange in the status line of the browser. As a result one still not open an image when clicking on a thumbnail on the main page.
The URLs looks like these now when moving the mouse over the thumbnails:

Code: Select all

http://bellicosus.be.funpic.de/index.php?option=com_wowroster&Itemid=26&p=util-RosterGallery&id=1\
http://bellicosus.be.funpic.de/index.php?option=com_wowroster&Itemid=26&p=util-RosterGallery&id=2\
http://bellicosus.be.funpic.de/index.php?option=com_wowroster&Itemid=26&p=util-RosterGallery&id=3\

Is it correct that all URLs end with a backslash ?

Ulminia wrote:open body_apimage.tpl in the inc folder of RosterGallery and change

$apimgdir = 'addons' . DIR_SEP . 'RosterGallery' . DIR_SEP . 'screenshots' . DIR_SEP . 'thumbs' . DIR_SEP;
to
$apimgdir = $addon['url_path'] . DIR_SEP . 'screenshots' . DIR_SEP . 'thumbs' . DIR_SEP;

same thing with body_uapimage.tpl

$upimgdir = $addon['url_path'] . DIR_SEP . 'screenshots' . DIR_SEP . 'thumbs' . DIR_SEP;

fixes the admin section as well

Unfortunately this does not work. I changed both TPL files. I can still not see any images (or thumbnails) in the admin section.

first thing im going to ask you is to download and install the lastest verson of RG in the download section here 3.0.3999 this will help some because of code changes i have made

Next after you upgrade what is the url of the images in the admin section?

Ulminia wrote:first thing im going to ask you is to download and install the lastest verson of RG in the download section here 3.0.3999 this will help some because of code changes i have made

Next after you upgrade what is the url of the images in the admin section?

I upgraded now to Build 3999 and then uploaded one test image but now I get an SQL error in the admin section:

Code: Select all

146: Table 'bellicosus.roster_addon_config' doesn't exist
SQL:
SELECT `config_name`, `config_value` FROM `roster_addon_config` WHERE `addon_id` = '13' ORDER BY `id` ASC;
File: lib/dbal/mysql.php
Line: 234

If you want I can give you the admin acount for the website... .

that will be ok ive noticed some coding errors i need 5 minits and ial have 3.0.4100 in the downloads section

ok 3.0.4100 is up ial be back thjisafternoon with more updates

Ulminia, thanks for Build 4100. I still get the same SQL error in the admin section. I have the feeling that there goes something wrong during the installation of the addon and that the tables are not correctly prepared on the SQL server ?
Ulminia, I am sending you my admin account data now .

Do not use DIR_SEP for url paths, always use a forward slash /
Otherwise users on windows system are going to have broken image paths

Only use DIR_SEP for file paths