Witam, mam taki oto skrypt na zmianę wielkości obrazka:

[PHP] pobierz, plaintext 
    function avatar_resizer($avatar_link, $avatardimensions="80|80")
    {
		$rozmiar = getsizeimage($avatar_link);
 
		if($avatar_link[0] > 80 && $avatar_link[1] > 80) {
 
 
                    $avatar_link = htmlspecialchars_uni($avatar_link);
                    $avatar_dimensions = explode("|", $avatardimensions);
                    if($avatar_dimensions[0] && $avatar_dimensions[1])
                    {
                            $avatar_width_height = "width=\"{$avatar_dimensions[0]}\" height=\"{$avatar_dimensions[1]}\"";
                    }
                    $avatar = "<img src=\"{$avatar_link}\" alt=\"\" $avatar_width_height />";
            }
            else
            {
                    $avatar = '';
            }
 
 
            return $avatar;
		}                                               
    }
[PHP] pobierz, plaintext 

Działał, dopóki nie dodałem instrukcji if, która sprawdza czy obrazek ma już 80x80, a jeżeli ma to przejść do końca funkcji.

Teraz wywala błąd: Parse error: syntax error, unexpected '}' in /virtual/w/o/workdomain.ugu.pl/global.php on line 547

Linijka 547 to ostatnia klamra zamykająca.

Witam.

Jakiego IDE używasz?

Usuń ostatnią klamrę nie będę złośliwy

Na to to ja też wpadłem Ale wtesy znowóż wyskakuje:

Fatal error: Call to undefined function getsizeimage() in /virtual/w/o/workdomain.ugu.pl/global.php on line 526

Klamer jest za dużo, ale i usunięcie tej ostatniej nie pomoże. W instrukcji warunkowej if posługujesz się zmienną $avatar_link, a powinieneś chyba użyć zmiennej $rozmiar.

A funkcja nazywa się getimagesize().

EDIT: Poza tym wydaje mi się, że obraz powinien być skalowany proporcjonalnie, bo inaczej będzie zniekształcony.

Okej, zmieniłem na getimagesize() oraz $avatar_link na $rozmiar, lecz teraz wyskoczyło ostrzeżenie:

Warning [2] getimagesize(images/avatars/php.gif?dateline=1325777488) [function.getimagesize]: failed to open stream: No such file or directory - Line: 526 - File: global.php PHP 5.2.17-ugu1 (Linux)

Wyniakłoby z niego że nie ma takiego obrazku, jaki chcę mieć w avatarze, ale sprawdziałem, i jest.

Tutaj obecna funkcja:

[PHP] pobierz, plaintext 
function avatar_resizer($avatar_link, $avatardimensions="80|80")
    {
		$rozmiar = getimagesize($avatar_link);
 
		if($rozmiar[0] > 80 && $rozmiar[1] > 80) {
 
 
                    $avatar_link = htmlspecialchars_uni($avatar_link);
                    $avatar_dimensions = explode("|", $avatardimensions);
                    if($avatar_dimensions[0] && $avatar_dimensions[1])
                    {
                            $avatar_width_height = "width=\"{$avatar_dimensions[0]}\" height=\"{$avatar_dimensions[1]}\"";
                    }
                    $avatar = "<img src=\"{$avatar_link}\" alt=\"\" $avatar_width_height />";
            }
            else
            {
                    $avatar = '';
            }
 
 
            return $avatar;
		}                                               
 
 
$avatar = avatar_resizer($mybb->user['avatar']);
[PHP] pobierz, plaintext 

Funkcja jako argument $avatar_link przyjmuje adres obrazka, a nie adres pliku PHP, który ten obrazek generuje (chyba).

No ale w takim razie funkcja powinna wygenerować to ostrzeżenie również jak nie było żadnej modyfikacji (instrukcja if sprawdzająca wielkość obrazka), a działała. ;/

Ja nie mówię, że funkcja nie działa, tylko że dostaje nieprawidłowe dane na wejście. Sprawdź, co kryje zmienna $mybb->user['avatar'], powinien to być adres obrazka z odpowiednim rozszerzeniem.

Okej, pozmieniałem teraz co nie co, i funkcja wygląda tak:

[PHP] pobierz, plaintext 
<?php
function avatar_resizer($avatar_link)
    {
	$rozmiar = getimagesize($avatar_link);
 
	if($rozmiar[0] > 80 || $rozmiar[1] > 80) {
 
	if($rozmiar[0] > 80) {
 
	$avatardimensions[0] = 80;
	$avatardimensions[1] = (80*$avatardimensions[1])/$avatardimensions[0];
 
	return $avatardimensions;
 
	} else if($rozmiar[1] > 80) {
 
	$avatardimensions[1] = 80;
	$avatardimensions[0] = (80*$avatardimensions[0])/$avatardimensions[1];
 
	return $avatardimensions;
	}
 
            if($avatar_link)
            {
                    $avatar_link = htmlspecialchars_uni($avatar_link);
 
                    if($avatardimensions[0] && $avatardimensions[1])
                    {
                            $avatar_width_height = "width=\"{$avatardimensions[0]}\" height=\"{$avatardimensions[1]}\"";
                    }
                    $avatar = "<img src=\"{$avatar_link}\" alt=\"\" $avatar_width_height />";
            }
            else
            {
                    $avatar = '';
            }
 
            return $avatar;
			}
 
    }
 
 
$avatar = avatar_resizer($mybb->user['avatar']);
?>
[PHP] pobierz, plaintext 

I teraz obrazek się wyświetla, ale tak samo kiedy nie było skali, więc jeżeli bok ma mniej niż 80 to go rozciąga.

Swojego syna też nazwiesz "Błąd" bo nie rozumiesz po co nazywa się pewne rzeczy?
Temat: Jak poprawnie zatytulowac watek

@Refresh

Ale przecież w tym żadnych cudów nie ma:

[PHP] pobierz, plaintext 
function avatar_resizer($avatar_link) {
  $defaultWidth = 80;
  $defaultHeight = 80;
  list($avatarWidth, $avatarHeight) = getimagesize($avatar_link);
  if($avatarWidth > $avatarHeight) {
    $newWidth = $defaultWidth;
    $newHeight = (int)($defaultWidth * $avatarHeight / $avatarWidth);
    if($newHeight > $defaultHeight) {
      $newWidth = (int)($defaultHeight * $avatarWidth / $avatarHeight);
      $newHeight = $defaultHeight;      
    }
  } elseif($avatarHeight > $avatarWidth) {
    $newWidth = (int)($defaultHeight * $avatarWidth / $avatarHeight);
    $newHeight = $defaultHeight;
    if($newWidth > $defaultWidth) {
        $newWidth = $defaultWidth;
        $newHeight = (int)($defaultWidth * $avatarHeight / $avatarWidth);
    }
  } else {
    $newWidth = $defaultWidth;
    $newHeight = $defaultHeight;
  }
  if($avatar_link) {
    $avatar_link = htmlspecialchars_uni($avatar_link);
    $avatar = "<img src=\"{$avatar_link}\" alt=\"\" width=\"{$newWidth}\" height=\"{$newHeight}\" />";
  } else {
    $avatar = '';
  }
  return $avatar;
}
[PHP] pobierz, plaintext 

Kodu na proporcjonalne przeliczenie wysokości i szerokości obrazka w sieci jest całe mnóstwo. Wystarczy dobrze poszukać.

EDIT: Jeśli coś dalej nie będzie działać, to sprawdź, co zwraca funkcja getimagesize() - var_dump($avatarWidth, $avatarHeight) w moim kodzie lub var_dump($rozmiar) w Twoim kodzie.

Hmm.. Jest tak jak powiedziałeś. Getimagesize pobiera obrazek z linku, a ja mam na dysku. Jest jakaś inna funkcja sprawdzająca wielkość obrazka?

Edit: Taki warning wyskakuje: Warning [2] getimagesize(images/avatars/php.gif?dateline=1325855477) [function.getimagesize]: failed to open stream: No such file or directory - Line: 527 - File: global.php PHP 5.2.17-ugu1 (Linux)

Problemem jest przekazywanie niewłaściwej wartości do funkcji avatar_resizer(), a nie nieprawidłowe działanie funkcji getimagesize(). Coś jest skopane w innym miejscu aplikacji, tzn. w miejscu w którym pobierasz adresy url obrazków.

[PHP] pobierz, plaintext 
$avatar = avatar_resizer($mybb->user['avatar']);
[PHP] pobierz, plaintext 

Tutaj pobieram obrazekdo funkcji.

Pisałem już, abyś sprawdził co zawiera zmienna $mybb->user['avatar']: var_dump($mybb->user['avatar']);. Jeśli zawiera adres pliku PHP, to poszukaj tego pliku i sprawdź, co on takiego robi (być może odpowiada za przeskalowanie obrazka).

string(44) "images/avatars/mysql.png?dateline=1325856623"


To mi wyszło, znalazłem też w usercp.php taką linijkę: "avatar" => $avatar['avatar'].'?dateline='.TIME_NOW

@refresh

Trzeba się pozbyć tej końcówki:

[PHP] pobierz, plaintext 
$avatar = avatar_resizer(substr($mybb->user['avatar'], 0, strpos($mybb->user['avatar'], '?') - 1);
[PHP] pobierz, plaintext 

Tzn. zamienić to:

[PHP] pobierz, plaintext 
$avatar = avatar_resizer($mybb->user['avatar']);
[PHP] pobierz, plaintext 

na to co podałeś?

Nie mogę tego usunąć całkowicie bo do tej zmiennej się odnoszę w szablonie MyBB.