Changeset 2541

Show
Ignore:
Timestamp:
08/01/07 10:28:41 (1 year ago)
Author:
Blackhex
Message:

DiscussionPlugin:

Bug #1875 fix.

  • Fixed preview of edited messages.
Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • discussionplugin/0.10/tracdiscussion/admin.py

    r1482 r2541  
    3434        req.args['component'] = 'admin' 
    3535 
    36         # Get database access 
    37         db = self.env.get_db_cnx() 
    38         cursor = db.cursor() 
    39  
    4036        # Retrun page content 
    4137        api = DiscussionApi(self, req) 
    42         content = api.render_discussion(req, cursor) 
    43         db.commit() 
    44         return content 
     38        return api.render_discussion(req) 

  • discussionplugin/0.10/tracdiscussion/api.py

    r2527 r2541  
    1616    # Main request processing function 
    1717 
    18     def render_discussion(self, req, cursor): 
     18    def render_discussion(self, req): 
    1919        # Get request mode 
    20         group, forum, topic, message = self._get_items(req, cursor) 
     20        group, forum, topic, message = self._get_items(req) 
    2121        modes = self._get_modes(req, group, forum, topic, message) 
    2222        self.log.debug('modes: %s' % modes) 
    …… 
    3030 
    3131        # Perform mode actions 
    32         self._do_action(req, cursor, modes, group, forum, topic, message, 
    33           is_moderator) 
     32        self._do_action(req, modes, group, forum, topic, message, is_moderator) 
    3433 
    3534        # Add CSS styles 
    …… 
    6867        return modes[-1] + '.cs', None 
    6968 
    70     def _get_items(self, req, cursor): 
     69    def _get_items(self, req): 
    7170        group, forum, topic, message = None, None, None, None 
     71 
     72        # Get database access. 
     73        db = self.env.get_db_cnx() 
     74        cursor = db.cursor() 
    7275 
    7376        # Populate active group 
    …… 
    273276                    return ['forum-list'] 
    274277 
    275     def _do_action(self, req, cursor, modes, group, forum, topic, message, 
     278    def _do_action(self, req, modes, group, forum, topic, message, 
    276279      is_moderator): 
     280 
     281        # Get database access. 
     282        db = self.env.get_db_cnx() 
     283        cursor = db.cursor() 
     284 
    277285        for mode in modes: 
    278286            self.log.debug('doing %s mode action' % (mode,)) 
    …… 
    313321 
    314322                # Redirect request to prevent re-submit. 
     323                db.commit() 
    315324                req.redirect(req.href.discussion('redirect', href = 
    316325                  req.path_info)) 
    …… 
    328337 
    329338                # Redirect request to prevent re-submit. 
     339                db.commit() 
    330340                req.redirect(req.href.discussion('redirect', href = 
    331341                  req.path_info)) 
    …… 
    348358 
    349359                # Redirect request to prevent re-submit. 
     360                db.commit() 
    350361                req.redirect(req.href.discussion('redirect', href = 
    351362                  req.path_info)) 
    …… 
    415426 
    416427                # Redirect request to prevent re-submit. 
     428                db.commit() 
    417429                req.redirect(req.href.discussion('redirect', href = 
    418430                  req.path_info)) 
    …… 
    438450 
    439451                # Redirect request to prevent re-submit. 
     452                db.commit() 
    440453                req.redirect(req.href.discussion('redirect', href = 
    441454                  req.path_info)) 
    …… 
    448461 
    449462                # Redirect request to prevent re-submit. 
     463                db.commit() 
    450464                req.redirect(req.href.discussion('redirect', href = 
    451465                  req.path_info)) 
    …… 
    465479 
    466480                # Redirect request to prevent re-submit. 
     481                db.commit() 
    467482                req.redirect(req.href.discussion('redirect', href = 
    468483                  req.path_info)) 
    …… 
    533548 
    534549                # Redirect request to prevent re-submit. 
     550                db.commit() 
    535551                req.redirect(req.href.discussion('redirect', href = 
    536552                  req.path_info)) 
    …… 
    563579 
    564580                # Redirect request to prevent re-submit. 
     581                db.commit() 
    565582                req.redirect(req.href.discussion('redirect', href = 
    566583                  req.path_info)) 
    …… 
    586603 
    587604                # Redirect request to prevent re-submit. 
     605                db.commit() 
    588606                req.redirect(req.href.discussion('redirect', href = 
    589607                  req.path_info)) 
    …… 
    598616 
    599617                # Redirect request to prevent re-submit. 
     618                db.commit() 
    600619                req.redirect(req.href.discussion('redirect', href = 
    601620                  req.path_info)) 
    …… 
    643662 
    644663                # Redirect request to prevent re-submit. 
    645                 self.log.debug(req.href(req.path_info) + '/') 
    646664                if req.args.get('component') != 'wiki': 
     665                    db.commit() 
    647666                    req.redirect(req.href.discussion('redirect', href = 
    648667                      req.path_info)) 
    …… 
    672691                # Redirect request to prevent re-submit. 
    673692                if req.args.get('component') != 'wiki': 
     693                    db.commit() 
    674694                    req.redirect(req.href.discussion('redirect', href = 
    675695                      req.path_info)) 
    …… 
    685705                # Redirect request to prevent re-submit. 
    686706                if req.args.get('component') != 'wiki': 
     707                    db.commit() 
    687708                    req.redirect(req.href.discussion('redirect', href = 
    688709                      req.path_info)) 
    …… 
    696717                # Set message list display mode to session 
    697718                req.session['message-list-display'] = display 
     719 
     720        # Commit database changes. 
     721        db.commit() 
    698722 
    699723    def _prepare_message_list(self, req, cursor, topic): 

  • discussionplugin/0.10/tracdiscussion/core.py

    r2527 r2541  
    4848        if req.path_info == '/discussion/redirect': 
    4949            # Proces redirection request. 
     50            self.log.debug(req.path_info) 
     51            self.log.debug(req.args.get('href')) 
    5052            req.redirect(req.href(req.args.get('href'))) 
    5153        else: 
    …… 
    6971        req.args['component'] = 'core' 
    7072 
    71         # Get database access 
    72         db = self.env.get_db_cnx() 
    73         cursor = db.cursor() 
    74  
    7573        # Return page content 
    7674        api = DiscussionApi(self, req) 
    77         content = api.render_discussion(req, cursor) 
    78         db.commit() 
    79         return content 
     75        return api.render_discussion(req) 

  • discussionplugin/0.10/tracdiscussion/templates/discussion-macros.cs

    r2236 r2541  
    107107      </div> 
    108108      <div class="body"> 
    109         <?cs var:message.body ?> 
     109        <?cs if:(args.message == message.id) && !args.submit && ((args.discussion_action == 'edit') || (args.discussion_action == 'post-edit')) ?> 
     110          <?cs var:discussion.body ?> 
     111        <?cs else?> 
     112          <?cs var:message.body ?> 
     113        <?cs /if ?> 
    110114      </div> 
    111115      <div class="controls"> 

  • discussionplugin/0.10/tracdiscussion/wiki.py

    r2236 r2541  
    5151                subject = req.path_info[6:] or 'WikiStart' 
    5252 
    53             # Get database access 
     53            # Get database access. 
    5454            db = self.env.get_db_cnx() 
    5555            cursor = db.cursor() 
    …… 
    6666                req.args['forum'] = topic['forum'] 
    6767                req.args['topic'] = topic['id'] 
    68             content = api.render_discussion(req, cursor) 
    69             db.commit() 
    70             return req.hdf.render(content[0]) 
     68            return req.hdf.render(api.render_discussion(req)[0]) 
    7169        else: 
    7270            raise TracError('Not implemented macro %s' % (name)) 
    …… 
    9088        id = params 
    9189 
     90        # Get database access. 
    9291        db = self.env.get_db_cnx() 
    9392        cursor = db.cursor() 
     93 
    9494        if ns == 'forum': 
    9595            columns = ('subject',)