Modify

Opened 9 years ago

Closed 9 years ago

#3145 closed defect (fixed)

[PATCH] Create New Wiki Page with a Tag

Reported by: Catalin BALAN Owned by: Alec Thomas
Priority: normal Component: TagsPlugin
Severity: major Keywords: unicode
Cc: Trac Release: 0.11

Description

test:

  • create new Page
  • enter text
  • add a tag
  • Submit Changes
  • all text is lost!

Attached patch fixes this.

Attachments (1)

tags.th.patch (663 bytes) - added by Catalin BALAN 9 years ago.

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by Catalin BALAN

Attachment: tags.th.patch added

comment:1 Changed 9 years ago by Alec Thomas

Resolution: fixed
Status: newclosed

(In [3808]) Fix for creation of new pages with tags. Thanks to Catalin Balin. Closes #3145.

comment:2 Changed 9 years ago by anonymous

Resolution: fixed
Status: closedreopened

Not for me. Running TracTags 0.6dev-r3882 trunk I have the same problem with editing existing pages which already have tags. You can 1. either change tags or change text, but not both at the same time. Changing tags changes the tags and does not change texts and vice versa. Trac 0.11 with sqlite and py25

comment:3 Changed 9 years ago by martin@…

Keywords: unicode added

I believe this is a reincarnation of #3206. For me, the latest drunk does not work with the symptoms of #3206. But this does work and does not have any problems. It also adds svn revision to the egg version as I run from trunk:

Index: tractags/wiki.py
===================================================================
--- tractags/wiki.py	(revision 3905)
+++ tractags/wiki.py	(working copy)
@@ -70,12 +70,6 @@
             # Always save tags if the page has been otherwise modified
             if page_modified:
                 self._update_tags(req, page)
-            elif page.version > 0:
-                # If the page hasn't been otherwise modified, save the tags
-                # and redirect so we don't get the "page has not been modified"
-                # warning
-                if self._update_tags(req, page):
-                    req.redirect(get_resource_url(self.env, page.resource, req.href, version=None))
         return []
 
     # IWikiChangeListener methods
Index: setup.cfg
===================================================================
--- setup.cfg	(revision 0)
+++ setup.cfg	(revision 0)
@@ -0,0 +1,3 @@
+[egg_info]
+tag_build = dev
+tag_svn_revision = true

comment:4 Changed 9 years ago by Alec Thomas

Resolution: fixed
Status: reopenedclosed

This should be fixed in r3877.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Alec Thomas.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.