Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

extend author field to indicate the community #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

extend author field to indicate the community #4

wants to merge 1 commit into from

Conversation

spookey
Copy link

@spookey spookey commented Sep 11, 2014

Hallo,
hier mein per Mail angekündigter Feature-Request.

Das Autorenfeld im Feed wird durch die jeweilige Community erweitert, damit man die einzelnen Einträge auf einen Blick zuordnen kann.

Das letzte mal, als ich ernsthaft was mit PHP gemacht habe, war vor gefühlten 20 Jahren ~ ebenso ist XML nicht unbedingt meine Welt. Bin mir unschlüssig, ob ein reiner String nicht die Feedreader verwirrt, möglicherweise muss man noch ein bisschen nachbessern. 😸

Würde mich sehr freuen, wenn dieses Feature Einzug findet.

@andibraeu
Copy link
Member

Hallo, wie schon in meiner Email geschrieben, wird die Community in das Source-Tag geschrieben. Das Author-Tag habe ich mit Absicht nicht verwendet, um niemanden vorzuspielen, dass hier jemand der Autor oder Urheber eines Eintrags ist.

auf freifunk.net wird auch das Source-Tag ausgewertet, um die Quellcommunity zu zeigen

@spookey
Copy link
Author

spookey commented Sep 11, 2014

Kein mir bekannter RSS-Reader wertet das Source-Tag aus, und zeigt dieses auch an.
Das Author-Tag nutzen hingegen alle.

Zweitens: Wenn Nutzer X für Community Y einen Eintrag verfasst, dann ist X sowohl der Autor als auch der Urheber des Eintrags!
Wem wird dann also eine falsche Information untergejubelt, wenn anstatt Autor: X folgendes im Feed steht: Autor: X (Y)?

Den Aggregator auf freifunk.net nutze ich (wie womöglich viele andere) nicht, ich nutze einen Feedreader.

@andibraeu
Copy link
Member

Dann machen das aber die Newsreader falsch. Das Source-Tag ist genau dafür vorgesehen: http://cyber.law.harvard.edu/rss/rss.html#ltsourcegtSubelementOfLtitemgt

Das Tag, das du verändern willst ist übrigens nicht das Author-Tag.

Ich möchte eigentlich so wenig wie möglich Daten in den aggregierten Feeds ändern. Felder mit Autoren sind oft gesetzt.

Du kannst deine Idee aber gern umsetzen. Bevor ich es hier aber übernehme möchte ich, dass das Feature konfigurierbar ist. Jeder, der sich eine Instanz des Aggregators installiert kann dann selbst entscheiden, ob er das so möchte.
Wo kann man deine Änderungen testen?

@spookey
Copy link
Author

spookey commented Sep 15, 2014

Vielen Dank für das Feedback!

Das Source-Tag ist mir bekannt, doch leider ist RSS und Atom ein Thema voller Missverständnisse, so dass man eher in Richtung Pi mal Daumen anstatt nach Spezifikation implementieren muss. Zum Beispiel sollte das Author-Tag eine Mail-Adresse stehen ~ kein mir geläufiger Feed-Generator tut das aber.
(Ich hatte mal einen Text gelesen, der die gängigsten Fehler von RSS-Generatoren auflistet, und wie man sich am besten beim implementieren eines Parsers drum rum windet, damit es trotzdem gut funktioniert. Leider ist mir der Titel entfallen, so dass ich nicht direkt danach suchen kann 😟)

Wirft man den aggregierten Feed in den Feedvalidator kommen entsprechend viele Warnings dabei raus ~ womöglich werden einige Fehler aus den Original-Feeds mit durchgereicht - Es ist wohl eine rein philosophische Frage, ob es die Aufgabe dieses Scriptes ist diese zu korrigieren, oder unverändert durchzureichen...

Hier kann man vorerst mal die Erweiterung des Author-Tags antesten. (Ist nicht dauerhaft, nur zum testen, also irgendwann auch wieder weg.)

Werfe ich diesen Feed in meinen Reader, sehe ich, dass ich noch ein bisschen nachbessern muss.

Weiterhin - dieses Feature konfigurierbar zu machen ist eine sehr gute Idee, werde mich die Tage nochmal dran setzen, falls jemand anderes schneller sein sollte, gebührt diesem Ruhm und Ehre 😸

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants