error bij sommige avatars in de avatar gallerij

[stomaatje]

• Adres van je forum: http://www.stomaforum.nl
  Event. modificaties op je forum: veel incl no dublicate avatar
  Wanneer ontstond het probleem? geen idee! wordt er nu op gewezen door een lid
  phpBB versie: 3.0.6
  
  
  Heb je onlangs iets veranderd aan je forum?
  Wat is het probleem?

Je wilt een avatar uit de gallerij hebben. klikt op het bolletje om hem te selecteren en druk op bevestig onderaan. bij sommige krijg je dan dit te zien:



volgens mij is het alleen bij de plaatjes die wat tussen ruimte in de naam hebben, maar ik heb ze niet allemaal geprobeerd. is dit het probleem? zo ja waarom nu ineens? of is het iets anders?

[Jim]

Die wat ruimte tussen de naam hebben? Je bedoeld dat er een spatie in de naam staat?

[stomaatje]

ja inderdaad met een spatie ertussen

[Jim]

Is het dan een idee om die avatars zo te hernoemen dat er geen spatie meer inzit?

[stomaatje]

Dat zal dan wel betekenen dat de mensen die die avatar gebruiken ineens hun avatar kwijt zijn.... waarom komt ineens die error opzetten terwijl er eerst niks mee aan de hand was?

suus

[Jeroen]

Er kunnen simpelweg geen spaties in dat soort bestanden staan, hoe het dan kan dat het eerst wel zo was weet ik ook even niet. Maar ik zou ze toch maar gaan hernoemen, en toekomstige bestanden zonder spaties doen.

[AirWarrior]

Maar net wat suus zegt: Het heeft wel altijd gewerkt en na de laatste update niet meer.
Dus er is iets veranderd waardoor het niet meer werkt.
Normaal als je een spatie zet, komt er %20 op de plaats van de spatie.

[EelkeB]

Dat is alleen bij de weergave van een spatie in een URL, als een script in PHP vraagt om een bestand met %20 aan het bestandssysteem, dan zal het bestandssysteem letterlijk naar dat bestand op zoek gaan. Dit soort vertalingsproblemen is eigenlijk de reden dat je gewoon beter spaties kunt vermijden in bestandsnamen, met name op een website. Het zou moeten werken, maar er zitten gewoon veel haken en ogen aan.

[AirWarrior]

Ja dat klopt, Als je de eigenschappen van de image opvraagt staat er in de url op de plek van de spatie %20

[EelkeB]

Ik denk dat ik de oorzaak van je probleem heb gevonden. In deze revisie is een wijziging geïntroduceerd waarbij bij het weergeven van de avatar-gallery de URL van de afbeeldingen door een urlencode heen gehaald wordt: http://github.com/phpbb/phpbb3/commit/8 ... 08b19aeae9

Het lijkt er echter op dat die waarde vervolgens niet weer ge-urldecode wordt als de afmetingen worden gecheckt van het bestand dat lokaal staat. Alleen wel vreemd dat dit dan niet eerder is ontdekt, want het lijkt toch of dit direct duidelijk moet worden als je zo'n avatar met een spatie uitkiest.

Je kunt het volgende proberen.

Open includes/functions_user.php

Zoek

Code: Selecteer alles

list($sql_ary['user_avatar_width'], $sql_ary['user_avatar_height']) = getimagesize($phpbb_root_path . $config['avatar_gallery_path'] . '/' . $category . '/' . $sql_ary['user_avatar']); 

Vervang door:

Code: Selecteer alles

list($sql_ary['user_avatar_width'], $sql_ary['user_avatar_height']) = getimagesize($phpbb_root_path . $config['avatar_gallery_path'] . '/' . $category . '/' . rawurldecode($sql_ary['user_avatar'])); 

Het verschil is de rawurldecode die er tussen zit, aan het eind.

[AirWarrior]

Ik denk dat ik de oorzaak van je probleem heb gevonden. In deze revisie is een wijziging geïntroduceerd waarbij bij het weergeven van de avatar-gallery de URL van de afbeeldingen door een urlencode heen gehaald wordt: http://github.com/phpbb/phpbb3/commit/8 ... 08b19aeae9

Het lijkt er echter op dat die waarde vervolgens niet weer ge-urldecode wordt als de afmetingen worden gecheckt van het bestand dat lokaal staat. Alleen wel vreemd dat dit dan niet eerder is ontdekt, want het lijkt toch of dit direct duidelijk moet worden als je zo'n avatar met een spatie uitkiest.

Je kunt het volgende proberen.

Open includes/functions_user.php

Zoek

Code: Selecteer alles

list($sql_ary['user_avatar_width'], $sql_ary['user_avatar_height']) = getimagesize($phpbb_root_path . $config['avatar_gallery_path'] . '/' . $category . '/' . $sql_ary['user_avatar']); 

Vervang door:

Code: Selecteer alles

list($sql_ary['user_avatar_width'], $sql_ary['user_avatar_height']) = getimagesize($phpbb_root_path . $config['avatar_gallery_path'] . '/' . $category . '/' . rawurldecode($sql_ary['user_avatar'])); 

Het verschil is de rawurldecode die er tussen zit, aan het eind.

Deze code gewijzigd en geen fout melding maar ook geen avatar. Ik heb de avatars in images/avatars/gallery/map1.
Zo heb ik dus 4 mappen in verschillende categoriën staan.
Heb de code iets aangepast en nu wekrt het bij mij wel goed.
Dus met mappen in images/avatars/gallery/

Code: Selecteer alles

			list($sql_ary['user_avatar_width'], $sql_ary['user_avatar_height']) = getimagesize($phpbb_root_path . $config['avatar_gallery_path'] . '/' . $category . '/' . rawurldecode($sql_ary['user_avatar']));
			$sql_ary['user_avatar'] = $category . '/' . $sql_ary['user_avatar'];

Ik vertel het misschien wat omslachtig maar hoop dat je begrijp wat ik bedoel.

[EelkeB]

Wat heb je nu precies nog aangepast om het te laten werken?

[AirWarrior]

Code: Selecteer alles

$sql_ary['user_avatar'] = $category . '/' . $sql_ary['user_avatar'];

Dit stukje er nog een keer achter gezet.

[EelkeB]

Hmm... Vreemd. Zijn er nog andere mensen die de avatar gallery gebruiken met 3.0.7 (-PL1)?

[Kevin]

Ik ja in versie 3.0.7 PL1 en heb nergens last van, het selecteren en toevoegen van een avatar uit een gallery werkt gewoon.

[EelkeB]

Precies.. Dat er iets mis gaat met avatars met een spatie in de bestandsnaam kan ik me wel voorstellen, maar de tweede wijziging van AirWarrior zou impliceren dat het uberhaupt niet werkt, volgens mij.

[Kevin]

Dat kan wel kloppen ja. Heb voor de zekerheid is een naam met een spatie in de gallery geplaatst en voila de volgende error.

Code: Selecteer alles

[phpBB Debug] PHP Notice: in file /includes/functions_user.php  on line 2358: getimagesize(./../images/avatars/gallery/Fish/PDT%20Fish_01.gif) [function.getimagesize]: failed to open stream: No such file or directory

Warning: Cannot modify header information - headers already sent by (output started at /usr/home/aquafora/domains/****.nl/public_html/dev/includes/functions.php:3516) in /usr/home/****/domains/****/public_html/includes/functions.php on line 3566
Keer terug naar de startpagina
Algemene fout
SQL ERROR [ mysql4 ]

Column 'user_avatar_height' cannot be null [1048]

SQL

UPDATE phpbb_users SET user_avatar_type = 3, user_avatar = 'Fish/PDT%20Fish_01.gif', user_avatar_height = NULL, user_avatar_width = NULL WHERE user_id = 34

BACKTRACE

FILE: includes/db/mysql.php
LINE: 174
CALL: dbal->sql_error()

FILE: includes/functions_user.php
LINE: 2439
CALL: dbal_mysql->sql_query()

FILE: includes/acp/acp_users.php
LINE: 1677
CALL: avatar_process_user()

FILE: includes/functions_module.php
LINE: 507
CALL: acp_users->main()

FILE: adm/index.php
LINE: 76
CALL: p_master->load_active()

[AirWarrior]

Precies.. Dat er iets mis gaat met avatars met een spatie in de bestandsnaam kan ik me wel voorstellen, maar de tweede wijziging van AirWarrior zou impliceren dat het uberhaupt niet werkt, volgens mij.

Als ik die wijziging er niet bij zet en ik kies een avatar uit de gallery dan krijg ik een mooi rood kruisje.
Als ik dan op eigenschappen klik staat er: /images/avatar/gallery/naamplaatje terwijl het dan /images/avatar/gallery/map1/naamplaatje moet zijn.
Als ik dus die extra wijziging er inzet dan verwijst ie wel naar de juiste map.


Zo staat bij mij de indeling

[EelkeB]

Hmm... Dus in je avatar directories zitten ook spaties... Ik had er al een bug over aangemaakt, ik zal dat er nog als opmerking bij zetten.

[AirWarrior]

Ik heb je rawurlcode ertussen gezet en nu werkt het weer prima