Attaching a patch for Moin to move the wiki edit buttons under the comments, something some users have asked for. Of course this messes with the actual Moin code and will be re-written in case of an upgrade. Nevertheless, it might be worth it (besides, if we don't apply the patch after the upgrade, nothing breaks really). I've also created an upstream RFE: http://moinmoin.wikiwikiweb.de/FeatureRequests/MoveButtonBarBelowCommentBox -d -- Dimitris Glezos Jabber ID: glezos@xxxxxxxxxx, GPG: 0xA5A04C3B http://dimitris.glezos.com/ "He who gives up functionality for ease of use loses both and deserves neither." (Anonymous) --
--- /home/glezos/Desktop/PageEditor.py 2007-03-25 22:50:12.000000000 +0100 +++ /usr/lib/python2.4/site-packages/MoinMoin/PageEditor.py 2007-03-25 23:14:01.000000000 +0100 @@ -306,53 +306,20 @@ self.request.write('<input type="hidden" name="rev" value="%d">' % (rev,)) # Create and send a ticket, so we can check the POST self.request.write('<input type="hidden" name="ticket" value="%s">' % wikiutil.createTicket(self.request)) # Save backto in a hidden input backto = form.get('backto', [None])[0] if backto: self.request.write(unicode(html.INPUT(type="hidden", name="backto", value=backto))) - # button bar - button_spellcheck = (SpellCheck and - '<input class="button" type="submit" name="button_spellcheck" value="%s" onClick="flgChange = false;">' - % _('Check Spelling')) or '' - - save_button_text = _('Save Changes') - cancel_button_text = _('Cancel') - - if self.cfg.page_license_enabled: - self.request.write('<p><em>', _( -"""By hitting '''%(save_button_text)s''' you put your changes under the %(license_link)s. -If you don't want that, hit '''%(cancel_button_text)s''' to cancel your changes.""") % { - 'save_button_text': save_button_text, - 'cancel_button_text': cancel_button_text, - 'license_link': wikiutil.getSysPage(self.request, self.cfg.page_license_page).link_to(self.request), - }, '</em></p>') - - - self.request.write(''' -<input class="button" type="submit" name="button_save" value="%s" onClick="flgChange = false;"> -<input class="button" type="submit" name="button_preview" value="%s" onClick="flgChange = false;"> -''' % (save_button_text, _('Preview'),)) - - if not (self.request.cfg.editor_force and self.request.cfg.editor_default == 'text'): - self.request.write(''' -<input id="switch2gui" style="display: none;" class="button" type="submit" name="button_switch" value="%s"> -''' % (_('GUI Mode'),)) - - self.request.write(''' -%s -<input class="button" type="submit" name="button_cancel" value="%s"> -<input type="hidden" name="editor" value="text"> -''' % (button_spellcheck, cancel_button_text,)) # Add textarea with page text # TODO: currently self.language is None at this point. We have # to do processing instructions parsing earlier, or move page # language into meta file. lang = self.language or self.request.cfg.language_default self.sendconfirmleaving() @@ -405,20 +372,56 @@ }) self.request.write("</p>") badwords_re = None if preview is not None: if SpellCheck and ( form.has_key('button_spellcheck') or form.has_key('button_newwords')): badwords, badwords_re, msg = SpellCheck.checkSpelling(self, self.request, own_form=0) self.request.write("<p>%s</p>" % msg) + + # button bar + button_spellcheck = (SpellCheck and + '<input class="button" type="submit" name="button_spellcheck" value="%s" onClick="flgChange = false;">' + % _('Check Spelling')) or '' + + save_button_text = _('Save Changes') + cancel_button_text = _('Cancel') + + if self.cfg.page_license_enabled: + self.request.write('<p><em>', _( +"""By hitting '''%(save_button_text)s''' you put your changes under the %(license_link)s. +If you don't want that, hit '''%(cancel_button_text)s''' to cancel your changes.""") % { + 'save_button_text': save_button_text, + 'cancel_button_text': cancel_button_text, + 'license_link': wikiutil.getSysPage(self.request, self.cfg.page_license_page).link_to(self.request), + }, '</em></p>') + + + self.request.write(''' +<input class="button" type="submit" name="button_save" value="%s" onClick="flgChange = false;"> +<input class="button" type="submit" name="button_preview" value="%s" onClick="flgChange = false;"> +''' % (save_button_text, _('Preview'),)) + + if not (self.request.cfg.editor_force and self.request.cfg.editor_default == 'text'): + self.request.write(''' +<input id="switch2gui" style="display: none;" class="button" type="submit" name="button_switch" value="%s"> +''' % (_('GUI Mode'),)) + + self.request.write(''' +%s +<input class="button" type="submit" name="button_cancel" value="%s"> +<input type="hidden" name="editor" value="text"> +''' % (button_spellcheck, cancel_button_text,)) + + self.request.write('</fieldset>') self.request.write("</form>") # QuickHelp originally by Georg Mischler <schorsch@xxxxxxxxxxxxxxxx> markup = self.pi_format or self.request.cfg.default_markup quickhelp = self.request.cfg.editor_quickhelp.get(markup, "") if quickhelp: self.request.write(self.request.formatter.div(1, id="editor-help")) self.request.write(_(quickhelp)) self.request.write(self.request.formatter.div(0))
# -*- coding: iso-8859-1 -*- """ MoinMoin - PageEditor class @copyright: 2000-2004 by Jürgen Hermann <jh@xxxxxx> @license: GNU GPL, see COPYING for details. """ import os, time, codecs, re, errno from MoinMoin import caching, config, user, util, wikiutil, error from MoinMoin.Page import Page from MoinMoin.widget import html from MoinMoin.widget.dialog import Status from MoinMoin.logfile import editlog, eventlog from MoinMoin.util import filesys, timefuncs import MoinMoin.util.web import MoinMoin.util.mail ############################################################################# ### Javascript code for editor page ############################################################################# # we avoid the "--" operator to make this XHTML happy! _countdown_js = """ %(countdown_script)s <script type="text/javascript"> var countdown_timeout_min = %(lock_timeout)s var countdown_lock_expire = "%(lock_expire)s" var countdown_lock_mins = "%(lock_mins)s" var countdown_lock_secs = "%(lock_secs)s" addLoadEvent(countdown) </script> """ ############################################################################# ### PageEditor - Edit pages ############################################################################# class PageEditor(Page): """Editor for a wiki page.""" # exceptions for .saveText() class SaveError(error.Error): pass class AccessDenied(SaveError): pass class Immutable(AccessDenied): pass class NoAdmin(AccessDenied): pass class EmptyPage(SaveError): pass class Unchanged(SaveError): pass class EditConflict(SaveError): pass class CouldNotLock(SaveError): pass def __init__(self, request, page_name, **keywords): """ Create page editor object. @param page_name: name of the page @param request: the request object @keyword do_revision_backup: if 0, suppress making a page backup per revision @keyword do_editor_backup: if 0, suppress making of HomePage/MoinEditorBackup per edit @keyword uid_override: override user id and name (default None) """ Page.__init__(self, request, page_name, **keywords) self._ = request.getText self.do_revision_backup = keywords.get('do_revision_backup', 1) self.do_editor_backup = keywords.get('do_editor_backup', 1) self.uid_override = keywords.get('uid_override', None) self.lock = PageLock(self) def mergeEditConflict(self, origrev): """ Try to merge current page version with new version the user tried to save @param origrev: the original revision the user was editing @rtype: bool @return: merge success status """ from MoinMoin.util import diff3 allow_conflicts = 1 # Get current editor text savetext = self.get_raw_body() # The original text from the revision the user was editing original_text = Page(self.request, self.page_name, rev=origrev).get_raw_body() # The current revision someone else saved saved_text = Page(self.request, self.page_name).get_raw_body() # And try to merge all into one with edit conflict separators verynewtext = diff3.text_merge(original_text, saved_text, savetext, allow_conflicts, "\n---- /!\ '''Edit conflict - other version:''' ----\n", "\n---- /!\ '''Edit conflict - your version:''' ----\n", "\n---- /!\ '''End of edit conflict''' ----\n") if verynewtext: self.set_raw_body(verynewtext) return True return False def sendconfirmleaving(self): """ to prevent moving away from the page without saving it """ _ = self._ self.request.write(u'''\ <script type="text/javascript"> var flgChange = false; function confirmleaving() { if ( flgChange ) return "%s"; } </script> ''' % _("Your changes are not saved!")) def sendEditor(self, **kw): """ Send the editor form page. @keyword preview: if given, show this text in preview mode @keyword staytop: don't go to #preview @keyword comment: comment field (when preview is true) """ from MoinMoin import i18n try: from MoinMoin.action import SpellCheck except ImportError: SpellCheck = None form = self.request.form _ = self._ self.request.disableHttpCaching(level=2) self.request.http_headers() raw_body = '' msg = None conflict_msg = None edit_lock_message = None preview = kw.get('preview', None) staytop = kw.get('staytop', 0) from MoinMoin.formatter.text_html import Formatter self.request.formatter = Formatter(self.request, store_pagelinks=1) # check edit permissions if not self.request.user.may.write(self.page_name): msg = _('You are not allowed to edit this page.') elif not self.isWritable(): msg = _('Page is immutable!') elif self.rev: # Trying to edit an old version, this is not possible via # the web interface, but catch it just in case... msg = _('Cannot edit old revisions!') else: try: # try to acquire edit lock ok, edit_lock_message = self.lock.acquire() if not ok: # failed to get the lock if preview is not None: edit_lock_message = _('The lock you held timed out. Be prepared for editing conflicts!' ) + "<br>" + edit_lock_message else: msg = edit_lock_message except OSError, err: if err.errno != errno.ENAMETOOLONG: raise err msg = _("Page name is too long, try shorter name.") # Did one of the prechecks fail? if msg: self.send_page(self.request, msg=msg) return # Check for preview submit if preview is None: title = _('Edit "%(pagename)s"') else: title = _('Preview of "%(pagename)s"') self.set_raw_body(preview, modified=1) # send header stuff lock_timeout = self.lock.timeout / 60 lock_page = wikiutil.escape(self.page_name, quote=1) lock_expire = _("Your edit lock on %(lock_page)s has expired!") % {'lock_page': lock_page} lock_mins = _("Your edit lock on %(lock_page)s will expire in # minutes.") % {'lock_page': lock_page} lock_secs = _("Your edit lock on %(lock_page)s will expire in # seconds.") % {'lock_page': lock_page} # get request parameters try: text_rows = int(form['rows'][0]) except StandardError: text_rows = self.cfg.edit_rows if self.request.user.valid: text_rows = int(self.request.user.edit_rows) if preview is not None: # Propagate original revision rev = int(form['rev'][0]) # Check for editing conflicts if not self.exists(): # page does not exist, are we creating it? if rev: conflict_msg = _('Someone else deleted this page while you were editing!') elif rev != self.current_rev(): conflict_msg = _('Someone else changed this page while you were editing!') if self.mergeEditConflict(rev): conflict_msg = _("""Someone else saved this page while you were editing! Please review the page and save then. Do not save this page as it is! Have a look at the diff of %(difflink)s to see what has been changed.""") % { 'difflink': self.link_to(self.request, querystr='action=diff&rev=%d' % rev) } rev = self.current_rev() if conflict_msg: # We don't show preview when in conflict preview = None elif self.exists(): # revision of existing page rev = self.current_rev() else: # page creation rev = 0 # Page editing is done using user language self.request.setContentLanguage(self.request.lang) # Setup status message status = [kw.get('msg', ''), conflict_msg, edit_lock_message] status = [msg for msg in status if msg] status = ' '.join(status) status = Status(self.request, content=status) wikiutil.send_title(self.request, title % {'pagename': self.split_title(self.request),}, page=self, pagename=self.page_name, msg=status, html_head=self.lock.locktype and ( _countdown_js % { 'countdown_script': self.request.theme.externalScript('countdown'), 'lock_timeout': lock_timeout, 'lock_expire': lock_expire, 'lock_mins': lock_mins, 'lock_secs': lock_secs, }) or '', editor_mode=1, ) self.request.write(self.request.formatter.startContent("content")) # Get the text body for the editor field. # TODO: what about deleted pages? show the text of the last revision or use the template? if preview is not None: raw_body = self.get_raw_body() elif self.exists(): # If the page exists, we get the text from the page. # TODO: maybe warn if template argument was ignored because the page exists? raw_body = self.get_raw_body() elif form.has_key('template'): # If the page does not exists, we try to get the content from the template parameter. template_page = wikiutil.unquoteWikiname(form['template'][0]) if self.request.user.may.read(template_page): raw_body = Page(self.request, template_page).get_raw_body() if raw_body: self.request.write(_("[Content of new page loaded from %s]") % (template_page,), '<br>') else: self.request.write(_("[Template %s not found]") % (template_page,), '<br>') else: self.request.write(_("[You may not read %s]") % (template_page,), '<br>') # Make backup on previews - but not for new empty pages if preview and raw_body: self._make_backup(raw_body) # Generate default content for new pages if not raw_body: raw_body = _('Describe %s here.') % (self.page_name,) # send form self.request.write('<form id="editor" method="post" action="%s/%s#preview" onSubmit="flgChange = false;">' % ( self.request.getScriptname(), wikiutil.quoteWikinameURL(self.page_name), )) # yet another weird workaround for broken IE6 (it expands the text # editor area to the right after you begin to type...). IE sucks... # http://fplanque.net/2003/Articles/iecsstextarea/ self.request.write('<fieldset style="border:none;padding:0;">') self.request.write(unicode(html.INPUT(type="hidden", name="action", value="edit"))) # Send revision of the page our edit is based on self.request.write('<input type="hidden" name="rev" value="%d">' % (rev,)) # Create and send a ticket, so we can check the POST self.request.write('<input type="hidden" name="ticket" value="%s">' % wikiutil.createTicket(self.request)) # Save backto in a hidden input backto = form.get('backto', [None])[0] if backto: self.request.write(unicode(html.INPUT(type="hidden", name="backto", value=backto))) # Add textarea with page text # TODO: currently self.language is None at this point. We have # to do processing instructions parsing earlier, or move page # language into meta file. lang = self.language or self.request.cfg.language_default self.sendconfirmleaving() self.request.write( u'''\ <textarea id="editor-textarea" name="savetext" lang="%(lang)s" dir="%(dir)s" rows="%(rows)d" onChange="flgChange = true;" onKeyPress="flgChange = true;">\ %(text)s\ </textarea>''' % { 'lang': lang, 'dir': i18n.getDirection(lang), 'rows': text_rows, 'text': wikiutil.escape(raw_body) }) self.request.write("<p>") self.request.write(_("Comment:"), ' <input id="editor-comment" type="text" name="comment" value="%s" maxlength="200"' ' onChange="flgChange = true;" onKeyPress="flgChange = true;">' % ( wikiutil.escape(kw.get('comment', ''), 1), )) self.request.write("</p>") # Category selection filter = re.compile(self.cfg.page_category_regex, re.UNICODE).search cat_pages = self.request.rootpage.getPageList(filter=filter) cat_pages.sort() cat_pages = [wikiutil.pagelinkmarkup(p) for p in cat_pages] cat_pages.insert(0, ('', _('<No addition>', formatted=False))) self.request.write("<p>") self.request.write(_('Add to: %(category)s') % { 'category': unicode(util.web.makeSelection('category', cat_pages)), }) if self.cfg.mail_enabled: self.request.write(''' <input type="checkbox" name="trivial" id="chktrivial" value="1" %(checked)s> <label for="chktrivial">%(label)s</label> ''' % { 'checked': ('', 'checked')[form.get('trivial',['0'])[0] == '1'], 'label': _("Trivial change"), }) self.request.write(''' <input type="checkbox" name="rstrip" id="chkrstrip" value="1" %(checked)s> <label for="chkrstrip">%(label)s</label> ''' % { 'checked': ('', 'checked')[form.get('rstrip',['0'])[0] == '1'], 'label': _('Remove trailing whitespace from each line') }) self.request.write("</p>") badwords_re = None if preview is not None: if SpellCheck and ( form.has_key('button_spellcheck') or form.has_key('button_newwords')): badwords, badwords_re, msg = SpellCheck.checkSpelling(self, self.request, own_form=0) self.request.write("<p>%s</p>" % msg) # button bar button_spellcheck = (SpellCheck and '<input class="button" type="submit" name="button_spellcheck" value="%s" onClick="flgChange = false;">' % _('Check Spelling')) or '' save_button_text = _('Save Changes') cancel_button_text = _('Cancel') if self.cfg.page_license_enabled: self.request.write('<p><em>', _( """By hitting '''%(save_button_text)s''' you put your changes under the %(license_link)s. If you don't want that, hit '''%(cancel_button_text)s''' to cancel your changes.""") % { 'save_button_text': save_button_text, 'cancel_button_text': cancel_button_text, 'license_link': wikiutil.getSysPage(self.request, self.cfg.page_license_page).link_to(self.request), }, '</em></p>') self.request.write(''' <input class="button" type="submit" name="button_save" value="%s" onClick="flgChange = false;"> <input class="button" type="submit" name="button_preview" value="%s" onClick="flgChange = false;"> ''' % (save_button_text, _('Preview'),)) if not (self.request.cfg.editor_force and self.request.cfg.editor_default == 'text'): self.request.write(''' <input id="switch2gui" style="display: none;" class="button" type="submit" name="button_switch" value="%s"> ''' % (_('GUI Mode'),)) self.request.write(''' %s <input class="button" type="submit" name="button_cancel" value="%s"> <input type="hidden" name="editor" value="text"> ''' % (button_spellcheck, cancel_button_text,)) self.request.write('</fieldset>') self.request.write("</form>") # QuickHelp originally by Georg Mischler <schorsch@xxxxxxxxxxxxxxxx> markup = self.pi_format or self.request.cfg.default_markup quickhelp = self.request.cfg.editor_quickhelp.get(markup, "") if quickhelp: self.request.write(self.request.formatter.div(1, id="editor-help")) self.request.write(_(quickhelp)) self.request.write(self.request.formatter.div(0)) if preview is not None: if staytop: content_id = 'previewbelow' else: content_id = 'preview' self.send_page(self.request, content_id=content_id, content_only=1, hilite_re=badwords_re) self.request.write(self.request.formatter.endContent()) wikiutil.send_footer(self.request, self.page_name) def sendCancel(self, newtext, rev): """ User clicked on Cancel button. If edit locking is active, delete the current lock file. @param newtext: the edited text (which has been cancelled) @param rev: not used!? """ _ = self._ self._make_backup(newtext) self.lock.release() backto = self.request.form.get('backto', [None])[0] page = backto and Page(self.request, backto) or self page.send_page(self.request, msg=_('Edit was cancelled.')) def renamePage(self, newpagename, comment=None): """ Rename the current version of the page (making a backup before deletion and keeping the backups, logs and attachments). @param comment: Comment given by user @rtype: unicode @return: success flag, error message """ _ = self._ if not newpagename: return False, _("You can't rename to an empty pagename.") newpage = PageEditor(self.request, newpagename) pageexists_error = _("""'''A page with the name {{{'%s'}}} already exists.''' Try a different name.""") % (newpagename,) # Check whether a page with the new name already exists if newpage.exists(includeDeleted=1): return False, pageexists_error # Get old page text savetext = self.get_raw_body() oldpath = self.getPagePath(check_create=0) newpath = newpage.getPagePath(check_create=0) # Rename page # NOTE: might fail if another process created newpagename just # NOW, while you read this comment. Rename is atomic for files - # but for directories, rename will fail if the directory # exists. We should have global edit-lock to avoid this. # See http://docs.python.org/lib/os-file-dir.html try: os.rename(oldpath, newpath) self.error = None # Save page text with a comment about the old name savetext = u"## page was renamed from %s\n%s" % (self.page_name, savetext) newpage.saveText(savetext, 0, comment=comment) return True, None except OSError, err: # Try to understand what happened. Maybe its better to check # the error code, but I just reused the available code above... if newpage.exists(includeDeleted=1): return False, pageexists_error else: return False, _('Could not rename page because of file system error: %s.') % unicode(err) def deletePage(self, comment=None): """ Delete the current version of the page (making a backup before deletion and keeping the backups, logs and attachments). @param comment: Comment given by user @rtype: unicode @return: success flag, error message """ _ = self._ success = True try: # First save a final backup copy of the current page # (recreating the page allows access to the backups again) msg = self.saveText(u"deleted\n", 0, comment=comment or u'') msg = msg.replace( _("Thank you for your changes. Your attention to detail is appreciated."), _('Page "%s" was successfully deleted!') % (self.page_name,)) # Then really delete it try: os.remove(self._text_filename()) except OSError, err: if err.errno != errno.ENOENT: raise err except self.SaveError, message: # XXX Error handling success = False msg = "SaveError has occured in PageEditor.deletePage. We need locking there." # reset page object self.reset() # delete pagelinks arena = self key = 'pagelinks' cache = caching.CacheEntry(self.request, arena, key) cache.remove() # forget in-memory page text self.set_raw_body(None) # clean the in memory acl cache self.clean_acl_cache() # clean the cache for formatter_name in self.cfg.caching_formats: arena = self key = formatter_name cache = caching.CacheEntry(self.request, arena, key) cache.remove() return success, msg def _sendNotification(self, comment, emails, email_lang, revisions, trivial): """ Send notification email for a single language. @param comment: editor's comment given when saving the page @param emails: list of email addresses @param email_lang: language of emails @param revisions: revisions of this page @param trivial: the change is marked as trivial @rtype: int @return: sendmail result """ _ = lambda s, formatted=True, r=self.request, l=email_lang: r.getText(s, formatted=formatted, lang=l) pagelink = self.request.getQualifiedURL(self.url(self.request)) if len(revisions) >= 2: pagelink += "?action=diff&rev2=%i&rev1=%i" % tuple(revisions[:2]) mailBody = _("Dear Wiki user,\n\n" 'You have subscribed to a wiki page or wiki category on "%(sitename)s" for change notification.\n\n' "The following page has been changed by %(editor)s:\n" "%(pagelink)s\n\n", formatted=False) % { 'editor': self.uid_override or user.getUserIdentification(self.request), 'pagelink': pagelink, 'sitename': self.cfg.sitename or self.request.getBaseURL(), } if comment: mailBody = mailBody + \ _("The comment on the change is:\n%(comment)s\n\n", formatted=False) % {'comment': comment} # append a diff (or append full page text if there is no diff) if len(revisions) < 2: mailBody = mailBody + \ _("New page:\n", formatted=False) + \ self.get_raw_body() else: lines = wikiutil.pagediff(self.request, self.page_name, revisions[1], self.page_name, revisions[0]) if lines: mailBody = mailBody + "%s\n%s\n" % (("-" * 78), '\n'.join(lines)) else: mailBody = mailBody + _("No differences found!\n", formatted=False) return util.mail.sendmail(self.request, emails, _('[%(sitename)s] %(trivial)sUpdate of "%(pagename)s" by %(username)s', formatted=False) % { 'trivial' : (trivial and _("Trivial ", formatted=False)) or "", 'sitename': self.cfg.sitename or "Wiki", 'pagename': self.page_name, 'username': self.uid_override or user.getUserIdentification(self.request), }, mailBody, mail_from=self.cfg.mail_from) def _notifySubscribers(self, comment, trivial): """ Send email to all subscribers of this page. @param comment: editor's comment given when saving the page @param trivial: editor's suggestion that the change is trivial (Subscribers may ignore this) @rtype: string @return: message, indicating success or errors. """ _ = self._ subscribers = self.getSubscribers(self.request, return_users=1, trivial=trivial) if subscribers: # get a list of old revisions, and append a diff revisions = self.getRevList() # send email to all subscribers results = [_('Status of sending notification mails:')] for lang in subscribers.keys(): emails = map(lambda u: u.email, subscribers[lang]) names = map(lambda u: u.name, subscribers[lang]) mailok, status = self._sendNotification(comment, emails, lang, revisions, trivial) recipients = ", ".join(names) results.append(_('[%(lang)s] %(recipients)s: %(status)s') % { 'lang': lang, 'recipients': recipients, 'status': status}) # Return mail sent results. Ignore trivial - we don't have # to lie. If mail was sent, just tell about it. return '<p>\n%s\n</p> ' % '<br>'.join(results) # No mail sent, no message. return '' def _expand_variables(self, text): """ Expand @VARIABLE@ in `text`and return the expanded text. @param text: current text of wikipage @rtype: string @return: new text of wikipage, variables replaced """ # TODO: Allow addition of variables via wikiconfig or a global # wiki dict. now = time.strftime("%Y-%m-%dT%H:%M:%SZ", timefuncs.tmtuple()) user = self.request.user signature = user.signature() variables = { 'PAGE': self.page_name, 'TIME': "[[DateTime(%s)]]" % now, 'DATE': "[[Date(%s)]]" % now, 'ME': user.name, 'USERNAME': signature, 'USER': "-- %s" % signature, 'SIG': "-- %s [[DateTime(%s)]]" % (signature, now), } if user.valid and user.name: if user.email: variables['MAILTO'] = "[[MailTo(%s)]]" % user.email # Users can define their own variables via # UserHomepage/MyDict, which override the default variables. userDictPage = user.name + "/MyDict" if self.request.dicts.has_dict(userDictPage): variables.update(self.request.dicts.dict(userDictPage)) # TODO: Use a more stream-lined re.sub algorithm for name in variables: text = text.replace('@%s@' % name, variables[name]) return text def normalizeText(self, text, **kw): """ Normalize text Make sure text uses '\n' line endings, and has a trailing newline. Strip whitespace on end of lines if needed. You should normalize any text you enter into a page, for example, when getting new text from the editor, or when setting new text manually. @param text: text to normalize (unicode) @keyword stripspaces: if 1, strip spaces from text @rtype: unicode @return: normalized text """ if text: lines = text.splitlines() # Strip trailing spaces if needed if kw.get('stripspaces', 0): lines = [line.rstrip() for line in lines] # Add final newline if not present, better for diffs (does # not include former last line when just adding text to # bottom; idea by CliffordAdams) if not lines[-1] == u'': # '' will make newline after join lines.append(u'') text = u'\n'.join(lines) return text def _make_backup(self, newtext, **kw): """ Make editor backup on user homepage Replace text of the user UserName/MoinEditorBackup with newtext. @param newtext: new text of the page @param kw: no keyword args used currently @rtype: unicode @return: url of page backup or None """ # Backup only if set to backup and user has a home page. homepage = wikiutil.getInterwikiHomePage(self.request) if homepage is None or not self.do_editor_backup: return None if homepage[0] != 'Self': # user has a remote homepage return None _ = self._ backuppage = PageEditor(self.request, homepage[1] + "/MoinEditorBackup", do_revision_backup=0) # We need All: at the end to prevent that the original page ACLs # make it possible to see preview saves (that maybe were never # really saved by the author). intro = u"#acl %s:read,write,delete All:\n" % self.request.user.name pagename = self.page_name date = self.request.user.getFormattedDateTime(time.time()) intro += _('## backup of page "%(pagename)s" submitted %(date)s') % { 'pagename': pagename, 'date': date,} + u'\n' backuppage._write_file(intro + newtext) return backuppage.url(self.request) def _get_pragmas(self, text): pragmas = {} for line in text.split('\n'): if not line or line[0] != '#': # end of pragmas break if len(line) > 1 and line[1] == '#': # a comment within pragmas continue verb, args = (line[1:]+' ').split(' ', 1) pragmas[verb.lower()] = args.strip() return pragmas def copypage(self): """ Copy a page from underlay directory to page directory """ src = self.getPagePath(use_underlay=1, check_create=0) dst = self.getPagePath(use_underlay=0, check_create=0) if src and dst and src != dst and os.path.exists(src): try: os.rmdir(dst) # simply remove empty dst dirs # XXX in fact, we should better remove anything we regard as an # empty page, maybe also if there is also an edit-lock or # empty cache. revisions subdir... except: pass if not os.path.exists(dst): filesys.copytree(src, dst) self.reset() # reinit stuff def _write_file(self, text, action='SAVE', comment=u'', extra=u''): """ Write the text to the page file (and make a backup of old page). @param text: text to save for this page @rtype: int @return: mtime_usec of new page """ _ = self._ #is_deprecated = self._get_pragmas(text).has_key("deprecated") was_deprecated = self._get_pragmas(self.get_raw_body()).has_key("deprecated") self.copypage() # Write always on the standard directory, never change the # underlay directory copy! pagedir = self.getPagePath(use_underlay=0, check_create=0) revdir = os.path.join(pagedir, 'revisions') cfn = os.path.join(pagedir,'current') clfn = os.path.join(pagedir,'current-locked') # !!! these log objects MUST be created outside the locked area !!! # The local log should be the standard edit log, not the # underlay copy log! pagelog = self.getPagePath('edit-log', use_underlay=0, isfile=1) llog = editlog.EditLog(self.request, filename=pagelog, uid_override=self.uid_override) # Open the global log glog = editlog.EditLog(self.request, uid_override=self.uid_override) if not os.path.exists(pagedir): # new page, create and init pagedir os.mkdir(pagedir) if not os.path.exists(revdir): os.mkdir(revdir) f = open(cfn, 'w') f.write('%08d\n' % 0) f.close() got_lock = False retry = 0 try: while not got_lock and retry < 100: retry += 1 try: filesys.rename(cfn, clfn) got_lock = True except OSError, err: got_lock = False if err.errno == 2: # there was no 'current' file time.sleep(0.1) else: raise self.CouldNotLock, _("Page could not get locked. Unexpected error (errno=%d).") % err.errno if not got_lock: raise self.CouldNotLock, _("Page could not get locked. Missing 'current' file?") # increment rev number of current(-locked) page f = open(clfn) revstr = f.read() f.close() rev = int(revstr) if not was_deprecated: if self.do_revision_backup or rev == 0: rev += 1 revstr = '%08d' % rev f = open(clfn, 'w') f.write(revstr+'\n') f.close() # save to page file pagefile = os.path.join(revdir, revstr) f = codecs.open(pagefile, 'wb', config.charset) # Write the file using text/* mime type f.write(self.encodeTextMimeType(text)) f.close() mtime_usecs = wikiutil.timestamp2version(os.path.getmtime(pagefile)) # set in-memory content self.set_raw_body(text) # reset page object self.reset() # write the editlog entry # for now simply make 2 logs, better would be some multilog stuff maybe if self.do_revision_backup: # do not globally log edits with no revision backup (like /MoinEditorBackup pages) # if somebody edits a deprecated page, log it in global log, but not local log glog.add(self.request, mtime_usecs, rev, action, self.page_name, None, extra, comment) if not was_deprecated and self.do_revision_backup: # if we did not create a new revision number, do not locally log it llog.add(self.request, mtime_usecs, rev, action, self.page_name, None, extra, comment) finally: if got_lock: filesys.rename(clfn, cfn) # add event log entry elog = eventlog.EventLog(self.request) elog.add(self.request, 'SAVEPAGE', {'pagename': self.page_name}, 1, mtime_usecs) return mtime_usecs, rev def saveText(self, newtext, rev, **kw): """ Save new text for a page. @param newtext: text to save for this page @param rev: revision of the page @keyword trivial: trivial edit (default: 0) @keyword extra: extra info field (e.g. for SAVE/REVERT with revno) @keyword comment: comment field (when preview is true) @keyword action: action for editlog (default: SAVE) @rtype: unicode @return: error msg """ _ = self._ backup_url = self._make_backup(newtext, **kw) action = kw.get('action', 'SAVE') #!!! need to check if we still retain the lock here #!!! rev check is not enough since internal operations use "0" # expand variables, unless it's a template or form page if not wikiutil.isTemplatePage(self.request, self.page_name): newtext = self._expand_variables(newtext) msg = "" if not self.request.user.may.save(self, newtext, rev, **kw): msg = _('You are not allowed to edit this page!') raise self.AccessDenied, msg elif not self.isWritable(): msg = _('Page is immutable!') raise self.Immutable, msg elif not newtext: msg = _('You cannot save empty pages.') raise self.EmptyPage, msg elif rev != 0 and rev != self.current_rev(): # check if we already saved that page other = False pagelog = self.getPagePath('edit-log', use_underlay=0, isfile=1) next_line = None for line in editlog.EditLog(self.request, pagelog).reverse(): if int(line.rev)==int(rev): break if not line.is_from_current_user(self.request): other = True next_line = line if next_line and next_line.is_from_current_user(self.request): saved_page = Page(self.request, self.page_name, rev=int(next_line.rev)) if newtext==saved_page.get_raw_body(): msg = _("You already saved this page!") return msg msg = _("""Sorry, someone else saved the page while you edited it. Please do the following: Use the back button of your browser, and cut&paste your changes from there. Then go forward to here, and click EditText again. Now re-add your changes to the current page contents. ''Do not just replace the content editbox with your version of the page, because that would delete the changes of the other person, which is excessively rude!'' """) if backup_url: msg += "<p>%s</p>" % _( 'A backup of your changes is [%(backup_url)s here].') % {'backup_url': backup_url} raise self.EditConflict, msg elif newtext == self.get_raw_body(): msg = _('You did not change the page content, not saved!') raise self.Unchanged, msg else: from wikiacl import parseACL # Get current ACL and compare to new ACL from newtext. If # they are not the sames, the user must have admin # rights. This is a good place to update acl cache - instead # of wating for next request. acl = self.getACL(self.request) if (not self.request.user.may.admin(self.page_name) and parseACL(self.request, newtext) != acl and action != "SAVE/REVERT"): msg = _("You can't change ACLs on this page since you have no admin rights on it!") raise self.NoAdmin, msg # save only if no error occurred (msg is empty) if not msg: # set success msg msg = _("Thank you for your changes. Your attention to detail is appreciated.") # determine action for edit log if action == 'SAVE' and not self.exists(): action = 'SAVENEW' comment = kw.get('comment', u'') extra = kw.get('extra', u'') trivial = kw.get('trivial', 0) # write the page file mtime_usecs, rev = self._write_file(newtext, action, comment, extra) self.clean_acl_cache() # send notification mails if self.request.cfg.mail_enabled: msg = msg + self._notifySubscribers(comment, trivial) if self.request.cfg.lupy_search: from MoinMoin import lupy index = lupy.Index(self.request) # When we have automatic index building, we can add to # the queue even if the index is missing. if index.exists(): index.update_page(self) # remove lock (forcibly if we were allowed to break it by the UI) # !!! this is a little fishy, since the lock owner might not notice # we broke his lock ==> but revision checking during preview will self.lock.release(force=not msg) # XXX does "not msg" make any sense? return msg class PageLock: """ PageLock - Lock pages """ # TODO: race conditions throughout, need to lock file during queries & update def __init__(self, pageobj): """ """ self.pageobj = pageobj self.page_name = pageobj.page_name request = pageobj.request self.request = request self._ = self.request.getText self.cfg = self.request.cfg # current time and user for later checks self.now = int(time.time()) self.uid = request.user.valid and request.user.id or request.remote_addr # get details of the locking preference, i.e. warning or lock, and timeout self.locktype = None self.timeout = 10 * 60 # default timeout in minutes if self.cfg.edit_locking: lockinfo = self.cfg.edit_locking.split() if 1 <= len(lockinfo) <= 2: self.locktype = lockinfo[0].lower() if len(lockinfo) > 1: try: self.timeout = int(lockinfo[1]) * 60 except ValueError: pass def acquire(self): """ Begin an edit lock depending on the mode chosen in the config. @rtype: tuple @return: tuple is returned containing 2 values: * a bool indicating successful acquiry * a string giving a reason for failure or an informational msg """ if not self.locktype: # we are not using edit locking, so always succeed return 1, '' _ = self._ #!!! race conditions, need to lock file during queries & update self._readLockFile() bumptime = self.request.user.getFormattedDateTime(self.now + self.timeout) timestamp = self.request.user.getFormattedDateTime(self.timestamp) owner = self.owner_html secs_valid = self.timestamp + self.timeout - self.now # do we own the lock, or is it stale? if self.owner is None or self.uid == self.owner or secs_valid < 0: # create or bump the lock self._writeLockFile() msg = [] if self.owner is not None and -10800 < secs_valid < 0: mins_ago = secs_valid / -60 msg.append(_( "The lock of %(owner)s timed out %(mins_ago)d minute(s) ago," " and you were granted the lock for this page." ) % {'owner': owner, 'mins_ago': mins_ago}) if self.locktype == 'lock': msg.append(_( "Other users will be ''blocked'' from editing this page until %(bumptime)s." ) % {'bumptime': bumptime}) else: msg.append(_( "Other users will be ''warned'' until %(bumptime)s that you are editing this page." ) % {'bumptime': bumptime}) msg.append(_( "Use the Preview button to extend the locking period." )) result = 1, '\n'.join(msg) else: mins_valid = (secs_valid+59) / 60 if self.locktype == 'lock': # lout out user result = 0, _( "This page is currently ''locked'' for editing by %(owner)s until %(timestamp)s," " i.e. for %(mins_valid)d minute(s)." ) % {'owner': owner, 'timestamp': timestamp, 'mins_valid': mins_valid} else: # warn user about existing lock result = 1, _( """This page was opened for editing or last previewed at %(timestamp)s by %(owner)s.[[BR]] '''You should ''refrain from editing'' this page for at least another %(mins_valid)d minute(s), to avoid editing conflicts.'''[[BR]] To leave the editor, press the Cancel button.""") % { 'timestamp': timestamp, 'owner': owner, 'mins_valid': mins_valid} return result def release(self, force=0): """ Release lock, if we own it. @param force: if 1, unconditionally release the lock. """ if self.locktype: # check that we own the lock in order to delete it #!!! race conditions, need to lock file during queries & update self._readLockFile() if force or self.uid == self.owner: self._deleteLockFile() def _filename(self): """get path and filename for edit-lock file""" return self.pageobj.getPagePath('edit-lock', isfile=1) def _readLockFile(self): """Load lock info if not yet loaded.""" _ = self._ self.owner = None self.owner_html = wikiutil.escape(_("<unknown>")) self.timestamp = 0 if self.locktype: try: entry = editlog.EditLog(self.request, filename=self._filename()).next() except StopIteration: entry = None if entry: self.owner = entry.userid or entry.addr self.owner_html = entry.getEditor(self.request) self.timestamp = wikiutil.version2timestamp(entry.ed_time_usecs) def _writeLockFile(self): """Write new lock file.""" self._deleteLockFile() try: editlog.EditLog(self.request, filename=self._filename()).add( self.request, wikiutil.timestamp2version(self.now), 0, "LOCK", self.page_name) except IOError: pass def _deleteLockFile(self): """Delete the lock file unconditionally.""" try: os.remove(self._filename()) except OSError: pass