Guestbook ummodeln

[Argos]

[beware: lousy attempt at writing German  ]

Mein Vorslag ist om der Name "Flohmarkt" zu änderen in ein Englisher Name, wie "Classifieds". WB ist ein internationaler CMS, so modules sollten in Princip ein Englischer Name haben.

Grüsse,
Jurgen

[karldehall]

Eine Frage an Mux:
Gibt es eine Möglichkeit, im Flohmarkt ein "Verfallsdatum" einzubauen?
Gilt von jetzt + 4 Wochen?
Aber ich gebs gleich zu: Das bekomme ich sicher nicht alleine in das vorhandene Modul rein.
Wir würden gerne Kollegen im Intranet die Möglichkeit geben, Angebote/Suchen selber reinzustellen ohne dass dauernd jemand nachgucken und alte Sachen entfernen muss.
Danke vorab für jede Antwort von Dir oder anderen Forumsmitgliedern.
Karl

[Mux]

Hallo Karl,
ich hab mal ein bisschen rumgespielt, dabei ist folgendes Ergebnis rausgekommen, probiere es mal aus. Ich kann nicht wirklich programmieren, ist alles zusammengegoogelt und umgemodelt, aber ich glaube, es funktioniert.

Du musst den Abschnitt 2.3.4 in der Datei view.php im flohmarkt-Ordner folgendermaßen editieren

vorher:

Code:

// STEP 2.3.4. Display the flohmarkt entries (via a loop)
if ($num_entries > 0) {
    while ($entry = $query_entries->fetchRow()) {
                // Cleaning variables
                $entry_name = "";
                $entry_email = "";
                $entry_www = "";
                $entry_txt = "";
                $entry_date = "";
                $entry_time = "";  

                // A: Get the enty date and time
$entry_date = gmdate(DATE_FORMAT, $entry['posted_when']+TIMEZONE);



//$entry_time = gmdate(TIME_FORMAT, $entry['posted_when']+TIMEZONE);

// B: Get the rest of the entries

// B.1. Modify the output for the 'name'
$entry_name = stripslashes($entry['name']);

// B.2. Modify the output for 'e-mail' (image or text, 1 == True)
if ($show_image_links == '1' AND !$entry["email"] == '') {
$img_email = '<br><a href="mailto:'.$entry['email'].'">'
.'<img class="button" src="'.$WB_URL.'/modules/flohmarkt/images/mail.gif" width="14" height="17" border="0" alt="'.$entry['email'].'"'
.' title="'.$entry['email'].'" /></a>';
     $entry_email = $img_email;
} elseif(!$entry["email"] == '') {
     $entry_email = '<br /><a href="mailto:'.stripslashes($entry['email']).'">'.stripslashes($entry['email']).'</a>';
}

// B.3. Modify the output for 'homepage' (image or text, 1 == True)
if ($show_image_links == '1' AND !$entry["homepage"] == '') {
$img_www = '<img class="button" src="'.$WB_URL.'/modules/flohmarkt/images/home.gif" width="14" height="17" border="0" alt="'.$entry['homepage'].'"'
.' title="'.$entry['homepage'].'" />';
     $entry_www = $img_www;
} elseif(!$entry["homepage"] == '') {
     $entry_www = '<br />'.$entry['homepage'].'';
}

// B.4. Modify the message output.
$entry_txt = stripslashes($entry['message']);
$entry_txt = str_replace(array("\r\n", "\n", "\r"), "<br />", $entry_txt);

//B.5. Check if smileys are activated
/*if ($show_smiley == '1') {
$entry_txt = smile_replace($entry_txt);
}*/

// C: Replace vars with values
$vars = array( '[NAME]', '[EMAIL]', '[HOMEPAGE]', '[MESSAGE]', '[DATE]', '[TIME]' );
$values = array( $entry_name, $entry_email, $entry_www, $entry_txt, $entry_date, $entry_time);
echo str_replace($vars, $values, $entry_loop);
  }
}

ersetzen durch folgenden Code:

Code:

// STEP 2.3.4. Display the flohmarkt entries (via a loop)
if ($num_entries > 0) {
    while ($entry = $query_entries->fetchRow()) {
                // Cleaning variables
                $entry_name = "";
                $entry_email = "";
                $entry_www = "";
                $entry_txt = "";
                $entry_date = "";
                $entry_time = "";  

                // A: Get the enty date and time
$entry_date = gmdate(DATE_FORMAT, $entry['posted_when']+TIMEZONE);

//Limit für Anzeigealter in Tagen
$LIMIT=28;//hier die Tage eintragen!
$TS_heute = time();
$TS_Rechnung=ROUND($TS_heute / 86400);
$TS_Alter = $TS_Rechnung -$LIMIT;
$TS_Limit= $TS_Alter * 86400;


if ($entry['posted_when'] > $TS_Limit) {

//$entry_time = gmdate(TIME_FORMAT, $entry['posted_when']+TIMEZONE);

// B: Get the rest of the entries

// B.1. Modify the output for the 'name'
$entry_name = stripslashes($entry['name']);

// B.2. Modify the output for 'e-mail' (image or text, 1 == True)
if ($show_image_links == '1' AND !$entry["email"] == '') {
$img_email = '<br><a href="mailto:'.$entry['email'].'">'
.'<img class="button" src="'.$WB_URL.'/modules/flohmarkt/images/mail.gif" width="14" height="17" border="0" alt="'.$entry['email'].'"'
.' title="'.$entry['email'].'" /></a>';
     $entry_email = $img_email;
} elseif(!$entry["email"] == '') {
     $entry_email = '<br /><a href="mailto:'.stripslashes($entry['email']).'">'.stripslashes($entry['email']).'</a>';
}

// B.3. Modify the output for 'homepage' (image or text, 1 == True)
if ($show_image_links == '1' AND !$entry["homepage"] == '') {
$img_www = '<img class="button" src="'.$WB_URL.'/modules/flohmarkt/images/home.gif" width="14" height="17" border="0" alt="'.$entry['homepage'].'"'
.' title="'.$entry['homepage'].'" />';
     $entry_www = $img_www;
} elseif(!$entry["homepage"] == '') {
     $entry_www = '<br />'.$entry['homepage'].'';
}

// B.4. Modify the message output.
$entry_txt = stripslashes($entry['message']);
$entry_txt = str_replace(array("\r\n", "\n", "\r"), "<br />", $entry_txt);

//B.5. Check if smileys are activated
/*if ($show_smiley == '1') {
$entry_txt = smile_replace($entry_txt);
}*/

// C: Replace vars with values
$vars = array( '[NAME]', '[EMAIL]', '[HOMEPAGE]', '[MESSAGE]', '[DATE]', '[TIME]' );
$values = array( $entry_name, $entry_email, $entry_www, $entry_txt, $entry_date, $entry_time);
echo str_replace($vars, $values, $entry_loop);
  }}
}

ziemlich weit oben findest du im folgenden Abschnitt die Einstellung für die Tage:

Code:

//Limit für Anzeigealter in Tagen
$LIMIT=28;//hier die Tage eintragen!

Hier kannst du dann die Anzahl der Tage (hier auf 28 gesetzt) eingeben, von denen noch Einträge angezeigt werden sollen.
Mit dieser Methode werden allerdings ältere Beiträge NICHT aus der Datenbank gelöscht, das musst du dann im Backend immernoch selbst machen...

Ich hoffe, es funktioniert

Gruß
Mux

[karldehall]

Hallo Mux,
vielen Dank! Es funktioniert beim ersten Ausprpbieren.
Von wegen: "Ich kann nicht wirklich programmieren"
Das hätte ich nicht hinbekommen.
Dankeschön nochmals. Ist ein tolles Forum hier!
Gruß
Karl

[Mux]

Hallo,
ich kann noch einen draufsetzen  
hab am Modul weiter gebastelt, nun gibt es auch in den Optionen die Möglichkeit die Verfallstage einzustellen.
Ich hoffe, dass ich das richtig gezipt habe, hab gerade keine Zeit zu testen.
Dafür musst du aber das alte Modul de- und das neue installieren (oder mir ne pm schreiben, dann erklär ich dir wie du's auch ohne Neuinstallation hinkriegst)
Ich könnte aus dieser Änderung auch eine neue Version des Flohmarkt-Moduls machen, aber die vergangenen Diskussionen und Berichte über zahllose Fehlermeldungen und "Schrott" lassen mich doch etwas vorsichtiger sein.
Also wenn jemand Lust hat, das verbesserte Modul zu testen, sagt Bescheid. Um das zu veröffentlichen müssten dann noch einige Sprachdateien angepasst werden.

Gruß
Mux

[karldehall]

Hallo Mux,
Dabkeschön!
ICh werds heute abend mal testen und Dir Rückmeldung geben.
Schöner Gruß
Karl

[dbs]

habe das mal in wb2.8.2 getestet.
bleibt in optionen die anzahl tage auf 0, dann erscheint im frontend nichts (außer die zahl 0).
gibt man die anzahl tage an, dann sieht man auch die anzeige. anzahl tage steht immer drüber.

das modul ist übrigens falsch gepackt. nur den inhalt vom ordner flohmarkt darfst packen.

[Mux]

danke für die Rückmeldung,
da waren noch zwei Fehler plus der Verpackungsfehler.
Hab die zip-Datei nun aktualisiert, wenn es jetzt läuft, poste ich es ins englische Forum und bitte um Übersetzung der fehlenden Sprachdateien...

Gruß
Mux

[dbs]

ja das sieht jetzt gut aus.

kleinigkeit noch: zwischen titel und email hast du einen falschen umbruch.
also <br> statt <br />

wegen übersetzung: dazu müsstest du das modul wohl auch noch umbenennen.
kleinanzeigen, flohmarkt, guestbook ... worum gehts hier?

[Mux]

die Kleinigkeit mach ich noch weg.
Ich werde mal schaun , ob ich die Zeit finde, dann werde ich das gesamte Modul nochmal neu schreiben, sprich z.B. die ganzen Guestbook-Variablen umbenennen, die Dateinamen ebenfalls usw.
Es geht hier erstmal um die Kleinanzeigen...des halb würde ich das Teil dann umbenennen wie von Argos vorgeschlagen in "Classifieds" - zu deutsch "Kleinanzeigen".

[Mux]

so, nun hab ich das mal umgebaut. Das Modul heißt jetzt "classifieds", alle Variablen und Datenbankeinträge sind neu, also keine guestbook- oder flohmarkt-Rudimente mehr vorhanden.
Ich hab's getestet, funktioniert, aber 2,3 andere solten das bitte auch nochmal testen.
Wenn ok, dann poste ich das mal als neues Modul ins englische Forum mit der Bitte um Übersetzung der entsprechenden Einträge in den Sprachdateien..


Grüße
Mux

[dbs]

das <br> in der view.php zeile 209 ist immer noch nicht korrekt.

sonst sieht es gut aus bis jetzt.

[karldehall]

Hallo Mux,
Du bist aber fix!
Habe eben die classified-Version reingemacht.
Es funzt. Bei der gestrigen flohmarkt hat das Modul noch die Anzahl der "Verfallstage" angezeigt; das macht es jetzt nicht mehr (ist mir recht!).
Ich werde meine Nutzer darüber informieren, dass auch Anzeigen, die abgelaufen sind, unten als Anzahl angegeben werden, so lange sie noch in der Datenbank vorhanden sind. Damit kann man gut leben.
Ein kleiner Butzen: Wenn man im Backend eine Nachricht aufmacht, zeigt er statt Tel.Nr. "Webseite" an. Das ist wahrscheinlich noch eine "Altlast".
Vielen herzlichen Dank nochmal!
Gruß
Karl

[Mux]

Hallo,
neue Version - ich habe das Ganze nun noch etwas weiter aufgemöbelt:

Änderungen:
- Wenn eine Anzeige abgelaufen ist, wird sie automatisch auf "inaktiv" also auf rot gesetzt - und wird automatisch nicht mehr angezeigt.
Dadurch stimmt dann auch die Anzahl der (sichtbaren) Anzeigen wieder.
- Die "Altlast" ("Webseite" statt "Telefon" im Backend-Editierbereich) ist korrigiert.

Bitte nochmal testen. Danke  

Mux

[karldehall]

Hallo Mux,
eben grade in mein Spielsystem reingemacht.
Funktioniert wunderbar.
Wenn man am Rechner das Datum umstellt und dann ins Backend der entsprechenden Seite geht, brauchts einen Reload, dann werden die Beiträge rot markiert.
Ich hoffe, Du nimmst das nicht als Mäkelei, sondern nur als Rückmeldung 
Im Frontend sieht man, wenn alle Beiträge abgelaufen sind:
1-0 von 0
Wahrscheinlich ist die 1 irgendwo fix eingestellt, habs aber in den Optionen nicht gefunden.
Es ist einfach klasse, was Du aus meiner vorsichtigen Anfrage vom 1.3. gemacht hast 
Vielen Dank!
Gruß
Karl

[Mux]

so, weiter geht's ...

Das reload-Problem war gravierend, da musste ich gleich ganz viel umstellen

neue Version mit folgenden Änderungen:
Frontend:
- falsche Anzeige, wenn null Beiträge ("1-0 von 0") ist weg, ab jetzt werden gar keine Zahlen unter den Beiträgen angezeigt, solange die Anzeigenliste nicht mehr als eine Seite lang ist.
-  wenn der "Sofort veröffentlichen"-Button in den Backend-Optionen ausgeschaltet ist, wird nun bei der Eingabe ein Hinweis angezeigt, dass die Anzeige vor der Veröffentlichung noch überprüft werden muss.

Backend:
- auf Freischaltung wartende Anzeigen haben nun einen gelben Button
- abgelaufene Anzeigen sind rot
- manuell ausgeschaltete Anzeigen sind rot mit'm Kreuz drin
- freigeschaltete Anzeigen sind grün wie gehabt
- diverse Fehler beseitigt
- Sprachdateien deutsch und englisch angepasst

Wäre schön, wenn nochmal welche rübergucken, wwenn soweit alles läuft, poste ich das ins englische Forum


Prost
Mux

P.S. @Karl: danke für das Lob, begreif dich einfach als mein php-Lehrer, der mich mit immer neuen Aufgaben tyrannisiert  

[karldehall]

Hallo Mux,
jetzt ist der Wurm drin.
Kopfzeile:

Code:

<div align="center">sdasdasd[ADD_ENTRY]<br /><br /></div>

Das "sdasdasd" kriege ich selber raus.
Aber beim Anlegen einer neuen Anzeige verschwindet das Backend und es kommt eine Seite mit "hallooo?!"
Apropos php-Lehrer: Wär ich froh, wenn ich das selber könnte ...
Schöner Gruß
Karl

[karldehall]

Hallo Mux,
nochmal, war zu schnell.
Die Anzeigen sind doch vorhanden. Seite schließen, aus dem Backend oder aus Start Menupunkt/Seite aufrufen, dann sind sie da.
HTH
Gruß
Karl

[Mux]

Mist, da hab ich jetzt irgendwas durcheinander gebracht beim packen.

hier nochmal neu.

[karldehall]

Hallo Mux,
bingoooooooooo, es funzt! Die Buttons im Backend funktionieren und sind sehr hilfreich zum schnellen Überblick, alles sehr, sehr chic.
Vielen Dank - werd die neue Version morgen in die Intranet-Seite einbauen.
Schöner Gruß
Karl

[dbs]

hallo mux,

hab deinen neues classifieds getestet, aber mein englisch nicht so doll, deshalb hier.

es funktioniert gut soweit. gute arbeit

ein paar kleinigkeiten:
- das falsche <br> hast du noch drin
- in der benachrichtigungsma il iklebt der webseitenlink direkt am vorhergehenden text (ohne leerzeichen)
- "Ihr Eintrag wird vor der Veröffentlichung überprüft..." ist links ausgerichtet, die eingabefelder aber zentriert. absicht?
- bei telefonnummer kann man auch buchstaben einfügen
- droplets zeigen in allen feldern wirkung (außer email), aber das ist ein altes problem.

gruß
dbs

[Mux]

hallo dbs,
danke für die rückmeldung,
die br's und die Telefonrückmeldung habe ich geändert,
Telefonnummer kann nun keine Buchstaben mehr, sondern nur noch Zahlen und +-)(/\

Code:

- droplets zeigen in allen feldern wirkung (außer email), aber das ist ein altes problem.

Das verstehe ich nicht. Kannst du mir das erklären, oder mich auf einen Thread zu dem Thema verweisen?

Außerdem hab ich noch die falschen Sprachfiles entfernt, die sollten gar nicht mehr dabei sein...
hab die neue Version hier und im engl. Forum gepostet.Hab sie mal 1.01 genannt, damit man (ich) nicht durcheinander kommt.

Gruß
Mux

[Mux]

Nachdem ich informiert wurde, dass es ein ernsthaftes Sicherheitsproblem mit dem alten Guestbook-Modul gibt, das die Grundlage des classifieds-Modul ist, ziehe ich es vor, dieses Modul erstmal nicht weiter zu veröffentlichen.
Ich werde mal schauen, ob ich das classifieds-Modul auf Grundlage des aktuellen Guestbook-Moduls nochmal neu zusammenbasteln kann.
Das Sicherheitsproblem existiert demnach auch bei den beiden bei AMASP liegenden Modulen Flohmarkt und Jokebook. Es wäre demnach ratsam, die beiden Module ebenfalls zu entfernen.

Grüße
Mux