[REL] Gallery Block 1.4.1

[Christian_N]

So, die neue Version ist draußen, alle Punkte erledigt
Gruß Chris

[Kevin]

Perfekt, genau jetzt will ich los legen.

[Kevin]

Eine Kleinigkeit noch: war nicht XHTML strict valid (div und span waren nicht sauber verschachtelt) und mir fehlte ein bisschen Abstand zwischen dem Bild und dem Bild Titel.
album_center.html - suche:

Code: Select all

				<span class="gensmall">
					<strong>{L_IMAGE_TITLE}:</strong> {images_ct.images_detail.TITLE}<br />
					<strong>{L_POSTER}:</strong> {images_ct.images_detail.POSTER}<br />
					<strong>{L_POSTED}:</strong> {images_ct.images_detail.TIME}<br />
					<strong>{L_VIEWS}:</strong> {images_ct.images_detail.VIEW}<br />
					{images_ct.images_detail.RATING}
					{images_ct.images_detail.COMMENTS}</div>
				</span>

ersetze durch:

Code: Select all

				<div style="margin: 5px 0 0 0">				
				<span class="gensmall">
					<strong>{L_IMAGE_TITLE}:</strong> {images_ct.images_detail.TITLE}<br />
					<strong>{L_POSTER}:</strong> {images_ct.images_detail.POSTER}<br />
					<strong>{L_POSTED}:</strong> {images_ct.images_detail.TIME}<br />
					<strong>{L_VIEWS}:</strong> {images_ct.images_detail.VIEW}<br />
					{images_ct.images_detail.RATING}
					{images_ct.images_detail.COMMENTS}
				</span></div></div>

album.html - suche:

Code: Select all

<div style="text-align: left;"><span class="gensmall">

ersetze durch:

Code: Select all

<div style="text-align: left; margin: 5px 0 0 0"><span class="gensmall">

Ansonsten: super - vielen Dank, Christian!

[Christian_N]

Da muss ich noch mal dem Paket auf meine Seite teste, den bei mir ist es XHTML strict valid.
Danke für die Meldung, setz dann gleich mal des neue Paket hoch.

[Christian_N]

Stimmt im Paket war die anders als auf mein Webspace, wie peinlich.
Ok habe die beide Dateien jetzt mit deine Änderung angepasst und muss zugeben sieht besser aus, danke dafür.

Die Versionsnummer hat sich deshalb nicht geändert, wer das Paket vor 16:20Uhr runtergeladen hat, muss die styles/prosilver/template/portal/block/album.html und styles/prosilver/template/portal/block/album_center.html ersetzen.

Gruß Chris

[Pionier]

Habe zwei kleine Fehler gefunden

1. Wenn bei "Anzahl der anzeigenden Bilder:" nicht mindestens eine 1 steht kommt im Portal folgende Fehlermeldung (bei beiden Boxen)

   Code: Select all

   [phpBB Debug] PHP Notice: in file /portal/block/album.php on line 66: Division by zero
   [phpBB Debug] PHP Notice: in file /includes/functions.php on line 3433: Cannot modify header information - headers already sent by (output started at /includes/functions.php:2938)
   [phpBB Debug] PHP Notice: in file /includes/functions.php on line 3435: Cannot modify header information - headers already sent by (output started at /includes/functions.php:2938)
   [phpBB Debug] PHP Notice: in file /includes/functions.php on line 3436: Cannot modify header information - headers already sent by (output started at /includes/functions.php:2938)
   [phpBB Debug] PHP Notice: in file /includes/functions.php on line 3437: Cannot modify header information - headers already sent by (output started at /includes/functions.php:2938)
   

   Diese Meldung kommt wenn bei beiden Box eine 0 eingetragen ist und wenn nur bei der rechten Box eingetragen ist. Es müssen aber beide Boxen eingeschaltet sein.
2. In der Install.xml hat sich der Fehlerteufel eingeschlichen

   Open: language/en/mods/lang_portal_acp.php
   Find
   Tip: This may be a partial find and not the whole line.

   Code: Select all

   Code:Select All
   ));
   
   ?>
   

   Code: Select all

   Add before
   Tip: Add these lines on a new blank line before the preceding line(s) to find.
   
   Code:Select All
   // Gallery
   'ACP_PORTAL_GALLERY_SETTINGS' => 'Preferences for the gallery block',
   'ACP_PORTAL_GALLERY_SETTINGS_EXPLAIN' => 'Here you can see the settings for the gallery block.',
   'ACP_PORTAL_GALLERY_SETTINGS_RIGHT' => 'Settings for the right gallery block',
   'ACP_PORTAL_GALLERY_SETTINGS_CENTER' => 'Settings for the middle gallery block',
   'PORTAL_GALLERY' => 'Display gallery block',
   'PORTAL_GALLERY_EXPLAIN' => 'This block on the portal.',
   'PORTAL_IMAGES_NUMBER' => 'Number of displaying images'
   'PORTAL_IMAGES_SORT' => 'Random images Show',
   'PORTAL_IMAGES_SORT_EXPLAIN' => 'For last images choose "no".'
   'PORTAL_ALBUM_ID' => 'ID of the albums',
   'PORTAL_ALBUM_ID_EXPLAIN' => 'Album we pull the images from, leave blank to pull from all albums, separate by comma for multi-albums, eg. 1,2,5',

   es müsste aber wie folgt lauten:
   
   Code:Select All
   ));
   
   Add before
   Tip: Add these lines on a new blank line before the preceding line(s) to find.
   
   Code:Select All
   // Gallery
   'ACP_PORTAL_GALLERY_SETTINGS' => 'Preferences for the gallery block',
   'ACP_PORTAL_GALLERY_SETTINGS_EXPLAIN' => 'Here you can see the settings for the gallery block.',
   'ACP_PORTAL_GALLERY_SETTINGS_RIGHT' => 'Settings for the right gallery block',
   'ACP_PORTAL_GALLERY_SETTINGS_CENTER' => 'Settings for the middle gallery block',
   'PORTAL_GALLERY' => 'Display gallery block',
   'PORTAL_GALLERY_EXPLAIN' => 'This block on the portal.',
   'PORTAL_IMAGES_NUMBER' => 'Number of displaying images',
   'PORTAL_IMAGES_SORT' => 'Random images Show',
   'PORTAL_IMAGES_SORT_EXPLAIN' => 'For last images choose "no".',
   'PORTAL_ALBUM_ID' => 'ID of the albums',
   'PORTAL_ALBUM_ID_EXPLAIN' => 'Album we pull the images from, leave blank to pull from all albums, separate by comma for multi-albums, eg. 1,2,5',
   
   Man achte auf die roten Kommas hinter images' und "no".

[Christian_N]

Danke Pionier für die Meldungen, habe beides in der CVS gefixt.
Solange noch nicht die neue Version draußen ist, was ich eigentlich auch erstmal sofern möglich hinziehen möchte, bis die neue Version vom Gallery MOD (nickvergessen) veröffentlicht ist.

Daher können alle die den Block einsetzen erstmal die Änderungen selbst durchführen: http://www.phpbb-projekt.de/bugtracker/viewbug.php?b=4

Gruß Chris

[Christian_N]

Version 1.0.2 erschienen.

Wichtiger Hinweis diese Version läuft aktuell nur unter den phpBB Gallery MOD 0.3.0, bei ältere Versionen kommt es zu Problemen!

[Kevin]

Läuft denn die vorige Version von deinem Block noch mit der 0.3.0 des Albums?

[Christian_N]

Das kann ich aktuell nicht beantworten, müsste ich testen erst auf der Testumgebung.
Könnte aber gut sein, den unter der eine Rev. von dem Galerie MOD lief die eigentlich auch noch nur das die Bilder aus persönlichen Alben angezeigt wurde, was ich persönlich nicht möchtet und so habe ich hier dann einfach weitere Einstellmöglichkeit hinzugefügt das man die getrennt bei bedarf deaktivieren kann.

Außerdem wird bei der v1.0.1 bei den persönlichen Alben die Berechtigungen nicht geprüft, sprich wenn im ACP beim Galerie MOD bei den "Persönliche Alben Berechtigungen" z.B. bei "Gäste/Bots --> Darf persönliche Alben ansehen" auf "Niemals" oder "Nein" einstellst, dann können die dieses trotzdem im Portal, was man eigentlich aber dann ja nicht wollte, dieses wird in der v1.0.2 geprüft und dann ausgeblendet, so das nur angezeigt wird wenn "Ja" ausgewählt ist.

Meine Empfehlung auch wenn v1.0.1 gut möglich unter 0.3.0 laufen könnte, wäre ein Update auf 1.0.2 durchzuführen.

[Kevin]

Code: Select all

	'PORTAL_VIEW_PERS_ALBUMS'				=> 'Zeigen Bilder aus persönlichen Alben an?',

Da fände ich dies besser:

Code: Select all

	'PORTAL_VIEW_PERS_ALBUMS'				=> 'Bilder aus persönlichen Alben anzeigen?',

[Christian_N]

Danke wird geändert.

In Englisch könnte man das ganze womöglich auch viel besser schreiben, so wie es momentan ist, ist großteils von Google übersetzt, wer mag kann es ja mal in des richtige Englisch verfassen, würde mich freue.

[Kevin]

Noch ne Kleinigkeit:
Nach dem Update war in ID der Alben: eine 0 eingetragen, das hat dazu geführt das gar nix mehr auf dem Portal angezeigt wurde.
Ansonsten sieht es auf den ersten Blick gut aus.

[Christian_N]

Das gibt es doch nicht, das hatte ich beim Installieren gestern auch gehabt, bevor ich das Paket hochgeschoben habe und noch vorher geändert, habe ich da des Update wohl übersehen.

Danke schau da noch mal nach.

[Redbull254]

Änderst Du das jetzt noch im Paket? Dann lade ich mir das noch einmal neu runter. Ich wollte es gleich installieren
Dann warte ich noch solange.