X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=portal%2Femailactivationview.py;h=02f56ddbc825d6a6e693f50e5e049435e04e4a77;hb=829c3ba3d1bcf6e1aae4e21a8f60ec72fc815340;hp=4680256ca8285547c98d5409b78dc4cda8e5aefa;hpb=f510a89e0f52c4756c564339d638b4018342ad65;p=myslice.git diff --git a/portal/emailactivationview.py b/portal/emailactivationview.py index 4680256c..02f56ddb 100644 --- a/portal/emailactivationview.py +++ b/portal/emailactivationview.py @@ -1,30 +1,53 @@ -from unfold.loginrequired import FreeAccessView -# -from manifold.core.query import Query -from manifoldapi.manifoldapi import execute_query, execute_admin_query -from portal.actions import manifold_update_user, manifold_update_account, manifold_add_account, manifold_delete_account, sfa_update_user, authority_get_pi_emails, make_request_user, create_user -# -from unfold.page import Page -from ui.topmenu import topmenu_items_live, the_user -# +import json +import os +import re +import itertools + from django.http import HttpResponse, HttpResponseRedirect from django.contrib import messages from django.contrib.auth.decorators import login_required -from myslice.theme import ThemeView -from portal.models import PendingUser from django.core.mail import EmailMultiAlternatives, send_mail -from django.contrib.sites.models import Site -# -import json, os, re, itertools +from manifold.core.query import Query +from manifoldapi.manifoldapi import execute_query, execute_admin_query + +from unfold.loginrequired import FreeAccessView + +from portal.actions import ( + manifold_update_user, manifold_update_account, manifold_add_account, + manifold_delete_account, sfa_update_user, authority_get_pi_emails, + make_request_user, create_user, send_email_to_pis) +from portal.models import PendingUser, PendingAuthority + +from unfold.page import Page +from ui.topmenu import topmenu_items_live, the_user + +from myslice.theme import ThemeView +from myslice.settings import logger def ValuesQuerySetToDict(vqs): return [item for item in vqs] - # requires login class ActivateEmailView(FreeAccessView, ThemeView): template_name = "email_activation.html" + def is_ple_enabled(self, pending_user): + pending_authorities = PendingAuthority.objects.filter(site_authority__iexact = pending_user.authority_hrn) + if pending_authorities: + return False + pending_user_email = pending_user.email + try: + query = Query.get('myplcuser').filter_by('email', '==', pending_user_email).select('enabled') + results = execute_admin_query(self.request, query) + for result in results: + # User is enabled in PLE + if 'enabled' in result and result['enabled']==True: + return True + except Exception as e: + logger.error("Exception in myplc query = {}".format(e)) + + return False + def dispatch(self, *args, **kwargs): return super(ActivateEmailView, self).dispatch(*args, **kwargs) @@ -33,13 +56,21 @@ class ActivateEmailView(FreeAccessView, ThemeView): page = Page(self.request) #page.add_js_files ( [ "js/jquery.validate.js", "js/my_account.register.js", "js/my_account.edit_profile.js" ] ) #page.add_css_files ( [ "css/onelab.css", "css/account_view.css","css/plugin.css" ] ) + if self.request.is_secure(): + current_site = 'https://' + else: + current_site = 'http://' + current_site += self.request.META['HTTP_HOST'] for key, value in kwargs.iteritems(): - #print "%s = %s" % (key, value) if key == "hash_code": hash_code=value - if PendingUser.objects.filter(email_hash__iexact = hash_code).filter(status__iexact = 'False'): + + #if PendingUser.objects.filter(email_hash__iexact = hash_code).filter(status__iexact = 'False'): + if PendingUser.objects.filter(email_hash__iexact = hash_code): activation = 'success' + pending_users = PendingUser.objects.filter(email_hash__iexact = hash_code) + pending_user = pending_users[0] # AUTO VALIDATION of PLE enabled users (only for OneLab Portal) if self.theme == "onelab": @@ -48,63 +79,37 @@ class ActivateEmailView(FreeAccessView, ThemeView): # as we currently need to do a Resolve on each user_hrn of the Registry in order to get its email # TODO in SFA XXX We need a Resolve based on email # TODO maybe we can use MyPLC API for PLE - pending_users = PendingUser.objects.filter(email_hash__iexact = hash_code) # by default user is not in PLE ple_user_enabled = False - if pending_users: - pending_user = pending_users[0] - pending_user_request = make_request_user(pending_user) - pending_user_email = pending_users[0].email - query = Query.get('myplcuser').filter_by('email', '==', pending_user_email).select('enabled') - results = execute_admin_query(self.request, query) - for result in results: - # User is enabled in PLE - if 'enabled' in result and result['enabled']==True: - ple_user_enabled = True - break - + if pending_user: # Auto Validation - if ple_user_enabled: + if self.is_ple_enabled(pending_user): + pending_user_request = make_request_user(pending_user) # Create user in SFA and Update in Manifold create_user(self.request, pending_user_request, namespace = 'myslice', as_admin = True) # Delete pending user - #PendingUser.objects.filter(email_hash__iexact = hash_code).delete() + PendingUser.objects.filter(email_hash__iexact = hash_code).delete() # template user auto validated activation = 'validated' - - # sending email after activation success - #try: - # # Send an email: the recipient is the user - # recipients = pending_user_eamil - # theme.template_name = 'user_request_email.html' - # html_content = render_to_string(theme.template, request) - # theme.template_name = 'user_request_email.txt' - # text_content = render_to_string(theme.template, request) - # theme.template_name = 'user_request_email_subject.txt' - # subject = render_to_string(theme.template, request) - # subject = subject.replace('\n', '') - # theme.template_name = 'email_default_sender.txt' - # sender = render_to_string(theme.template, request) - # sender = sender.replace('\n', '') - # msg = EmailMultiAlternatives(subject, text_content, sender, recipients) - # msg.attach_alternative(html_content, "text/html") - # msg.send() - #except Exception, e: - # print "Failed to send email, please check the mail templates and the SMTP configuration of your server" - # import traceback - # traceback.print_exc() PendingUser.objects.filter(email_hash__iexact = hash_code).update(status='True') + u = {} + u['first_name'] = pending_user.first_name + u['last_name'] = pending_user.last_name + u['authority_hrn'] = pending_user.authority_hrn + u['email'] = pending_user.email + u['user_hrn'] = pending_user.user_hrn + u['pi'] = pending_user.pi + u['public_key'] = pending_user.public_key + u['current_site'] = current_site + + send_email_to_pis(self.request, u, 'user') else: activation = 'failed' - # get the domain url - current_site = Site.objects.get_current() - current_site = current_site.domain - context = super(ActivateEmailView, self).get_context_data(**kwargs) context['activation_status'] = activation @@ -118,10 +123,8 @@ class ActivateEmailView(FreeAccessView, ThemeView): #context['first_name'] = first_name #context['last_name'] = last_name #context['authority_hrn'] = authority_hrn - #context['public_key'] = public_key #context['email'] = email #context['user_hrn'] = user_hrn - #context['current_site'] = current_site context['theme'] = self.theme # context ['firstname'] = config['firstname'] prelude_env = page.prelude_env()