Feature Requests

Please do NOT use this section of the forum to request help if Joomdle does not work for you in some particular way. Your post will be deleted.

[Closed] Joomdlehooks breaks changing profile type JomSocial

  • Ben Cheek
  • Topic Author
  • Offline
  • New Member
  • New Member
More
14 years 3 months ago - 13 years 5 months ago #1 by Ben Cheek
I've set up JomSocial 2.4.2 to use the native Multiple Profiles functionality (Components >> JomSocial >> Multiprofile). Users can select from several profile types, some of which have different fields and which require admin approval (like in the case of faculty). Users have the opportunity to change their profile on the Profile Edit page. If they make the change to a profile type that does not require approval, everything works as expected. If they make the change to one that does require approval, they are logged out as expected, but the expected message does not show (instead they're sent to the site index) and infact their profile type is not changed or disabled for approval.

Shutting off joomdlehooks returns JomSocial to the expected behavior.

Could I have confirmation of this issue and/or some guidance about how to correct?
Last edit: 13 years 5 months ago by Antonio Durán.
The topic has been locked.
More
14 years 3 months ago #2 by Antonio Durán
Replied by Antonio Durán on topic Joomdlehooks breaks changing profile type in JomSocial
Thanks for the bug report. I will try to replicate it when I have the time and let you know what I find.
The topic has been locked.
More
14 years 3 months ago #3 by Antonio Durán
Replied by Antonio Durán on topic Joomdlehooks breaks changing profile type in JomSocial
Hi again.

I tested and could replicate the problem. A quick look at Jomsocial code shows what is happening: Jomsocial first logs the user out, and later saves his new profile type and redirects the user.

Problem is Joomdle uses redirection to log the user out of Moodle, so code after the logout call is never executed.

There is an easy workaround we can implement, but it will only work when both Jooomla and Moodle are on the same domain.
If you want to test it, you just need to edit the joomdlehooks plugin, and comment out all the code between /* NEW */ comments.

I guess we should include an option for redirect-less log out, so you can use the procedure that better suits your needs.
The topic has been locked.
  • Ben Cheek
  • Topic Author
  • Offline
  • New Member
  • New Member
More
14 years 3 months ago #4 by Ben Cheek
Thanks! You're providing really great support for your software!

I've implemented the change above (for quick reference of readers, this was lines 38-44 in the 0.7 version of /plugins/user/joomdlehooks.php). Look forward to seeing the setting you mention in future versions of Joomdle.
The topic has been locked.
More
14 years 3 months ago #5 by Chris
Moved to feature requests for future work
The topic has been locked.