Edgewall Software
Modify

Ticket #720 (closed defect: fixed)

Opened 8 years ago

Last modified 8 years ago

Timeline should show filename for changeset commit

Reported by: utopiste Owned by: utopiste
Priority: normal Milestone: 0.8
Component: timeline Version: devel
Severity: normal Keywords:
Cc:
Release Notes:
API Changes:

Description

currently the changeset doesn't list the filename modified/added/deleted by a commit.

currently this option is disable by default

for active this feature

[timeline]

changeset_show_files=True

changeset_files_count=3

patch will follow in trunk

Attachments

Change History

comment:1 Changed 8 years ago by utopiste

  • Resolution set to fixed
  • Status changed from new to closed

Closed in #881

comment:2 Changed 8 years ago by cmlenz

This is actually the same request as #343 AFAICT. I'll close that one as duplicate then.

comment:3 Changed 8 years ago by cmlenz

BTW, the link was supposed to be [881] and not #881 I guess.

comment:4 Changed 8 years ago by tbrkic

  • Resolution fixed deleted
  • Status changed from closed to reopened

I tried to install r996 and got error when viewing the timeline

Traceback (most recent call last):
  File "/usr/lib/python2.3/site-packages/trac/ModPythonHandler.py", line 194, in handler
    core.dispatch_request(mpr.path_info, args, mpr, env)
  File "/usr/lib/python2.3/site-packages/trac/core.py", line 436, in dispatch_request
    module.run()
  File "/usr/lib/python2.3/site-packages/trac/Module.py", line 44, in run
    self.render()
  File "/usr/lib/python2.3/site-packages/trac/Timeline.py", line 232, in render
    changeset, wiki, milestone)
  File "/usr/lib/python2.3/site-packages/trac/Timeline.py", line 132, in get_info
    max_node = int(self.env.get_config('timeline', 'changeset_show_files',0))
ValueError: invalid literal for int(): False

This seems to be because of the line in Timeline:

max_node = int(self.env.get_config('timeline', 'changeset_show_files',0))

It seems that when you write false/true in trac.ini you get
a string back and not boolean.

In Notify.py this code is used:

        enabled = self.env.get_config('notification', 'smtp_enabled', '0')
        if not enabled.lower() in TRUE:

So I suggesst the max_node to change to:

    max_node = self.env.get_config('timeline', 'changeset_show_files',0)        
    if not max_node.lower() in TRUE:



comment:5 Changed 8 years ago by vittorio

  • Resolution set to fixed
  • Status changed from reopened to closed

the trac.ini format has changed. see TracIni and #841

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
to The owner will be changed from utopiste. Next status will be 'closed'
Author


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

 
Note: See TracTickets for help on using tickets.