Frage zu 'zufälligem Medium'

[son]

Hallo zusammen,
ich habe gerade das Modul 'Zufälliges Medium' angepasst und eingebunden. Es funktioniert soweit auch, also dass bei jedem Aufruf ein zufälliges Bild angezeigt wird, allerdings bekomme ich immer folgende Fehlermeldung:

Warning: Invalid argument supplied for foreach() in /usr/local/etc/httpd/htdocs/kunden/fotomarathon/redaxo3_2/redaxo/include/classes/class.oomedia.inc.php on line 536

Hier ist der Quelltext des Moduls:

Code: Alles auswählen

<?php
$mediacat = 4;

 $cat = OOMediaCategory::getCategoryById($mediacat);
 $bilder = $cat->getFiles();

 shuffle( $bilder);
 $randIndex = mt_rand( 0, count( $bilder) -1);
 $bild = $bilder[ $randIndex];

echo $bild->toHTML();
?>

Hat jemand von Euch eine Idee, was falsch sein könnte? Ich habe leider keinen blassen Schimmer...
Vielen Dank schon einmal im voraus und viele Grüße,
son

[son]

Die Fehlermeldung bezieht sich ja auf diese Methode in der class.oomedia.inc.php... Nur was für ein Argument muss ich im Modul angeben, dass der Fehler verschwindet?

Code: Alles auswählen

// Zusatzattribute anfügen 
    foreach ($params as $name => $value)
    {
      $additional .= ' '. $name.'="'.$value.'"';
    }

    return sprintf('<img src="%s"%s />', $path.$file, $additional);
  }

???

[Markus.Staab]

Hi son,

das sieht ja komisch aus.. Ich habe dieses Modul bereits ohne Probleme im Einsatz... Evtl mal die class.oomedia aus dem CVS verwenden..

Gruß,
Markus

[son]

Hi Markus,
ja, irgendwie habe ich ein Talent für seltsame Fehler...
Ich habe redaxo 3.2 installiert und an der class.oomedia.inc.php nach der Installation nie was verändert. Mit welcher Datei, meinst Du, könnte ich es nochmal probieren?
Viele Grüße und vielen Dank für Deine Antwort,
son

[Markus.Staab]

Hi son,

versuchs mal mit der neusten Version:

Gruß,
Markus

[son]

Hi Markus,
habe es gerade probiert, funktioniert leider auch nicht... immernoch die gleiche Fehlermeldung...

Aber vielen lieben Dank für Deine Hilfe und
Grüße,
son

[Markus.Staab]

HI son,

teste ich bei gelegenheit, klappt aber jetzt wohl leider nicht mehr diese Woche.

Gruß,
Markus

[Thomas.Blum]

Hej,

welche PHP-Version wird denn verwendet?

vg Thomas

[son]

Hallo Markus und Thomas,

teste ich bei gelegenheit, klappt aber jetzt wohl leider nicht mehr diese Woche.

das wäre klasse! Vielen lieben Dank schon einmal!

welche PHP-Version wird denn verwendet?

php Version 4.3.4

Viele Grüße,
Sonja

[benny0903]

Wurde schon eine Lösung gefunden?

Erhalte gleiche Fehlermeldung. Habe ebenfalls Version 3.2 im Einsatz.
PHP Version 4.4.2

Danke!

[son]

bis jetzt leider noch nicht, bin immer noch ratlos...
Grüße, Sonja

[discsniffer]

Hallo Sonja,

ich befürchte zu wissen weshalb du den Fehler hast. durch den Aufruf:

Code: Alles auswählen

echo $bild->toHTML();

übergibst du toHTML() nichts, intern ersetzt toHTML deine Übergabe in:

Code: Alles auswählen

echo $bild->toHTML('');

dieser Wert wird an eine interne Funktion toImage() weitergereicht. Dort löst das letztendlich den Fehler auf weil '' eben kein Array ist den man durchlaufen kann. Ich konnte den folgenden Lösungsvorschlag leider noch nicht testen, da ich das zufällige Medium usw. nicht benutze. Deshalb probier mal, ohne Gewähr, aus den Aufruf wie folgt durchzuführen:

Code: Alles auswählen

echo $bild->toHTML(array());

Sollte dies nicht helfen müßte man vor dieser Foreach-Schleife abfangen ob es sich bei $Param um einen Array handelt. Das machst du am besten wie folgt:

Code: Alles auswählen

// Zusatzatrribute anfügen
if (is_array($params)) //Handelt es sich bei $params um einen Array?
{
foreach ($params as $name => $value)
{
$additional .= ' '. $name.'="'.$value.'"';
}
}

Das beeinträchtigt auch nicht die weitere Funktionalität des Skriptes.

Ich hoffe das war die Lösung deines Problems. Nette Grüße,

Flo

[son]

Hi Flo,
Klasse, das funktioniert!!


Vielen, vielen lieben Dank!!!!
und Grüße,
Sonja

[discsniffer]

Hi Sonja,
das freut mich!
Liebe Grüße,
Flo

[benny0903]

Das klappt supi!

Kleine Nachfrage:
Wie schaffe ich es, dass ich nur den Dateipfad erhalte?
bsp: files/beispielbild.jpg

Dann kann ich ein Zufallshintergrundbild nutzen.

[discsniffer]

Auch wenn ich das AddOn immer noch nicht bei mir installiert hab müßtest du den Dateinamen wie folgt erhalten:

Code: Alles auswählen

$Dateipfad = 'files/' . $bild->getFileName();

Also bitte selbst ausprobieren. Ich muß darauf hinweisen das in diesem Fall die Bilder nicht resized werden, wie es u.U. der Fall ist wenn man regulär toHtml() benutzt.
Viel erfolg.

Grüße,

Flo

[Markus.Staab]

Hi,

Code: Alles auswählen

$dateiname = $media->getFileName();

der von Flo genannte Fix, ist jetzt auch im CVS.

Gruß,
Markus

[discsniffer]

Vielen Dank Markus! - Flo

[benny0903]

Funktioniert einwandfrei!

DANKE.

[discsniffer]

Bitte..

[Joachim.Dörr]

Hi der Code:

Code: Alles auswählen

echo $bild->toHTML(); 

gibt kein valides XHTML aus. dazu fehlt das "alt" atribut.

geht das irgendwie, dass man da die Ausgabe modifizieren kann? wo man zumindest ein pseudo alt einstellen kann.

lg
Joachim

[Joachim.Dörr]

Weiß keiner weiter?

könnte ne lösung wirklich gut gebrauchen.

[Joachim.Dörr]

Ok habs selbst gefunden

In der Datei "class.oomedia.inc.php" in der Zeile 547 hab ich diesen Code:

Code: Alles auswählen

return sprintf('<img src="%s"%s />', $path.$file, $additional);

etwa so geänder:

Code: Alles auswählen

return sprintf('<img src="%s"%s alt="" />', $path.$file, $additional);

damit freud sich der Validator schon etwas mehr und sagt jetzt ist es xhtml

lg
Joachim

[Markus.Staab]

Hi,

Ist fürs nächste Release gefixed...

Gruß,
Markus

[sawro]

hi zusammen,

in der class.oomedia.inc.php ( ab ca. zeile 523) habe ich das:

Code: Alles auswählen

    // Alternativtext hinzufügen    
    if (!isset($params['alt']))
    {
      if (($desc = $this->getDescription()) != '')
      {
        $params['alt'] = htmlspecialchars($desc);
      }
	  else //falls nichts eingetragen wird
	  {
        $params['alt'] = htmlspecialchars(' ');
	  }
    }
    // Titel hinzufügen    
    if (!isset($params['title']))
    {
      if (($title = $this->getTitle()) != '')
      {
        $params['title'] = htmlspecialchars($title);
      }
    }
    // Breite hinzufügen    
    if (!isset($params['width']))
    {
      if (($width = $this->getWidth()) != '')
      {
        $params['width'] = htmlspecialchars($width);
      }
    }
    // Höhe hinzufügen    
    if (!isset($params['height']))
    {
      if (($height = $this->getHeight()) != '')
      {
        $params['height'] = htmlspecialchars($height);
      }
    }

= die valide ausgabe vom zufallsbild. alt-tag wird gefüllt aus dem beschreibungstext im medienpool. wenn kein beschreibungstext vorhanden ist, wird ein leeres alt-tag ausgegeben.

bis denne....

lg
sabine