Album nach lösche, bleibt des Bild im upload/cache Ordner...

[Christian_N]

Deine phpBB Version: 2.0.15
Typ Deines phpBB Forums: Standard phpBB (auch Vanilla genannt)
MODs: Ja
Dein Wissensstand: Grundwissen

Fehlerbeschreibung und Nachricht

Hallo,
ich habe jetzt gemerkt, wenn ich im Album ein Bild lösche zeigt es zwar nicht mehr an aber er benutzt weiterhin die KB des bildes meines Servers...
Weil das Bild im upload und cache Ordner bleibt, gibt es eine möglichkeit, das beim löschen eines Bildes es auch aus die ordner gelöscht wird?

Oder kann man es irgendwie nachträglich aus den ordner jetzt entferne über eine abfrage oder so?

Ich hoffe das dazu eine möglichkeit gibt um auch platz zu sparen des Servers.

Ich mein KB ist nicht viel aber wenn jetzt 1000 Bilder gelöscht wurde sind es schon ein paar MB die unnötig belegt würden.

Im voraus vielen dank für jeden antwort und ggf. abhilfe dazu.

EDIT: Ich denk mal das es möglich ist zu machen da beim Attachment MOD es auch geht, wenn eine Datei gelöscht wird wird sie auch aus dem verzeichnis "files" gelöscht.

[Hotflame]

hi SuesseMaus28884

Frag mal blondi oder such mal bei phpbb, ich glaube da gibt es einen Mod der sowas kann.

mfg

Hoti

[Max]

Hallo,


das hier: >Klick!> ist in der 2.0.53 von Haus aus drin. Dabei werden auch die Bilddateien vom Server gelöscht, wenn man Bilder im Album löscht.

Aber grundsätzlich wäre ein Tool interessant, was vorhandene Bilddateien auf dem Server mit den Einträgen in der Datenbank abgleicht. Wenn da jemand ein MOD für kennt, wäre das recht hilfreich - schließlich ist das im Album noch nicht sehr lange gefixt.



Gruß Max



EDIT:

nun habe ich mal am Delete Unused Avatars - MOD gebastelt.
Grund, der macht ja im Grunde das, was gewünscht ist, nur eben für Avatare. Mein Gedanke war dann, einfach die Pfade und DB-Abfragen ändern und dann müsste doch theoretisch jede Datei im upload-Ordner gefunden werden, für die es keinen Datenbankeintrag gibt.
Das funktioniert aber nur für den upload-Ordner, die Thumbnails wären dann eine extra-Geschichte (aber die kann man ja eh mittels Album-MOD via "Cache leeren" entfernen)

Nun zu den Sorgen dabei, es gibt eine nette Fehlermeldung:

Warning: Wrong parameter count for strrchr() in /www/htdocs/ssvform/forum/admin/admin_avatar_delete.php on line 98
Warning: Wrong parameter count for strrchr() in /www/htdocs/ssvform/forum/admin/admin_avatar_delete.php on line 98
Warning: Wrong parameter count for strrchr() in /www/htdocs/ssvform/forum/admin/admin_avatar_delete.php on line 98
Warning: Wrong parameter count for strrchr() in /www/htdocs/ssvform/forum/admin/admin_avatar_delete.php on line 98

Wie man die wegbekommt weiß ich nicht, ich habe nur Wege gefunden, diese Fehlermeldung zu verändern und dabei die Funktion des Ganzen total zu beseitigen.
Hier mal der Code-Abschnitt:

Code: Alles auswählen

// Ignore following files Start
if($file=='.'||$file=='..'||$file=='index.htm'||$file=='index.html'||$file==substr(strrchr('album_mod/upload/'), 1))
continue;

$sql = "SELECT COUNT( * ) AS total FROM " . $table_prefix . "album WHERE pic_filename = '" . $file . "'";
if ( !($result = $db->sql_query($sql)) )
{
message_die(GENERAL_MESSAGE, $lang['Avatar_Delete_7'], '', __LINE__, __FILE__, $sql);
}
$row = $db->sql_fetchrow($result);
if($row['total'] > 0)
continue;
// Ignore following files End

Davon die erste Zeile soll den Fehler verursachen.

Außerdem passt es sich nicht ins Template ein, sondern ist einfach nur weiß mit schwarzer Schrift - das tut der Funktion aber keinen Abbruch.

Wenn da größeres allgemeines Interesse besteht, wäre es gut, wenn mal ein Profi den Code durchsieht, um Fehler zu vermeiden.
So, wie es derzeit ist, mag ich das jedenfalls nicht im Ganzen hier öffentlich posten.



nu ist es im richtigen Beitrag...

[Christian_N]

Oki hab mir mal das von Max angeschaut und hab ja die version 2.0.53 drauf und da ist dies schon behoben in der album_modcp.php aber das problem bei mir ist wenn man in album_cat.php das bild löscht wird es nicht vom server entfernt... daher vermute ich mal muss man dies auch irgendwie in album_cat oder album_delete.php was anpassen wohl?

EDIT: In der Version ist doch kein Fehler mehr war wieder ein menschlicher Fehler vor mir *g*
Habe mal 10 Bilder hochgeladen 5 gelöscht über album_cat.php und er hatte auch dann 5 nur noch drin gehabt gelöscht über album_modcp.php die andere 5 und 0 bilder waren drin...

Der menschlicher Fehler vor mir war das ich das verzeichnis in mein FTP nicht aktuallisiert habe, ich musste um es zu testen es aktuallisieren, was er ja automatisch tut, wenn ich mich neu einlogge bzw. in ein andere verzeichnis bin und später drauf zugriff nochmal.

Jedenfalls weiss ich jetzt das die doch gelöscht werden und somit kann das thema ggf. geschlossen werden da das Problem behoben ist.

[Max]

Hallo,


also ich habe mal eben das Teil ausprobiert.....

Es werden ne Menge Bilder angezeigt, die nicht vollständig hochgeladen wurden - also wahrscheinlich Reste von Uploadabbrüchen, die faktisch ja nie im Album erschienen sind - das Zeug waren locker 15 mb Müll auf dem Server, immer viertel oder halbe Bilder in Größen so um die 700 bis 1000 kb. Das Zeug habe ich damit gelöscht.

Nur leider besteht nach wie vor das Problem mit der Fehlermeldung - sie erscheint allerdings abhängig von der gefundenen Bilderzahl in ihrer Häufigkeit (also 3 Dateien gefunden, 4mal Fehlermeldung) bei dem laufenden Album habe ich das nicht mehr nachgezählt, da das zig Dateien waren.

Es wäre ja cool, wenn sich damit mal einer, der sich auskennt befassen könnte, denn völlig sinnlos scheint die Suche nach Dateileichen im Upload-Ordner nicht zu sein.



Gruß Max

[oxpus]

Die Fehlermeldung wäre hier zu finden:

if($file=='.'||$file=='..'||$file=='index.htm'||$file=='index.html'||$file==substr(strrchr('album_mod/upload/'), 1))

Genau diese Klammer ist an der falschen Stelle und daher muss die Zeile so aussehen:

Code: Alles auswählen

if($file=='.'||$file=='..'||$file=='index.htm'||$file=='index.html'||$file==substr(strrchr('album_mod/upload/',1)))

[Max]

Hi,


das hat nur eine Fehlerverschiebung gebracht:

Warning: Wrong parameter count for substr() in /www/htdocs/v135058/phpBB2/admin/admin_albumpic_delete.php on line 98

Geht da immer noch um die selbe Zeile, nur dass der Fehler für den ersten Teil genannt wird, also kam ich auf die "Idee", das mal so zu probieren:

Code: Alles auswählen

if($file=='.'||$file=='..'||$file=='index.htm'||$file=='index.html'||$file==substr(strrchr('album_mod/upload/',1 ),1))

und schon geht´s.

Nun habe ich das auf drei Foren getestet, es hat jedes Mal nichts versaut, deshalb habe ich das mal im Ganzen umbenannt und die Lang-Anpassung geändert, so dass man den Avatar-Lösch-MOD parallel nutzen könnte und es im ACP beim Albumbereich steht.

Besteht Interesse, das mal zu testen?
Wenn ja, dann packe ich das mal zusammen und hänge das hier mit an.



Gruß Max

[oxpus]

Ja gerne. Ich denke, daß ich auch so manche Leiche im Keller habe.
Verwende ich das Album doch auch schon seit einigen Versionen.

[Christian_N]

Wär nicht schlecht, gerad bei des Plus Board wo ich die version 2.0.51 noch drin hab sind einige bilder drin die garnet mehr in die datenbank sind gerade hab ich schon 100 aussotiert, aber wenn man es damit beheben kann kann ich dies ja hoffentlich jetzt sparn sie manuell aussotiern

[Max]

Hallo,


also ich habe AWSW, den Autoren des Original-MOD angeschrieben, wie es mit seinem Einverständnis aussieht, das generell anzubieten.

Bis dahin sollte das hier auch ok sein, wenn es für die Member hier als Attachment verfügbar ist.



Gruß Max



EDIT: Attachment gelöscht, neue Version einige Beiträge später zu finden

[Christian_N]

So hab mal an die Datei noch ein wenig gebastelt das er die Bilder
a) in Thumbnail darstellt also vom cache ordner lädt.
b) der Header und Footer vom ACP.

Dies kann man so machen:

Öffne admin_albumpic_delete

Finde

Code: Alles auswählen

$phpbb_root_path = './../';

Eine Zeile darunter dieses Einfügen [After, Add]

Code: Alles auswählen

$phpbb_admin_root_path = './';

Dann finde:

Code: Alles auswählen

include($phpbb_root_path . 'includes/page_header.'.$phpEx);

ersetze mit diesem:

Code: Alles auswählen

include($phpbb_admin_root_path . 'page_header_admin.'.$phpEx);

Finde nun diese:

Code: Alles auswählen

// Read directory Start
$base_dir = "./../album_mod/upload/";

ersetzen mit diesem:

Code: Alles auswählen

// Read directory Start
$base_dir = "./../album_mod/upload/cache/";

Finde nun zum schluss diese:

Code: Alles auswählen

include($phpbb_root_path . 'includes/page_tail.'.$phpEx);

und ersetzen mit diesem:

Code: Alles auswählen

include($phpbb_admin_root_path . 'page_footer_admin.'.$phpEx);

Nun wird Header und Footer des ACP geladen und die Bilder die im Server liegen aber nicht mehr in Datenbank sind werde als Thumbnail dargestellt sieht dann gleich viel besser aus

ggf. könnt ihr auch meine bearbeitete Datei einfach runterladen und bei euch in ./admin/ verzeichnis laden (ersetzen)

EDIT: vielleicht kann man doch irgendwie es bezwecken das auch die tote bilder vom cache ordner gelöscht werden, mal schaun ob ich ne lösung oder jemand anders von hier noch findet

Gruß SuesseMaus28884

[ Attachment gelöscht am Mo 23.Mai, 2005 16:53 von AmigaLink ]

[Max]

[quote="SuesseMaus28884 - So 22.Mai, 2005 20:47";p="32729"] vielleicht kann man doch irgendwie es bezwecken das auch die tote bilder vom cache ordner gelöscht werden, mal schaun ob ich ne lösung oder jemand anders von hier noch findet [/quote]

Naja, dafür müsste man den Ablauf, der da fürs Album stattfindet, nochmal für den Cache machen.
Aber wozu die Mühe, man kann den Cache des Albums doch eh vom Album selber löschen lassen, ist zwar der ganze Cache, aber eigentlich reicht es.
Die Thumbnails zu löschen bringt auch nicht den Megagewinn, bei mir sind ca. 1000 Bilder im Album, der Thumbnailordner hat grade mal unter 100kb.


Und nun werde ich mir mal die von Dir bearbeitete Datei reinziehen - so wird das ja ein richtiges Teamprojekt :-)



Gruß Max

[Christian_N]

Jep stimmt auch wieder...
hab zwar irgendwo ein Problem lösch ich was bleibt das bild erhalten also im enddefekt hat er es garnet gelöscht...

Hab noch gemacht bei dein Code den oben genannt hattes:

Code: Alles auswählen

if($file=='.'||$file=='..'||$file=='index.htm'||$file=='index.html'||$file==substr(strrchr('album_mod/upload/',1 ),1))

in geändert

Code: Alles auswählen

if($file=='.'||$file=='..'||$file=='.htaccess'||$file=='index.htm'||$file=='index.html'||$file==substr(strrchr('album_mod/upload/',1 ),1))

sozusage das er auch die datei .htaccess ingoniert da er sonst dies anzeigt im ACP als ein rotes Kreuz.

Allerdings muss ich mal schaun wo der wurm hängt weil nach dem löschen lädt er die Seite und das Bild ist immer noch drin *grubel*

[oxpus]

AWSW?
Original-Mod?
Also der Avatar-Check-Mod?

Und ja Danke. Werde ich die Tage testen.

[Max]

Hi,


also Du hast ja diesen Code als Ersatz eingesetzt:

Code: Alles auswählen

// Read directory Start 
$base_dir = "./../album_mod/upload/cache/";

an Stelle von:

Code: Alles auswählen

// Read directory Start 
$base_dir = "./../album_mod/upload/";

Dadurch werden Zwar Thumbnails im ACP angezeigt, aber dafür nur die Thumbnails, und natürlich werden auch nur die Thumbnails gelöscht.


Gruß Max


EDIT: so, wie das mit Thumbnails geregelt wurde, konnte das ja nicht funktionieren - Du hast als Anzeigeverzeichnis cache genommen (weil ja so die Thumbnails angezeigt werden) und als Löschverzeichnis upload gelassen (so, wie ich das ursprünglich eingerichtet hatte). Dadurch ist in der Anzeige immer alles aus dem Thumbnailordner zu sehen und beim Löschen wird das reguläre File entfernt, aber nach dem Löschen wird das Thumnail wieder angezeigt.

[Christian_N]

Net ganz richtig, im ACP werden die Thumbnails angezeigt ja aber gelöscht werden die aus dem upload verzeichnis aber das wird ja nicht angezeigt er lädt im enddefekt ja die sache vom cache ordner hoch...

Gelöscht wurde es aus dem verzeichnis "upload" hab im FTP geschaut im cache ist er noch drin, würde ich cache leeren würde es auch net mehr angezeigt, daher muss man schaun das er auch gleichzeitig es doch aus cache löscht auch wenns net viel platz wegnimmt, aber dann würde das problem gelöst, mal einfach schaun was ich da noch basteln kann

EDIT: So hab mal weiter gebasteltet, jetzt wenn man das Bild löscht zeigt er es nicht mehr im ACP an aber die bilder bleiben weiterhin (thumbnails) im ACP als ansicht, nach klick auf die dateiname bzw. des Bild öffnet sich das orginal bild aus dem upload verzeichnis

Dann siehe da hab es hinbekommen das er auch es gleichzeitig trotzdem aus den Cache Ordner löscht das bild so wird es wenn es nicht mehr in der Datenbank gibt aus upload und cache gelöscht vom Server

Auch wenn die im Chache nicht viel platz wegnehme aber wie sagt man so schön kleinmist macht auch viel mist und

Die Datei werde ich nachher nochmal anhängen (den obrigen beitrag aktuallisieren)

Aber später erst da ich noch was schauen will das nachdem abgesendet wurde eine Information Nachricht kommt das die erfolgreich gelöscht wurden.

Gruß SuesseMaus28884

[Max]

Hi,


also ich lass das so, wie es war.

Praktisch, dass nur der Uploadordner ausgelesen wird und dort auch gelöscht. Grund dafür ist, dass ich in erster Linie Dateileichen weg haben will, die u.U. auch bei einem mißglückten Upload entstehen können - und die haben ja kein Thumbnail entstehen lassen.
Dazu kommt, dass selbst 50 Thumbnails keine nennenswerte Menge an Space verbrauchen, ansonsten kann man ja mal vom Album selbst den Cache leeren lassen.



Gruß Max

[Christian_N]

Nun jeder sowie er möchte man kann ja den cache ignorieren beim löschen und sich die bilder in thumbnail darstellen lasse.

Weil sieht so wie er war nicht schön aus riesen bild zerstört die tabelle und man kan scollen nach rechts um an des häkchen zu setzen und bei Thumbnails ist tabelle schön und wird nicht zerstört.

Das mit der nachricht bekomm ich nur halb irgendwie hin, naja ist auch net so wichtig

[Max]

Naja, klar ist das mit Thumbnails schöner, aber wie schon gesagt, wenn man von dort anzeigen lässt, kann man ja schlecht die Files löschen, die nur im Upload-Ordner liegen, aber kein Thumbnail dazu haben, so wie die Upload-Abbruch-Leichen, deshalb, weil das die größten Brocken waren mit bis zu einem MB, lass ich dort die Prämisse.


Gruß Max

[Christian_N]

Doch klar kann man sich die in Thumbnails darstellen und nur dieFiles die im Upload-Ordner liegen löschen, dazu muss mein Code etwas anders lauten...

Ich poste nachher mal die änderrungen...

Gruß SuesseMaus28884

[Max]

na da ist ja auch kein Problem, aus dem einen Ordner die Anzeige zu bekommen und im anderem zu löschen.
Aber das Problem ist, dass die Uploadleichen garnicht erst ein Thumbnail entstehen lassen haben - daher nützt die Anzeigevariante nichts.


Gruß Max

[Christian_N]

Hm, stande auf der Leitung *g*
Dann bringt der Code auch nichts da ich jetzt dein Problem verstanden habe.

aber mit height und width die darstellung auf eine bestimmte größe darstellen lassen nur so das die tabelle nicht zerstört wird... meine so ungefähr finde:

Code: Alles auswählen

// Ignore following files End


$t="<tr>
<td align='center' class='row2'>&nbsp;<span class='genmed'><a href=$path$file target='_blank' class='gensmall'><img src=$path$file border='0'></a></img></span>&nbsp;</td><td align='center' class='row2'>&nbsp;<span class='genmed'><a href=$path$file target='_blank' class='gensmall'>$file</a></span>&nbsp;</td>";
echo $t.substr($space,0,40-strlen($file)) ;

Ersetzen mit

Code: Alles auswählen

// Ignore following files End


$t="<tr>
<td align='center' class='row2'>&nbsp;<span class='genmed'><a href=$path$file target='_blank' class='gensmall'><img src=$path$file border='0' height='150' width='150'></a></img></span>&nbsp;</td><td align='center' class='row2'>&nbsp;<span class='genmed'><a href=$path$file target='_blank' class='gensmall'>$file</a></span>&nbsp;</td>";
echo $t.substr($space,0,40-strlen($file)) ;

jetzt werde die in eine Pixel von 150x150 dargestellt und er tut nur des upload-verzeichnis wie vorher einlesen alles andere bleibt nähmlich unverändert...

Nur so als vorschlag

[oxpus]

Besser die angehängte Datei, die mit richtigem Seitenverhältnis runterrechnet.
Dazu habe ich auch die Sprache in die lang_admin_album.php gelegt.
Die $lang-Einträge müssten also in diese Datei verschoben werden.
Und auch die Anpassung an das ACP habe ich nochmals korrigiert.

Datei? Datei!

[ Attachment gelöscht am Mo 23.Mai, 2005 01:10 von oxpus ]

[Christian_N]

[quote="oxpus - Mo 23.Mai, 2005 00:52";p="32770"]....
Dazu habe ich auch die Sprache in die lang_admin_album.php gelegt.
....
[/quote]

Hab die auch in lang_admin_album.php gemacht gehabt, den in lang_main.php brauch die eigentlich ja net zu sein da er ja sie nirgends ausserhalb des ACP benötigt

EDIT: Hab mal deine Datei hochgeladen jetzt zeigt er allerdings wieder die ".htaccess" Datei an, hab sie wieder ignoniert so wie oben schon beschrieben...
Da wenn mehre datei drin sind (100) z.B. und man alle auswählen tut würde man die gleich mitlöschen wär ja net so praktisch dann.

[oxpus]

Die Datei habe ich nun angepasst und auch noch das Formular korrigert, damit er beim Löschversuch nicht den Index lädt. append_sid() ist im ACP megawichtig!

[ Attachment gelöscht am Mo 23.Mai, 2005 16:53 von AmigaLink ]