Project

General

Profile

Bug #6535

Terms and conditions on course popup if following link

Added by Yannick Warnier about 7 years ago. Updated almost 7 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Daniel Barreto
Category:
Course settings
Target version:
Start date:
11/08/2013
Due date:
% Done:

100%

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

Description

When settings terms and conditions on a course, the first time, I have to accept them. This is OK.
The problem is when I follow a link that I receive by e-mail (for example a task handed over or a forum update), if I'm not connected to the platform at that moment, I get the terms and conditions again, each time, and after that it tells me I'm not authorized to enter the platform and I have to go to the login.
That's just 3 pages when I could have had only one (with the login box).
Actually, sometimes it even gets stuck on the terms and conditions page... (which is even worse).

Make sure in the future I just get a login box...

Example link: https://cursos.beeznest.com/main/forum/viewthread.php?cidReq=CHAPA19ESB&id_session=2&gidReq=0&forum=10&thread=17 (you can try it, even if you can't login, at least you'll see the problem).

Associated revisions

Revision 43a9f5d5 (diff)
Added by Yoselyn Castillo about 7 years ago

removing term an conditions check y user foollow a link -refs #6535

Revision c0c673ef (diff)
Added by Yoselyn Castillo about 7 years ago

removing term an conditions check y user foollow a link -refs #6535

History

#1

Updated by Yannick Warnier about 7 years ago

  • Description updated (diff)
#2

Updated by Yoselyn Castillo about 7 years ago

checking...

#3

Updated by Yoselyn Castillo about 7 years ago

  • % Done changed from 0 to 70

Done! I just have avoided asking for legal agreement if user is anonymous. This doesn´t make sense and was adding an unnecessary page when a user followed a link.
https://github.com/chamilo/chamilo-lms/pull/125

#4

Updated by Julio Montoya almost 7 years ago

merged

#5

Updated by Yoselyn Castillo almost 7 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Yoselyn Castillo)
#6

Updated by Yannick Warnier almost 7 years ago

  • Assignee set to Daniel Barreto
#7

Updated by Daniel Barreto almost 7 years ago

  • Status changed from Needs testing to Bug resolved
  • % Done changed from 70 to 100

Tested and working, requested for login and not terms and conditions

Also available in: Atom PDF