ps/cvs-proxy
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: filter out (none) tags from patchsets

Browse source 
- Modified cvs_client.py to skip tags with value '(none)' using case-insensitive comparison
- Updated ui.js to also check for '(none)' tags when displaying patchsets
- Patchsets without tags no longer show any tag badge in the UI
This commit is contained in:
Juan José Gutiérrez de Quevedo Pérez 2025-11-21 18:49:10 +01:00
parent d3b40ae93f
commit df78bab6f4
2 changed files with 5 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
cvs_proxy/cvs_client.py
View file

@ -366,7 +366,10 @@ class CVSClient:
current_patchset['author'] = author_match.group(1) current_patchset['author'] = author_match.group(1)
if tag_match: if tag_match:
current_patchset['tag'] = tag_match.group(1) tag_value = tag_match.group(1)
# Don't store tags that are "(none)" or "(None)" - treat them as no tag (case-insensitive)
if tag_value.lower() != '(none)':
current_patchset['tag'] = tag_value
if log_match: if log_match:
# Log section starts, capture until next PatchSet or end # Log section starts, capture until next PatchSet or end

2
ui/ui.js
View file

@ -509,7 +509,7 @@ class UIManager {
<div class="patchset-number">PatchSet #${ps.patchset}</div> <div class="patchset-number">PatchSet #${ps.patchset}</div>
<div class="patchset-log-section"> <div class="patchset-log-section">
<div class="patchset-log">${ps.log ? this.escapeHtml(ps.log) : ''}</div> <div class="patchset-log">${ps.log ? this.escapeHtml(ps.log) : ''}</div>
${ps.tag && ps.tag !== 'N/A' ? `<span class="patchset-tag">${ps.tag}</span>` : ''} ${ps.tag && ps.tag !== 'N/A' && ps.tag.toLowerCase() !== '(none)' ? `<span class="patchset-tag">${ps.tag}</span>` : ''}
</div> </div>
<div class="patchset-author">${ps.author || 'Unknown'}</div> <div class="patchset-author">${ps.author || 'Unknown'}</div>
<div class="patchset-date" title="${ps.date || 'N/A'}">${this.getRelativeDate(ps.date) || 'N/A'}</div> <div class="patchset-date" title="${ps.date || 'N/A'}">${this.getRelativeDate(ps.date) || 'N/A'}</div>