Merge pull request #4384 from awsr/git-url-review

Minor git url handling review
This commit is contained in:
Vladimir Mandic
2025-11-08 19:58:41 -05:00
committed by GitHub
2 changed files with 6 additions and 13 deletions
+5 -12
View File
@@ -124,11 +124,8 @@ def check_updates(_id_task, disable_list, search_text, sort_column):
return create_html(search_text, sort_column), "Extension update complete | Restart required"
def normalize_git_url(url):
if url is None:
return ""
url = url.replace(".git", "")
return url
def normalize_git_url(url) -> str:
return '' if url is None else url.removesuffix('.git')
def install_extension_from_url(dirname, url, branch_name, search_text, sort_column):
@@ -139,19 +136,15 @@ def install_extension_from_url(dirname, url, branch_name, search_text, sort_colu
shared.log.error('Extension: url is not specified')
return ['', '']
if dirname is None or dirname == "":
*parts, last_part = url.split('/') # pylint: disable=unused-variable
last_part = normalize_git_url(last_part)
dirname = last_part
dirname = normalize_git_url(url.split('/')[-1])
target_dir = os.path.join(extensions.extensions_dir, dirname)
shared.log.info(f'Installing extension: {url} into {target_dir}')
if os.path.exists(target_dir):
shared.log.error(f'Extension: path="{target_dir}" directory already exists')
return ['', '']
normalized_url = normalize_git_url(url)
assert len([x for x in extensions.extensions if normalize_git_url(x.remote) == normalized_url]) == 0, 'Extension with this URL is already installed'
url = normalize_git_url(url)
assert len([x for x in extensions.extensions if normalize_git_url(x.remote) == url]) == 0, 'Extension with this URL is already installed'
tmpdir = os.path.join(paths.data_path, "tmp", dirname)
if url.endswith('.git'):
url = url.replace('.git', '')
try:
import git
shutil.rmtree(tmpdir, True)
+1 -1
View File
@@ -19,7 +19,7 @@ def get_version():
origin = origin.splitlines()[0]
version.branch = i.git('rev-parse --abbrev-ref HEAD')
version.branch = version.branch.splitlines()[0]
version.url = origin + '/tree/' + version.branch
version.url = origin.removesuffix('.git') + '/tree/' + version.branch
ver = i.git('log --pretty=format:"%h %ad" -1 --date=short')
ver = ver.splitlines()[0]