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

issue: sendAccessLink On NULL v1.11 #4829

Merged

Conversation

Projects
None yet
2 participants
@JediKev
Copy link
Member

commented Apr 4, 2019

This addresses an issue in v1.11 where sending an Access Link via email to a Collaborator fails with fatal error. This is due to the type of User class that is sent to the Mailer::send() function. The ClientSession User class is not appropriate for this method, rather, TicketOwner or Collaborator User classes are appropriate. This adds a function called getSessionUser() to the ClientSession class so we can retrieve the TicketOwner/Collaborator User class from the session.

issue: sendAccessLink On NULL v1.11
This addresses an issue in v1.11 where sending an Access Link via email to a
Collaborator fails with fatal error. This is due to the type of User class
that is sent to the `Mailer::send()` function. The ClientSession User class
is not appropriate for this method, rather, TicketOwner or Collaborator User
classes are appropriate. This adds a function called `getSessionUser()` to
the ClientSession class so we can retrieve the TicketOwner/Collaborator User
class from the session.
@JediKev

This comment has been minimized.

Copy link
Member Author

commented Apr 4, 2019

This needs #4828 merged into develop then develop needs to be merged into develop-next first before merging this pull request to develop-next. 😁

@protich protich merged commit 4d8de20 into osTicket:develop-next Apr 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.