Changeset 3117 in products


Ignore:
Timestamp:
Apr 14, 2011 5:49:27 PM (13 years ago)
Author:
kroman0
Message:

Pyflakes and pep8 fixes #4

Location:
quintagroup.plonecomments/trunk/quintagroup/plonecomments
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • quintagroup.plonecomments/trunk/quintagroup/plonecomments/browser/comments.py

    r3116 r3117  
    2828        portal_properties = getToolByName(self.context, 'portal_properties') 
    2929        try: 
    30             return portal_properties.qPloneComments.getProperty('enable_moderation', None) 
     30            return portal_properties.qPloneComments.getProperty( 
     31                'enable_moderation', None) 
    3132        except AttributeError: 
    3233            return False 
    3334 
    3435    def can_moderate(self): 
    35         """ Returns a boolean indicating whether the user has the 'Moderate Discussion' 
    36             permission 
     36        """ Returns a boolean indicating whether the user has 
     37            the 'Moderate Discussion' permission 
    3738        """ 
    38         return getSecurityManager().checkPermission('Moderate Discussion', aq_inner(self.context)) 
     39        return getSecurityManager().checkPermission('Moderate Discussion', 
     40                                                    aq_inner(self.context)) 
    3941 
    4042    def getGravatar(self, reply): 
    … …  
    4547 
    4648        creator = reply.Creator() 
    47         if creator and not creator=='Anonymous User': 
     49        if creator and not creator == 'Anonymous User': 
    4850            mtool = getToolByName(self.context, "portal_membership") 
    4951            member = mtool.getMemberById(creator) 
    50             email = member and member.getProperty('email','') or '' 
    51             mem_id = getattr(member,'getId',lambda:'Anonymous User')() 
     52            email = member and member.getProperty('email', '') or '' 
     53            mem_id = getattr(member, 'getId', lambda: 'Anonymous User')() 
    5254            portrait = mtool.getPersonalPortrait(mem_id) 
    5355            portrait_url = portrait.absolute_url() 
    5456        else: 
    55             email = reply.getProperty('email',d='') 
     57            email = reply.getProperty('email', d='') 
    5658 
    5759        if not email or not 'defaultUser.gif' in portrait_url: 
    … …  
    6163        gravatar_url = "http://www.gravatar.com/avatar.php?" 
    6264        # construct the url 
    63         gravatar_url += urllib.urlencode({'gravatar_id':md5.md5(email).hexdigest(), 
    64             'default':portrait_url, 'size':str(size)}) 
     65        gravatar_url += urllib.urlencode({ 
     66            'gravatar_id': md5.md5(email).hexdigest(), 
     67            'default': portrait_url, 
     68            'size': str(size)}) 
    6569        return gravatar_url 
    6670 
    … …  
    6973        portal_properties = getToolByName(self.context, 'portal_properties') 
    7074        prop_sheet = portal_properties['qPloneComments'] 
    71         value = prop_sheet.getProperty('enable_authenticated_report_abuse', False) 
     75        value = prop_sheet.getProperty('enable_authenticated_report_abuse', 
     76                                       False) 
    7277        return value 
    7378 
    … …  
    107112 
    108113    def submit_abuse_report(self): 
    109         """ Send an email with the abuse report message and hide abuse report form. 
     114        """ Send an email with the abuse report message and 
     115            hide abuse report form. 
    110116        """ 
    111117        errors = {} 
    … …  
    123129            # get the form controller 
    124130            controller = portal.portal_form_controller 
    125             # send the validate script to the form controller with the dummy state object 
    126             controller_state = controller.validate(dummy_controller_state, request, ['captcha_validator',]) 
     131            # send the validate script to the form controller 
     132            # with the dummy state object 
     133            controller_state = controller.validate(dummy_controller_state, 
     134                                   request, ['captcha_validator', ]) 
    127135            errors.update(controller_state.errors) 
    128136 
    … …  
    141149                                     member=member, 
    142150                                     **request.form) 
    143             node = ksscore.getHtmlIdSelector('span-reply-form-holder-%s' % comment_id) 
     151            node = ksscore.getHtmlIdSelector('span-reply-form-holder-%s' % \ 
     152                                             comment_id) 
    144153            ksscore.replaceInnerHTML(node,  html) 
    145154            return self.render() 
    … …  
    151160                                 member=member, 
    152161                                 **request.form) 
    153         node = ksscore.getHtmlIdSelector('span-reply-form-holder-%s' % comment_id) 
    154         html = '<br/><span style="color:red">You have reported this comment for abuse.</span>' 
     162        node = ksscore.getHtmlIdSelector('span-reply-form-holder-%s' % \ 
     163                                         comment_id) 
     164        html = '<br/><span style="color:red">' \ 
     165            'You have reported this comment for abuse.</span>' 
    155166        self.commands.addCommand('remove_abuse_report_form',  
    156167                                 node,  

  • quintagroup.plonecomments/trunk/quintagroup/plonecomments/patch.py

    r3116 r3117  
    5959    return id 
    6060 
     61 
    6162def getReplies(self): 
    6263    """Return a sequence of the DiscussionResponse objects which are 
    … …  
    7677    return objects 
    7778 
     79 
    7880perms = DiscussionItemContainer.__ac_permissions__ 
    7981new_perms = [] 

  • quintagroup.plonecomments/trunk/quintagroup/plonecomments/skins/quintagroup_plonecomments/deleteDiscussion.py

    r822 r3117  
    2222 
    2323# remove the discussion item 
    24 talkback.deleteReply( obj.getId() ) 
     24talkback.deleteReply(obj.getId()) 
    2525manage_mails(obj, context, 'deleting') 
    2626 
    … …  
    3131context.plone_utils.addPortalMessage(PloneMessageFactory(u'Reply deleted.')) 
    3232 
    33 context.REQUEST['RESPONSE'].redirect( redirect_target.absolute_url() + '/%s' % view ) 
     33context.REQUEST['RESPONSE'].redirect(redirect_target.absolute_url() + \ 
     34                                     '/%s' % view ) 

  • quintagroup.plonecomments/trunk/quintagroup/plonecomments/tests/testQPloneCommentsConfiglet.py

    r3116 r3117  
    1616    self.membership = getToolByName(self.portal, 'portal_membership', None) 
    1717    for user_id in USERS.keys(): 
    18         self.membership.addMember(user_id, USERS[user_id]['passw'] , USERS[user_id]['roles'], []) 
     18        self.membership.addMember(user_id, USERS[user_id]['passw'], USERS[user_id]['roles'], []) 
    1919 
    2020    # Add users to Discussion Manager group 

  • quintagroup.plonecomments/trunk/quintagroup/plonecomments/tests/testQPloneCommentsReportAbuse.py

    r3114 r3117  
    2222        except: 
    2323            raise "Anonymous user CAN'T report abuse in turned ON *Anonymous report abuse mode*." 
    24  
    2524 
    2625    def testAuthenticatedReportAbuse(self): 

  • quintagroup.plonecomments/trunk/quintagroup/plonecomments/utils.py

    r3116 r3117  
    4242 
    4343    prop_sheet = context.portal_properties['qPloneComments'] 
    44     props = filter(lambda x: prop_sheet.getProperty(x), prop_sheet.propertyIds()) 
     44    props = filter(lambda x: prop_sheet.getProperty(x), \ 
     45                   prop_sheet.propertyIds()) 
    4546 
    4647    actions = { 
    … …  
    8687                [obj.Creator(), ] 
    8788            userid = creators and creators[0] or "" 
    88             creator = getToolByName(context, 'portal_membership').getMemberById(userid) 
     89            portal_membership = getToolByName(context, 'portal_membership') 
     90            creator = portal_membership.getMemberById(userid) 
    8991            if creator and allowEmail(context, reply, state, creator): 
    9092                return creator.getProperty('email', '') 
    … …  
    105107    def getParentOwnerEmail(reply, context): 
    106108        creator_id = getParent(reply).getOwnerTuple()[1] 
    107         creator = getToolByName(context, 'portal_membership').getMemberById(creator_id) 
     109        portal_membership = getToolByName(context, 'portal_membership') 
     110        creator = portal_membership.getMemberById(creator_id) 
    108111        if creator and allowEmail(context, reply, state, creator): 
    109112            return creator.getProperty('email', '') 
    … …  
    117120    organization_name = getProp(context, 'email_subject_prefix', '') 
    118121    creator_name = reply.getOwnerTuple()[1] 
    119     admin_email = context.portal_url.getPortalObject().getProperty('email_from_address') 
     122    portal = getToolByName(context, 'portal_url').getPortalObject() 
     123    admin_email = portal.getProperty('email_from_address') 
    120124    translate = getToolByName(context, 'translation_service').translate 
    121125    subject = '' 
    … …  
    123127        subject = translate(_(u"approve_user_notification_subject", 
    124128            default=u"Your comment on ${title} is now published", 
    125             mapping={u"title": safe_unicode(getParent(context).title_or_id())}), 
     129            mapping={u"title": 
     130                safe_unicode(getParent(context).title_or_id())}), 
    126131            context=context.REQUEST) 
    127132        if user_email: 
    … …  
    138143        subject = translate(_(u"rejected_user_notification_subject", 
    139144            default=u"Your comment on ${title} was not approved", 
    140             mapping={u"title": safe_unicode(getParent(context).title_or_id())}), 
     145            mapping={u"title": 
     146                safe_unicode(getParent(context).title_or_id())}), 
    141147            context=context.REQUEST) 
    142148        if user_email: 
    … …  
    154160        subject = translate(_(u"reply_user_notification_subject", 
    155161            default=u"Someone replied to your comment on ${title}", 
    156             mapping={u"title": safe_unicode(getParent(context).title_or_id())}), 
     162            mapping={u"title": 
     163                safe_unicode(getParent(context).title_or_id())}), 
    157164            context=context.REQUEST) 
    158165        di_parrent = getDIParent(reply) 
    … …  
    200207            args = {} 
    201208 
    202     elif state in ('enable_authenticated_report_abuse', 'enable_anonymous_report_abuse'): 
     209    elif state in ('enable_authenticated_report_abuse', 
     210                   'enable_anonymous_report_abuse'): 
    203211        template = 'report_abuse_template' 
    204212        user_email = getProp(context, "email_discussion_manager", None) 
    … …  
    230238    if args: 
    231239        msg = getMsg(context, template, args) 
    232         charset = context.portal_properties.site_properties.getProperty('default_charset', 'utf-8') 
     240        site_properties = context.portal_properties.site_properties 
     241        charset = site_properties.getProperty('default_charset', 'utf-8') 
    233242        msg = msg.encode(charset) 
    234243        host = context.plone_utils.getMailHost() 
    … …  
    236245            host.secureSend(msg, user_email, admin_email, subject=subject, 
    237246                            subtype='plain', debug=False, charset=charset) 
    238         except (smtplib.SMTPRecipientsRefused,smtplib.SMTPServerDisconnected): 
     247        except (smtplib.SMTPRecipientsRefused, smtplib.SMTPServerDisconnected): 
    239248            setStatusMsg(None, context, 
    240249                _('Could not send the email notification. Have you configured an email server for Plone?')) 
Note: See TracChangeset for help on using the changeset viewer.