mirror of
				https://gitcode.com/gitea/gitea.git
				synced 2025-10-26 13:16:28 +08:00 
			
		
		
		
	 ea1d09718c
			
		
	
	ea1d09718c
	
	
	
		
			
			It is not correct to return tag data when commit data is requested, so remove the hacky code that overwrote parts of a commit with parts of a tag. This fixes commit retrieval by tag for both the latest commit in the UI and the commit info on tag webhook events. Fixes: https://github.com/go-gitea/gitea/issues/21687 Replaces: https://github.com/go-gitea/gitea/pull/21693 <img width="324" alt="Screenshot 2022-11-13 at 15 26 37" src="https://user-images.githubusercontent.com/115237/201526975-736c6ea7-ad6a-467a-a823-9a63d6ecb718.png"> <img width="789" alt="image" src="https://user-images.githubusercontent.com/115237/201526876-90a13ffc-1e5c-4d76-911b-f1ae51e8eaab.png"> --------- Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
		
			
				
	
	
		
			102 lines
		
	
	
		
			2.8 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			102 lines
		
	
	
		
			2.8 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| // Copyright 2015 The Gogs Authors. All rights reserved.
 | |
| // Copyright 2019 The Gitea Authors. All rights reserved.
 | |
| // SPDX-License-Identifier: MIT
 | |
| 
 | |
| //go:build gogit
 | |
| 
 | |
| package git
 | |
| 
 | |
| import (
 | |
| 	"strings"
 | |
| 
 | |
| 	"github.com/go-git/go-git/v5/plumbing"
 | |
| 	"github.com/go-git/go-git/v5/plumbing/object"
 | |
| )
 | |
| 
 | |
| // GetRefCommitID returns the last commit ID string of given reference (branch or tag).
 | |
| func (repo *Repository) GetRefCommitID(name string) (string, error) {
 | |
| 	ref, err := repo.gogitRepo.Reference(plumbing.ReferenceName(name), true)
 | |
| 	if err != nil {
 | |
| 		if err == plumbing.ErrReferenceNotFound {
 | |
| 			return "", ErrNotExist{
 | |
| 				ID: name,
 | |
| 			}
 | |
| 		}
 | |
| 		return "", err
 | |
| 	}
 | |
| 
 | |
| 	return ref.Hash().String(), nil
 | |
| }
 | |
| 
 | |
| // SetReference sets the commit ID string of given reference (e.g. branch or tag).
 | |
| func (repo *Repository) SetReference(name, commitID string) error {
 | |
| 	return repo.gogitRepo.Storer.SetReference(plumbing.NewReferenceFromStrings(name, commitID))
 | |
| }
 | |
| 
 | |
| // RemoveReference removes the given reference (e.g. branch or tag).
 | |
| func (repo *Repository) RemoveReference(name string) error {
 | |
| 	return repo.gogitRepo.Storer.RemoveReference(plumbing.ReferenceName(name))
 | |
| }
 | |
| 
 | |
| // ConvertToSHA1 returns a Hash object from a potential ID string
 | |
| func (repo *Repository) ConvertToSHA1(commitID string) (SHA1, error) {
 | |
| 	if len(commitID) == SHAFullLength {
 | |
| 		sha1, err := NewIDFromString(commitID)
 | |
| 		if err == nil {
 | |
| 			return sha1, nil
 | |
| 		}
 | |
| 	}
 | |
| 
 | |
| 	actualCommitID, _, err := NewCommand(repo.Ctx, "rev-parse", "--verify").AddDynamicArguments(commitID).RunStdString(&RunOpts{Dir: repo.Path})
 | |
| 	if err != nil {
 | |
| 		if strings.Contains(err.Error(), "unknown revision or path") ||
 | |
| 			strings.Contains(err.Error(), "fatal: Needed a single revision") {
 | |
| 			return SHA1{}, ErrNotExist{commitID, ""}
 | |
| 		}
 | |
| 		return SHA1{}, err
 | |
| 	}
 | |
| 
 | |
| 	return NewIDFromString(actualCommitID)
 | |
| }
 | |
| 
 | |
| // IsCommitExist returns true if given commit exists in current repository.
 | |
| func (repo *Repository) IsCommitExist(name string) bool {
 | |
| 	hash := plumbing.NewHash(name)
 | |
| 	_, err := repo.gogitRepo.CommitObject(hash)
 | |
| 	return err == nil
 | |
| }
 | |
| 
 | |
| func (repo *Repository) getCommit(id SHA1) (*Commit, error) {
 | |
| 	var tagObject *object.Tag
 | |
| 
 | |
| 	gogitCommit, err := repo.gogitRepo.CommitObject(id)
 | |
| 	if err == plumbing.ErrObjectNotFound {
 | |
| 		tagObject, err = repo.gogitRepo.TagObject(id)
 | |
| 		if err == plumbing.ErrObjectNotFound {
 | |
| 			return nil, ErrNotExist{
 | |
| 				ID: id.String(),
 | |
| 			}
 | |
| 		}
 | |
| 		if err == nil {
 | |
| 			gogitCommit, err = repo.gogitRepo.CommitObject(tagObject.Target)
 | |
| 		}
 | |
| 		// if we get a plumbing.ErrObjectNotFound here then the repository is broken and it should be 500
 | |
| 	}
 | |
| 	if err != nil {
 | |
| 		return nil, err
 | |
| 	}
 | |
| 
 | |
| 	commit := convertCommit(gogitCommit)
 | |
| 	commit.repo = repo
 | |
| 
 | |
| 	tree, err := gogitCommit.Tree()
 | |
| 	if err != nil {
 | |
| 		return nil, err
 | |
| 	}
 | |
| 
 | |
| 	commit.Tree.ID = tree.Hash
 | |
| 	commit.Tree.gogitTree = tree
 | |
| 
 | |
| 	return commit, nil
 | |
| }
 |