Ticket #5486 (closed enhancement: fixed)

Opened 4 years ago

Last modified 2 years ago

[Patch] "Back to ticket #X" should be located in contextual navigation

Reported by: rjollos Assigned to: rjollos
Priority: high Component: TicketModifiedFilesPlugin
Severity: minor Keywords: patch
Cc: Trac Release: 0.11

Description

It would be more consistent with the layout of Trac if the "Back to ticket #X." link appeared in the contextual navigation rather than at the bottom of the page. For instance, when you select a particular ticket from a report page, the contextual navigation contains: | ← Previous Ticket | Back to Query | Next Ticket → |

Attachments

ticketmodifiedfiles.html.patch (0.9 kB) - added by rjollos on 09/13/09 11:30:12.
Patch for ticketmodifiedfiles.html
ctxtnav.patch (1.8 kB) - added by rjollos on 01/28/10 02:45:59.
Improved patch

Change History

07/08/09 04:40:38 changed by anonymous

  • type changed from defect to enhancement.

(follow-ups: ↓ 3 ↓ 6 ) 07/08/09 17:04:31 changed by e2jk

  • severity changed from normal to minor.

Are you familiar with trac plugins creation? If no, this might be a good opportunity to start by creating a patch so that the "Back to ticket #X" link appears in the contextual navigation ;)

I have to admit that I don't have much time to develop this myself, so if you want to see this enhancement I'd be happy to review any patch that you submit.

(in reply to: ↑ 2 ) 07/08/09 17:09:29 changed by anonymous

Hi,

I am not familiar with Trac development, or event Python for that matter (my experience is with C/C++/C# and MATLAB), but I am somewhat familiar with Trac administration and intend to learn to do plug-in development.

I see that there is some fairly nice looking development documentation so I imagine it will be fairly straightforward to get started. Thank you for offering to review the patch. I will let you know what I have within a few weeks.

- Ryan

Replying to e2jk:

Are you familiar with trac plugins creation? If no, this might be a good opportunity to start by creating a patch so that the "Back to ticket #X" link appears in the contextual navigation ;) I have to admit that I don't have much time to develop this myself, so if you want to see this enhancement I'd be happy to review any patch that you submit.

07/08/09 17:42:10 changed by e2jk

The documentation is pretty good, in fact this was my first Trac plugin and I did it all following the documentation. And Python is really not hard, learning it could be of great use to you. If you need a good Python resource, I greatly recommend Dive Into Python: http://diveintopython.org/

Have fun learning! I hope to hear back from you in a few weeks!

08/06/09 18:04:52 changed by e2jk

  • status changed from new to assigned.
  • priority changed from normal to lowest.

(in reply to: ↑ 2 ) 09/13/09 11:26:51 changed by rjollos

  • status changed from assigned to new.
  • owner changed from e2jk to rjollos.

Replying to e2jk:

Are you familiar with trac plugins creation? If no, this might be a good opportunity to start by creating a patch so that the "Back to ticket #X" link appears in the contextual navigation ;)

Hello again,

I have created a patch to address this enhancement. It took me quite a while to get up and running with the Trac install, but after that, it was a fairly straightforward enhancement. I will upload shortly.

- Ryan

09/13/09 11:30:12 changed by rjollos

  • attachment ticketmodifiedfiles.html.patch added.

Patch for ticketmodifiedfiles.html

09/13/09 11:42:54 changed by rjollos

Hello,

I have one other quick question for you. Although the plugin install seemed to go fine, I was seeing an error at the very end of the output of the install. The presumed error is bad local file header. I installed the plugin by checking out from svn and using easy_install. When I made a change to the source, I would run easy_install again and restart apache. That seemed to effectively 'upgrade' the plugin. 

rjollos@pc-rjollos-ubuntu-server-9:~/ticketmodifiedfilesplugin$ sudo easy_install 0.11
Processing 0.11
Running setup.py -q bdist_egg --dist-dir /home/rjollos/ticketmodifiedfilesplugin/0.11/egg-dist-tmp-Bo_lYC
zip_safe flag not set; analyzing archive contents...
TicketModifiedFiles 1.00 is already the active version in easy-install.pth

Installed /usr/local/lib/python2.6/dist-packages/TicketModifiedFiles-1.00-py2.6.egg
Processing dependencies for TicketModifiedFiles==1.00
Traceback (most recent call last):
  File "/usr/local/bin/easy_install", line 8, in <module>
    load_entry_point('setuptools==0.6c9', 'console_scripts', 'easy_install')()
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/setuptools/command/easy_install.py", line 1671, in main
    
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/setuptools/command/easy_install.py", line 1659, in with_ei_usage
    
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/setuptools/command/easy_install.py", line 1675, in <lambda>
    
  File "/usr/lib/python2.6/distutils/core.py", line 152, in setup
    dist.run_commands()
  File "/usr/lib/python2.6/distutils/dist.py", line 975, in run_commands
    self.run_command(cmd)
  File "/usr/lib/python2.6/distutils/dist.py", line 995, in run_command
    cmd_obj.run()
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/setuptools/command/easy_install.py", line 211, in run
    
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/setuptools/command/easy_install.py", line 427, in easy_install
    
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/setuptools/command/easy_install.py", line 478, in install_item
    
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/setuptools/command/easy_install.py", line 519, in process_distribution
    
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/pkg_resources.py", line 529, in resolve
    requirements.extend(dist.requires(req.extras)[::-1])
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/pkg_resources.py", line 2108, in requires
    dm = self._dep_map
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/pkg_resources.py", line 2100, in _dep_map
    for extra,reqs in split_sections(self._get_metadata(name)):
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/pkg_resources.py", line 2519, in split_sections
    for line in yield_lines(s):
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/pkg_resources.py", line 1813, in yield_lines
    for ss in strs:
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/pkg_resources.py", line 2122, in _get_metadata
    for line in self.get_metadata_lines(name):
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/pkg_resources.py", line 1140, in get_metadata_lines
    return yield_lines(self.get_metadata(name))
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/pkg_resources.py", line 1137, in get_metadata
    return self._get(self._fn(self.egg_info,name))
  File "/usr/local/lib/python2.6/dist-packages/setuptools-0.6c9-py2.6.egg/pkg_resources.py", line 1197, in _get
    return self.loader.get_data(path)
zipimport.ZipImportError: bad local file header in /usr/local/lib/python2.6/dist-packages/TicketModifiedFiles-1.00-py2.6.egg

09/13/09 11:44:46 changed by rjollos

  • status changed from new to assigned.

10/29/09 21:15:23 changed by anonymous

  • summary changed from "Back to ticket #X" should be located in contextual navigation to [Patch] "Back to ticket #X" should be located in contextual navigation.

01/28/10 02:44:48 changed by rjollos

  • status changed from assigned to new.
  • owner changed from rjollos to e2jk.

Improved patch is attached: ctxtnav.patch.

01/28/10 02:45:59 changed by rjollos

  • attachment ctxtnav.patch added.

Improved patch

01/28/10 02:47:12 changed by rjollos

The first attempt I made at this patch was to modify the template, which was incorrect. The correct way was using the add_ctxtnav function. Tested under Trac 0.11.7stable-r9029.

02/09/10 09:23:46 changed by rjollos

Hello,

Just wanted to inquire as to whether you might be able to apply this simple patch to the trunk.

Thanks, - Ryan

02/09/10 09:47:23 changed by e2jk

Hi Ryan,

This is on my TODO list, but I haven't found the time to test it yet. I'll have a look at it soon...

12/04/10 13:14:46 changed by rjollos

If its okay with you, I'd like to just go ahead and apply the patch. Its a simple patch and I have global r/w access, so I could commit it given the go-ahead by you ;)

(follow-up: ↓ 16 ) 12/06/10 14:36:55 changed by e2jk

Hi rjollos,

As I am no longer working for my previous company Rentalia, I have to ask them permission before (they wanted to add a specific term in the licence). I have thus sent them an email to ask if the patch can be applied, and also (more importantly IMO) if we can release this plugin to the community, and have someone adopt it.

I will let you know, so please don't apply the patch in the meantime.

Just a question, would you potentially be interested in owning this plugin?

(in reply to: ↑ 15 ) 12/06/10 21:04:12 changed by rjollos

Replying to e2jk:

I will let you know, so please don't apply the patch in the meantime.

Certainly.

Just a question, would you potentially be interested in owning this plugin?

I'd be happy to take over maintainership.

12/20/10 11:55:57 changed by e2jk

  • keywords set to patch.

12/20/10 22:35:50 changed by e2jk

  • owner changed from e2jk to rjollos.

Reassigning as part of maintainership transfer of TicketModifiedFiles? to rjollos.

12/24/10 07:40:41 changed by rjollos

  • status changed from new to assigned.
  • priority changed from lowest to high.

12/24/10 07:41:13 changed by rjollos

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [9703]) Moved 'Back to Ticket #X' link to the contextual navigation. Fixes #5486.

Add/Change #5486 ([Patch] "Back to ticket #X" should be located in contextual navigation)




Change Properties
Action