Project

General

Profile

Bug #7713

Announcement is always sent to all

Added by Andre Boivin over 4 years ago. Updated about 4 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Announcements
Target version:
Start date:
19/05/2015
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

Avec l'outil Annonce d'un cours tous les utilsateurs recoivent le message même si la case Envoyer à tous n'est pas cochée. Pourtant dans la table c_item_property la colonne to_user_id est à NULL mais dans le détail du message j'ai:
Dernière modification : Mardi 19 Mai 2015 à 08:13
Envoyé à : Tous

Associated revisions

Revision da1aba2f (diff)
Added by Julio Montoya about 4 years ago

Fix announcement visibility see #7713

Revision c8ae7304 (diff)
Added by Julio Montoya about 4 years ago

Fix error that send emails to every user see #7713

Revision 0c713278 (diff)
Added by Julio Montoya about 4 years ago

Fix error in multi select see #7713

History

#1

Updated by Yannick Warnier over 4 years ago

  • Subject changed from Announcement is always send to all to Announcement is always sent to all
  • Category set to Announcements
  • Target version set to 1.10.0
#2

Updated by Yannick Warnier about 4 years ago

  • Status changed from New to Assigned
  • Assignee set to Julio Montoya

Pour l'instant c'est encore pire: les boîtes de sélection des utilisateurs sont vides...
Je l'assigne à Julio pour quand il revient de vacances.

#3

Updated by Julio Montoya about 4 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Julio Montoya)

Je viens de corriger l'erreur.

#4

Updated by Andre Boivin about 4 years ago

  • Assignee set to Julio Montoya

Je fais quoi pour la version 1.9.8 sur test.ticfp?

#5

Updated by Julio Montoya about 4 years ago

  • Assignee changed from Julio Montoya to Andre Boivin

Andre Boivin wrote:

Je fais quoi pour la version 1.9.8 sur test.ticfp?

je ne peut pas reproduire l'erreur sur test.tic.
J'ai 2 utilisateurs avec mon email jmontoya et jmontoya2

J'envoi un annonce a jmontoya, et jmontoya2 ne recoit pas le message.

Le cours est "système de sécurité (Centre 24-Juin)"

#6

Updated by Andre Boivin about 4 years ago

  • Assignee changed from Andre Boivin to Julio Montoya

Il n'y a aucune annonce à ton nom comme expéditeur sur ce cours?? Je viens de publier une annonce sans sélectionner de destinataire, tu devrais avoir reçu quand même cette annonce.

#7

Updated by Julio Montoya about 4 years ago

  • Assignee deleted (Julio Montoya)
#8

Updated by Andre Boivin about 4 years ago

  • Assignee set to Julio Montoya

Ça fonctionne mais si on ne sélectionne pas d'utilisateur, on a seulement un message "erreur" sans autre information. J'ai remplacé la ligne 426 de announcements.php:
// $message = get_lang('ErrorOccurredAnnounce');
Display::display_warning_message(get_lang('ErrorOccurredAnnounce'));

et ajouté dans announcements.inc.php:
$ErrorOccurredAnnounce= "Sélectionnez au moins un destinataire, l'annonce n'a pas été créée";

#9

Updated by Yannick Warnier about 4 years ago

  • Status changed from Needs testing to Assigned
  • % Done changed from 0 to 50

J'ai créé la variable $AnnouncementErrorNoUserSelected plutôt (c'est plus correct).

Cela dit, Julio, je ne vois pas la même logique dans 1.10.x que celle qui est indiquée dans https://github.com/chamilo/chamilo-lms/commit/c8ae730422c7063a60d37f776343b8d4e9c13a15. Je ne sais pas si tu l'as appliquée...?

#10

Updated by Julio Montoya about 4 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Yannick Warnier

Yannick Warnier wrote:

J'ai créé la variable $AnnouncementErrorNoUserSelected plutôt (c'est plus correct).

Cela dit, Julio, je ne vois pas la même logique dans 1.10.x que celle qui est indiquée dans https://github.com/chamilo/chamilo-lms/commit/c8ae730422c7063a60d37f776343b8d4e9c13a15. Je ne sais pas si tu l'as appliquée...?

Dans 1.10.x c'est le formvalidator qui fait la validation.

#11

Updated by Yannick Warnier about 4 years ago

  • Status changed from Needs more info to Bug resolved
  • Assignee changed from Yannick Warnier to Julio Montoya
  • % Done changed from 50 to 100

Also available in: Atom PDF