nextcloud/server

Plural missing on CalDAV Activity when creating multiple Calendar events

Open

#5528 aperta il 24 giu 2017

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)PHP (4865 fork)batch import
1. to develop32-feedbackbugfeature: activity and notificationfeature: caldavfeature: language/translations (l10n/i18n)good first issuepapercut

Metriche repository

Star
 (34.953 star)
Metriche merge PR
 (Merge medio 20g) (627 PR mergiate in 30 g)

Descrizione

Steps to reproduce

  1. Create multiple events around the same time on Nextcloud Calendar
  2. Look at your Activities
  3. An activity stating "You created event Event 1, Event 2..."

Expected behaviour

If multiple events are created, activity should read as "You created events Event 1, Event 2..."

Actual behaviour

The word "event" is not plural.

Server configuration

Nextcloud 12

Operating system: N/A

Web server: N/A

Database: N/A

PHP version: N/A

Nextcloud version: 12

Updated from an older Nextcloud/ownCloud or fresh install: N/A

Where did you install Nextcloud from: git source

Client configuration

Browser: Noticed on Nextcloud-dev Android client, but client agnostic

Relevant code seems to be here: https://github.com/nextcloud/server/blob/master/apps/dav/lib/CalDAV/Activity/Provider/Event.php#L89

Guida contributor

Direzione di ricerca
Trova la stringa in apps/dav/lib/CalDAV/Activity/Provider/Event.php intorno alla riga 89 e aggiungi la logica di pluralizzazione per la parola 'event'.
Tech stack
php
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PHP
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
95