ckruse / cforum Goto Github PK

Write exhaustive unit, integration and functional tests

Wenn man auf seinen Account geht kann man seine Einstellungen nicht ändern, man muss erst ganz unten den grauen unscheinbaren Knopf "Bearbeiten" klicken damit man zur edit ansicht gelangt.

Mir ist nicht ganz klar wozu man eine show ansicht mit disabelten input-Feldern eigentlich braucht, sinvoller erscheint mir dass wenn ich auf meinen Account gehe ich sofort in der edit ansicht bin und sachen verändern kann.

Alle Symbole sollten ein Title-Attribut (oder anderweitigen Tooltip) bekommen, damit man ihre Bedeutung auch in Worten entnehmen kann.

• Pager für die All-Ansicht
• Benutzerverwaltung: Anzahl der Nutzer pro Seite in der Übersicht

Im Benutzerbereich sind an einigen Stellen noch die Englischen Übersetzungen hinterlegt. So steht als Button z.b. noch "Back" drinn und bei "Threads schliessen wenn gelesen" das kleine "Yes".

Beim Ändern von Benutzerdaten hatte ich vergessen mein Passwort zu wiederholen. Die Fehlermeldung hat mich darüber nur mit einem "Error" informiert. Hier wäre eine Prüfung vor dem Absenden (javascript, ob beide Felder gefüllt sind) praktisch. Als "Normale" Fehlermeldung wäre ein rot makiertes Feld angenehmer, damit ich direkt sehe, was ich falsch gemacht habe!

Nach der Registrierung kann man sofort arbeiten und schreiben, erst nach einem Ausloggen kann man sich nicht wieder neu einloggen.

Selfhtml wird das vielleicht nicht brauchen, aber ich finde, dass zu den Standard-Forenberechtigungen "Kein Zugriff", "Lesen" und "Schreiben" noch "Schreiben nach Anmeldung" hinzukommen sollte.
Eventuell auch noch "Lesen nach Anmeldung", wobei ich mir hier noch nicht sicher bin, ob das überhaupt sinnvoll ist und auch noch nach "Forum(sname) ist sichtbar, aber sein Inhalt nicht ohne Anmeldung" aufgeteilt werden sollte.

In der Posting-Ansicht wären Bedienelemente zum Löschen des gesamten Thread, mindestens aber dem aktuellen Posting ganz nett. Zurück zur Thread-Ansicht und das Posting dort suchen ist etwas umständlich.

Ein Feld für den Lösch-Grund (als Dialog vielleicht?) und wer es getan hat, wären sinnvoll.

"We're sorry, but something went wrong."

Offenbar, weil der Betreff zu kurz ist. Ich erhalte aber die Nachricht, dass mein Posting erfolgreich gespeichert wurde.

When I try to change my password and misspell it in the confirmation input then it doesn't say that, the error message says that I don't have permission to change what I wanted to change.

Unangemeldete und Nutzer ohne Leserecht bekommen beim Zugriff auf nicht öffentliche Foren nur die allgemeine Fehlermeldung "We're sorry, but something went wrong." und keine Aussage über den wirklichen Grund.
Gewünscht: spezifischere Fehlermeldung, besser: Fehlermeldung und Anmeldeseite oder wenn er schon angemeldet ist, dann nur Fehlermeldung.

Wenn "Einträge pro Seite" ausgefüllt ist, wird zwar der Pager korrekt dargestellt, aber nur auf Seite 1 sind Postings zu sehen.

Tags sind derzeit ohne Leerraum aneinandergehängt.

Erstmal: Herzlich willkommen, liebe Vorschau!

Aber leider schaut sie nicht nur vor, sondern speichert auch gleich. Ein anschließendes Speichern bringt einen "The page you were looking for doesn't exist."

Wäre es möglich, ein "Posting ungelesen machen" einzubauen oder meinetwegen auch ein anderweitiges Markieren ("interessant" o.ä.), woraufhin der Thread aufgeklappt bleibt. Ziel ist es, später darauf antworten zu können und sich so zu merken, dass man das wollte.

Es wäre schön zumindest die Möglichkeit zu haben, Text fett und kursiv setzen zu können in Postings.

Vorschlag zur Umsetzung: BB Code (plus entsprechende Buttons beim Eingabefeld)

Siehe http://forumtest.selfhtml.org/cforum/2013/jan/14/secured-name/142 ff.

Beim erstellen von Nachrichten, sind die ersten Felder vom Design her ausgefüllt. Ich nehme da aber überhaupt nicht wahr, dass dies nur Plathalter sein sollen. Hier wäre eine abgeschwächte Schriftfarbe praktischer!

"Nacht wurde erfolgreich wieder hergestellt" sagte der Mond zur Sonne.

Für ein Forum sollte "Öffentlich" unterteilt werden in Nur-Lesen und Lesen/Schreiben.

zumal der Löschversuch natürlich ins Leere läuft.

oder es sollte eine Funktion geben "angemeldet bleiben (mit diesem Broswser maximal x Tage)" Es müssen ja nicht 180 sein, wie in der WIkipedia.

Geht zwar prinzipiell, aber als Ergebnis kommt "die" Fehlermeldung.

Wenn ich einen Hervorgehobene Benutzer eintragen möchte, bekomme ich beim Speichern eine Fehlermeldung. Optimierungswünsche an der Stelle wäre die Abschaffung der Kommaseparierten Liste, wobei auch nicht angemeldete User hervorhebenswert sind. Eventuell kann dort der "%" Operator als Like ermöglicht werden!

Ein Nutzer (in einem nicht öffentlichen Forum) nur mit Leseberechtigung bekommt bei einem Posting den Antwort-Link (ohne Übersetzung) zu sehen und beim Aufrufen die allgemeine Meldung.
Gewünscht: Antwort-Link weg und eine spezifischere Meldung beim Aufruf der Antwort-URL.

Leider hab ich noch keine Option für den Automatischen Reload gesehen. Wäre toll wenn das integriert wäre, damit man es nicht erst noch groß in die eigenen JS-Dateien schreiben müsste!

Irgendwo hatten wir das schon:

Man klickt ganz häufig versehentlich auf den Erstellerlink statt auf die Nachricht. Daran hat auch die neue Zweizeiligkeit nichts geändert.

Warum muss der Ersteller überhaupt verlinkt sein? Welchen Mehrwert bietet:

"Benutzer ckruse

User has been created at Donnerstag, 15. November 2012, 20:52 Uhr. He created 28 messages. User is an admin. User is active. "

Wer will/muss das wissen?

Achja, wenn ich eine Antwort verfasse, dann habe ich zwar (nun) das Zitat im Text stehen - aber nur solange ich es noch nicht weggekürzt habe. Dann sehe ich nicht mehr den Text auf den ich antworte und ich müsste mich anderweitig in einem neuen Tab zum Ausgangstext bewegen. Sinnvoller fände ich, wenn der Text weiterhin zu sehen ist. Das heißt, es müsste zwischen new und answer unterschieden werden (oder zwischen /new und /forum/thread/posting/new).

Von Dedlfix

Falls nein, liegt es daran, dass es ein a-Element ist.

Ich hab die Session verfallen lassen und erhalte zwei Meldungen (div class alert alert-error)

• Deine Anmeldung ist abgelaufen, bitte melde Dich erneut an.
• true

Wenn man eine neue Nachricht erstellt, die Schaltfläche sollte besser "auf dies Nachricht antworten" heißen, und sich die Vorschau anschaut, wird nach einem Abbruch die Nachricht trotzdem erstellt.

Wenn ich einen neuen Thread erstelle, wird in der Vorschau als Betreff "neuer Thread" angezeigt, statt meines gewählten Betreffs.

Da du sinnvollerweise nicht nach Posteingang und Postausgang trennst, sollte

• die Überschrift nicht "Post von ... " sondern "Kommunikation mit ..." heißen
• der Link rechts zum Verfassen einer neuen Mail immer zum Partner und nicht zu einem selbst weisen

Die Links unter Autor, Empfänger und Betreff sind identisch?

Die Lokalisierung von "Secured Name" in den Persönlichen Informationen fehlt.

Ich bin mir nicht sicher, ob es sonderlich nützlich für den Anwender ist, die Tag Cloud über komplett alles zu führen. Da stehen vermutlich sowieso nur die üblichen Verdächtigen ganz groß drin. Vielleicht könnte es hilfreich sein, eine zweite Tag-Cloud über die aktuellen Postings zu bilden, oder auch anderweitig die aktuellen Tags zu präsentieren, zum Beispiel als Liste.

imho braucht da nicht "Metainformationen" und "Inhalt" als Überschrift stehen. - Was soll es sonst sein?

Sollte ein Eintrag den ich grad selbst verfasst habe nicht automatisch schon als gelesen markiert sein? Sonst denke ich die ganze zeit dass jemand etwas neues geschrieben hat das ich noch nicht gelesen habe.

Ich habe heute (Sonntag, 25.November) dein Posting unter bodybuilding wieder hergestellt. Dies wird jetzt als letzte Änderung gezeigt. Das Posting war aber schon viel früher. Das könnte zu Verwirrungen führen.

Was ich aufgrund der Erfahrungen im bisherigen Forum schön fände wäre, wenn bei Beiträgen von Benutzern ohne Registrierung (anonym) ersichtlich wäre, wenn der OP eines Threads antwortet (also irgendeine Cookie basierte Lösung - natürlich nur, wenn der jeweilige User Kekse akzeptiert - wenn nicht, ist das halt Pech).

Der Vorschlag geht auf folgenden Beitrag zurück: http://forum.intern.selfhtml.org/cforum/2013/jan/14/secured-name/132

Gruß
Gunther

dedlfix: Das Icon links vom Thread verweist zwar auf open=x, trotzdem bleibt bei mir der Thread zu.

ckruse: Der Thread ist vollständig gelesen?

Das aufklappen mit dem Zuklappenwenngelesen-Feature ist noch nicht implementiert, da mir noch nicht 100% klar ist, wie ich das sauber implementieren kann.

dedlfix: Ja, alles ist gelesen und zugeklappt hat er sich. Deshalb will ich ihn ja öffnen.

Ich sehe auf http://forum.intern.selfhtml.org/test die beiden zugeklappten und klicke auf den Ordner, aber außer dass in der URL nun http://forum.intern.selfhtml.org/test?open=2 steht ändert sich nichts.

http://forum.intern.selfhtml.org/test/2012/nov/20/fehler-beim-loschen/57

Benutzernamen mit Großbuchstaben sind im Moment nicht erlaubt. Dies ist weder auf der Registrierungsseite dokumentiert noch aus der Fehlermeldung erkennbar:

Konnte benutzer nicht speichern: ein Fehler.

Benutzer-Name ist nicht gültig

https://github.com/ckruse/cforum/blob/master/Gemfile.lock#L96
3.2.10 ist vulnerable
http://weblog.rubyonrails.org/2013/1/8/Rails-3-2-11-3-1-10-3-0-19-and-2-3-15-have-been-released/
Sollte aktualisiert werden, oder?

P.S. Schön zu sehen, dass es hier voran geht!

64 Zeichen lässt sich streiten, aber wenn es eine Begrenzung gibt, dann bitte schon bei der Eingabe und nicht erst, wenn der neue Thread erstellt werden soll.

Grund dafür, dass der Fehler überhaupt auftritt, ist die identische Formatierung von placeholder und tatsächlichem Inhalt.

Posting kann nicht abgeschickt werden, da

• betreff fehlt
• betreff zu kurz ist

Ich wollte gerade ein JavaScript Forum aufsetzen und habe da als slug und Kurzname "js" genommen weil das die einzig sinnvolle Kurzschreibweise ist aber der Validator lässt mich das nicht benutzen, er meint die Namen müssten mindestens 3 Zeichen sein.

Im Hinblick auf andere Foren und Boards wie z.B. 4chan kann ich nicht nachvollziehen warum die Namen der foren nun unbedingt mindestens 3 Zeichen lang sein müssten. Dort sind die Namen auch nur einen Buchstaben lang wie a, c oder f und es funktioniert primast.