Project

General

Profile

Actions

Feature #2065

closed

Commit graph should merge homonyms

Added by Thomas Lecavelier about 16 years ago. Updated about 10 years ago.

Status:
Closed
Priority:
Low
Category:
SCM
Target version:
-
Start date:
Due date:
% Done:

80%

Estimated time:
1.00 h
Resolution:
Wont fix

Description

When looking to "commits by authors" graph (Repository/Statistics), redmine don't merge bars with homonyms. In this redmine instance you can see my name appears twice.
I suppose it's due to the facts I'm using a git repository with multiple account, where I haven't put the same email address.

IMHO, Redmine should merge bars with homonyms.


Files

redmine-merge-homonyms.patch (1.7 KB) redmine-merge-homonyms.patch Patch merging homonyms for app app/controllers/repositories_controller.rb Guillaume Ayoub, 2009-11-11 15:39
redmine-merge-homonyms-39c9727.patch (1.81 KB) redmine-merge-homonyms-39c9727.patch Patch for revision 39c9727 Guillaume Ayoub, 2010-02-25 13:46

Related issues

Related to Redmine - Defect #13487: Honor committer => user mapping in repository statisticsClosedJean-Baptiste Barth

Actions
Actions #1

Updated by Patrick Naubert almost 16 years ago

We have the same problem. We are also using Git with multiple accounts. Perhaps a regex search-and-replace capability at repo import/update could be done ?

Actions #2

Updated by Guillaume Ayoub about 15 years ago

Here is a patch (for version 0.8.6) merging homonyms. I don’t know ruby very well and my syntax may be a little bit strange, but some ruby guru should clean these few lines easily.

Works with git, not tested with other version control systems.

Actions #3

Updated by Ammler _ over 14 years ago

I would like this feature quite much in too.

might it be possible to update to current heads of 0.9 and trunk?

Actions #4

Updated by Guillaume Ayoub over 14 years ago

Here is the previous patch updated for revision 39c9727 of git. As I said, this code works here with git, but it should be cleaned and tested with other version control systems.

Actions #5

Updated by Ibrahim Awwal over 14 years ago

Is there any chance of this being integrated into a release sometime? I will have to try applying your patch when I get a chance.

Actions #6

Updated by Ibrahim Awwal over 14 years ago

Ibrahim Awwal wrote:

Is there any chance of this being integrated into a release sometime? I will have to try applying your patch when I get a chance.

Ah, it seems to work on 0.9.5. Great!

Actions #7

Updated by Toshi MARUYAMA almost 14 years ago

  • Category changed from UI to SCM
Actions #8

Updated by Anonymous over 11 years ago

This feature is implemented (as a side effect) by my patch #13487.

Actions #9

Updated by Jean-Baptiste Barth about 10 years ago

  • Status changed from New to Closed
  • Assignee set to Jean-Baptiste Barth
  • Resolution set to Wont fix
Actually I don't think homonyms should be merged automatically. I know this answer comes very late in this issue, but let's imagine the following case (which is a real case for me at work :/) :
  • first user commits with John <john.smith@foo.net>
  • second user commits with John <john.doe@foo.net>

It's pretty obvious those users are not the same person, they don't even share the same family name. But it will be very difficult to detect that. So now that we have committers<->users mapping that can be defined per repository, I guess people could rely on that (or even script that!) to fit their needs.

I close this for now but please re-open it if you strongly disagree.

Actions #10

Updated by txemi M about 10 years ago

I would like not seeing this issue closed. I do not agree with the argument given above. I also think that there is no reliable way for redmine to guess identity mapping but I also think that this should not be done automatically. The administrator who added repository to some project should manually define the mapping.

Actions

Also available in: Atom PDF