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

Question help needs changing: "person" referred to as "item" multiple times - RDMO, DFGv5 catalog #703

Closed
sa-wetter opened this issue Aug 27, 2023 · 14 comments · Fixed by rdmorganiser/rdmo-catalog#206

Comments

@sa-wetter
Copy link
Member

Hi,

I have come across this bug multiple times. I am working with the RDMO, the brief questionnaire and DFGv5 catalog.

in the RDMO catalog: a question is e.g. asking for persons/institutions involved in the project coordination and the question help then refers to them as "items".
items instead of names

Here an example from the DFGv5 catalog:
special_responsibility

I know this is tedious, but someone should look into this. People, institutions, project partner etc. are not items. However, this bug is also not consistent since many times this specific question help is actually also fine (-> first screenshot: top question gets the "disciplines" right). Maybe the word "items" can be a variable, depending on the question. Or if it is already a variable then its value is not assigned correctly.

Best,
Sarah

@Zack-83 Zack-83 transferred this issue from rdmorganiser/rdmo-catalog Sep 12, 2023
@jochenklar
Copy link
Member

@Zack-83 I think this is a problem in the catalog not in the code. vebose_name and verbose_name_plural are not set in the xml files, I guess.

@Zack-83
Copy link
Member

Zack-83 commented Sep 18, 2023

OK, wir kümmern uns darum, in dieser neuen Pull Request: https://github.com/rdmorganiser/rdmo-catalog/pull/206/files

@sa-wetter : Kannst du bitte weiter editieren? Der RDMO-Katalog ist in /rdmorganiser/questions und der DFG-Katalog in /shared/ub_fau_erlangen_nuernberg

@jochenklar : Kannst du uns bitte lotsen? Betrifft das alle Fragen mit <is_collection>True</is_collection>?
Lohnt es sich bis zum Erscheinen der Version 2.0 abzuwarten, als die Kataloge modular aufgebaut sein werden?

@sa-wetter
Copy link
Member Author

@Zack-83 mach ich.

@jochenklar
Copy link
Member

@Zack-83 ja, fast, wenn is_collection werden die Buttons angezeigt. Nur bei widget_type=checkbox nicht. Wenn ihr bis 2.0 wartet, können die XML files auch nur von 2.0 importiert werden. Die XML files sehen auch (leicht) anders aus. Ich würde erstmal beim alten Format bleiben.

@jochenklar
Copy link
Member

@Zack-83 und bei Fragensets mit is_collection=True natürlich auch.

@sa-wetter
Copy link
Member Author

So wie es aussieht ist das ganze etwas komplizierter: ich habe es auf einem Testserver mal geändert und bin auf einen Fehler gestoßen.
Im Bild zwei Beispiele.
Screenshot 2023-09-21 at 15 57 37

Wenn die verbose_name und verbose_name_plural für Objekte wie "Schlagwort" benutzt werden, macht der Satz Sinn. Wenn man allerdings nach Namen und fragt, dann muss man die Grammatik beachten und dann kommt es zu Fehlern.
Da in beiden Sätzen das verbose_name_plural benutzt wird, aber im zweiten der Fall manchmal beachtet werden muss.

Fällt euch etwas ein?

@jochenklar
Copy link
Member

Ja, das ist genau das Problem das ich im Meeting meinte: #59 und #602. Ich werde da demnächst dran arbeiten, und es soll Teil von RDMO 2.1.0 werden.

@sa-wetter
Copy link
Member Author

@jochenklar ich lasse meinen Vorschlag mal hier stehen, bevor ich ihn wieder vergesse:
"Bitte nutzen Sie für die Schlagwörter/Verantwortlichen jeweils eine Zeile. Mit dem grünen Button können Sie weitere hinzufügen und mit dem blauen Kreuz (wieder) entfernen." Somit entfällt zumindest das Problem im zweiten Satz. Ich schaue mal, bei welchen Fragen/ verbose_name_plural das doch nicht klappt. Bei den Fragen auf Englisch gäbe es zumindest keine Probleme.

@jochenklar
Copy link
Member

Ja, vll nutzen wir einfach das Issue hier um das durchzudiskutieren. Ich glaube langsam auch das wir eher eine generische Formulierung finden sollten anstatt das anpassbar zu machen. Noch mehr Felder wie verbose_name macht es einfach noch komplizierter Kataloge zu bearbeiten. Vll werden wir ja verbose_name_plural auch noch los.

@jochenklar
Copy link
Member

jochenklar commented Sep 29, 2023

Das einfachste wäre, wenn wir verbose_name_plural entfernen, und verbose_name nur für die Buttons benutzen. Die grauen Hilfetexte könnten wir generisch Formulieren (aber mit entry statt item. Also:

Please enter the entries line by line. You can add entries using the green button and remove them using the blue cross (×).

Bitte nutzen Sie für die Einträge jeweils eine eigene Zeile. Sie können weitere Einträge mit dem grünen Button hinzufügen und mit dem blauen Kreuz (×) entfernen.

Bei dem Hilfetext für die Page-collections ist das nicht so gut machbar:

Bitte füllen Sie das Formular für jeden Eintrag aus. Die verschiedenen Einträge werden eventuell in späteren Fragen wieder verwendet. Sie können einen neuen Eintrag mit dem grünen Button hinzufügen. Bereits angelegte Einträge können mit den Buttons oben rechts bearbeitet oder wieder entfernt werden.

klingt nicht passend.

Ein Problem ist auch das der verbose_name eigentlich auch nicht reicht. Wenn ich "Person hinzufügen" auf Französisch übersetze, dann bekomme ich "Ajouter une personne" nicht "Ajouter personne", also mit Artikel. Das deutet eher darauf hin das es mit dem zusammensetzen von verbose_name und template-Übersetzung nie zufriedenstellend funktionieren wird.

@jochenklar
Copy link
Member

Hier nochmal die Liste wo verbose_name und verbose_name_plural vorkommen:

  • add field button (question)
  • add field help (question, plural, probably buggy)
  • add set button (question, should be questionset?)
  • add set help (question, plural, should be questionset?, probably buggy)
  • delete value set (page)
  • value set form (page)
  • questionset head (page)
  • questionset head help (page, this issue)
  • remove field (question)
  • remove set (page)

@jochenklar
Copy link
Member

Ok, jetzt habe ich eine Idee: generische Hilfetexte und Symbole auf den Buttons und nur noch verbose_name.

Siehe:

Screenshot 2023-09-29 at 19-05-22 example com

Screenshot 2023-09-29 at 19-15-51 example com

@sa-wetter
Copy link
Member Author

@jochenklar das ist eine richtig gute Idee!

@Zack-83
Copy link
Member

Zack-83 commented Nov 13, 2023

Teilweise in rdmorganiser/rdmo-catalog#206 , teilweise durch Umstrukturierung in RDMO2.0 gelöst

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 a pull request may close this issue.

3 participants