Karma-Plugin im Backend (2.0-beta)

[thh]

Der Menüpunkt Karma administrieren für das Karma-Plugin im Backend der 2.0-Beta scheint mir noch Verbesserungspotential zu haben. Zum einen ist der Code offenbar aus der Kommentarbearbeitung wiederverwendet und hat teilweise fehlende Übersetzungen, zum anderen scheint der Abstimmungswert nicht angezeigt zu werden?

Dazu zwei Screenshots:

• https://www.dropbox.com/s/cd2vsu55pponp ... arma-1.jpg
  • Fehlende Übersetzung der Eingabefelder
  • im Text "Kommentare" statt Karma
  • der Abstimmungswert für das Karma wird nicht angezeigt
• https://www.dropbox.com/s/2zhlo3xxd71vg ... arma-2.jpg
  • "Löschen" ist ja okay, aber "Bewilligen" macht da keinen Sinn, oder?

Alles keine Showstopper, aber ich dachte, ich dokumentiere das hier einfach mal, damit man das vielleicht irgendwann später einmal[tm] fixen kann. Ich habe bei einer Suche durch Forum und Issues nichts in der Art gefunden - falls ich einen solchen Eintrag übersehen habe, bitte ich schon jetzt um Nachsicht.

Ich verwende das Karma-Plugin mit Datenbank-Logging.

Grüßt,
-thh

[garvinhicking]

Hi!

Die fehlende Übersetzung liegt daran dass wir generell noch ein paar Übersetzungen, vor allem bei Plugins, brauchen könnten. Da ist Übersetzungshilfe gerne willkommen...

Der aktuelle Abstimmungswert scheint tatsächlich nicht berücksichtigt zu we rden, ich habe gerade im 2.0 branch einen patch committed der hoffentlich die Votes nun mit ausgibt. Gerne mal probieren ob es so geht?

Die Sprachkonstante für das Blättern wird auss dem Core genommen. Kommentare und Karmakommentare sind IMO nicht sooo verschieden dass man unbedingt einen neue, unübersetzte Sprachkonstante dafür aufmachen müsste?

Der "Bewilligen"-Button hatte für mich keine erkennbare Funktionalität, den habe ich mal entfernt, danke!

Viele Grüße,
Garvin

[thh]

Die fehlende Übersetzung liegt daran dass wir generell noch ein paar Übersetzungen, vor allem bei Plugins, brauchen könnten. Da ist Übersetzungshilfe gerne willkommen...

Ich schau mal, ob ich durchschaue, wie da der Workflow ist ...

Der aktuelle Abstimmungswert scheint tatsächlich nicht berücksichtigt zu we rden, ich habe gerade im 2.0 branch einen patch committed der hoffentlich die Votes nun mit ausgibt. Gerne mal probieren ob es so geht?

Getestet - sehr schick, danke! Sieht gut aus.

Die Sprachkonstante für das Blättern wird auss dem Core genommen. Kommentare und Karmakommentare sind IMO nicht sooo verschieden dass man unbedingt einen neue, unübersetzte Sprachkonstante dafür aufmachen müsste?

Ja, klar. Das ist mehr Nitpicking. (Genau wie die Spielerei, ob man ggf. den Singular für einen einzelnen Kommentar ("1 Kommentare insgesamt") bilden will. )

Für 3.0 soll ja auch noch was übrig bleiben. *g*

Der "Bewilligen"-Button hatte für mich keine erkennbare Funktionalität, den habe ich mal entfernt, danke!

Ich habe zu danken!

Nachdem der HTML-Validator sich über das durch das Plugin generierte Markup für die Links arg beschwert, bin ich da mal drübergegangen und habe das geändert ([] in URLs müssen wohl encoded werden) und getestet. Die Änderungen habe ich mit zwei vergleichbaren Anpassungen als pull request bei Github eingekippt.

Grüße,
-thh

[thh]

Die fehlende Übersetzung liegt daran dass wir generell noch ein paar Übersetzungen, vor allem bei Plugins, brauchen könnten. Da ist Übersetzungshilfe gerne willkommen...

Ich schau mal, ob ich durchschaue, wie da der Workflow ist ...

Hm, wenn ich das richtig sehe, sind die Benennungen der Filterfunktionen hard-coded.

Code: Select all

        <div class='clearfix'>
            <div class='form_field'>
                <label for='serendipity_filter_useragent'>User Agent</label>
                [...]
            </div>

            <div class='form_field'>
                <label for='serendipity_filter_ip'>".IP."</label>
                [...]
            </div>

            <div class='form_field'>
                <label for='serendipity_filter_entryid'>Entry ID</label>
                [...]
            </div>

            <div class='form_field'>
                <label for='serendipity_filter_title'>Entry title</label>
                [...]
            </div>
        </div>

Da fummele ich mal erstmal lieber nicht dran rum ...

-thh