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

Ion Auth initializes Email library even if use_ci_email is off #352

Closed
JoseTomasTocino opened this issue Nov 4, 2012 · 1 comment
Closed

Comments

@JoseTomasTocino
Copy link

Using CI's Email library, I was wondering "who" had set the mailtype to html, and I found out it was Ion Auth. Even having use_ci_email to FALSE (which is the default setting), it stills loads and initializes CI's email library, I think this is not correct.

@benedmunds
Copy link
Owner

Fixed. Thanks!

Dentxinho added a commit to Dentxinho/CodeIgniter-Ion-Auth that referenced this issue Jan 4, 2013
# By Ben Edmunds (9) and others
# Via Ben Edmunds (14) and Vytenis (1)
* upstream/2: (24 commits)
  Update language/lithuanian/ion_auth_lang.php
  Add ability to change groups for a user.
  Added conditional loading of the session lib/driver based on CI2 or 3 version
  - Forgot password label based on identity
  fix benedmunds#368
  Optimized create_group existing group check
  Added CI3 compatible session driver loading
  Added sess_create back to logout for CI 2
  Removed sess_create() from logout() method for CI3.0 compatibility
  Fixed reference to 'groups' table
  Some naming changes, and moved new password hashing into where it is needed.
  Removed unused $db_password from ion_auth_model::change_password().
  Some improvements in Persian translation
  Updated misguided docs for users()
  Added check for use_ci_email before loading email config.  This resolves issue benedmunds#352.
  Persian Translation
  Bug Fix, and Enhancements
  Updated spanish translation (via ibarrajo).  Closes PR benedmunds#336
  Added romanian translation (via Adrian Voicu)
  forgotten password did not work if identity was anything other than email. now it does.
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants