Skip to content

AvistaZ Network: Fix raw DVD names#1269

Open
wastaken7 wants to merge 2 commits intomasterfrom
az-net-name-fix
Open

AvistaZ Network: Fix raw DVD names#1269
wastaken7 wants to merge 2 commits intomasterfrom
az-net-name-fix

Conversation

@wastaken7
Copy link
Collaborator

@wastaken7 wastaken7 commented Feb 25, 2026

Summary by CodeRabbit

  • Bug Fixes
    • Improved naming normalization for DVDRIP content: preserves prior cleanup while adding explicit handling of source and audio segments to ensure consistent formatting.
    • Enhanced DVD disc metadata handling: removes region when appropriate, consolidates source/resolution display, and appends video codec to audio segment when present for clearer, more accurate labels.

@github-actions
Copy link

Thanks for taking the time to contribute to this project. Upload Assistant is currently in a complete rewrite, and no new development is being conducted on this python source at this time.

If you have come this far, please feel free to leave open, any pull requests regarding new sites being added to the source, as these can serve as the baseline for later conversion.

If your pull request relates to a critical bug, this will be addressed in this code base, and a new release published as needed.

If your pull request only addresses a quite minor bug, it is not likely to be addressed in this code base.

Details for the new code base will follow at a later date.

@coderabbitai
Copy link

coderabbitai bot commented Feb 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c817c7c and cca7ee3.

📒 Files selected for processing (1)
  • src/trackers/AVISTAZ_NETWORK.py
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/trackers/AVISTAZ_NETWORK.py

📝 Walkthrough

Walkthrough

The edit_name function in AVISTAZ_NETWORK now extracts local source and audio variables, preserves DVDRIP source-removal behavior, and adds DVD disc handling: removes region, replaces source with resolution when present, and appends video codec info to the audio segment. (48 words)

Changes

Cohort / File(s) Summary
DVD Naming Normalization
src/trackers/AVISTAZ_NETWORK.py
Modified edit_name to introduce local source and audio extraction, keep DVDRIP source cleanup, and add is_disc == "DVD" handling: strip region, replace source with resolution when both exist, and append video codec suffix to the audio segment.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~18 minutes

Poem

🐇 I nibbled through names, tidying each line,
DVDRIPs stay tidy, DVD discs now shine,
Regions peeled away, resolutions take stage,
Audio and codec join the naming page,
A hop, a fix — metadata in rhyme!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely identifies the main change: fixing how raw DVD names are handled in the AvistaZ Network tracker module.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch az-net-name-fix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@wastaken7 wastaken7 changed the title AvistaZ Network: Fixing raw DVD names AvistaZ Network: Fix raw DVD names Feb 25, 2026
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/trackers/AVISTAZ_NETWORK.py`:
- Around line 879-880: The current DVD branch calls upload_name.replace(source,
...) and replace(audio, ...) unconditionally which corrupts upload_name when
source or audio are empty; update the block handling meta.get("is_disc", "") ==
"DVD" to perform each replace only when the corresponding variable is non-empty
(e.g., check "if source:" then do upload_name = upload_name.replace(source,
meta.get("resolution","")) and similarly "if audio:" then replace audio with
f"{audio} {meta.get('video_codec','')}" ), preserving the existing region
replacement logic that runs unconditionally.

ℹ️ Review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8a1ee44 and c817c7c.

📒 Files selected for processing (1)
  • src/trackers/AVISTAZ_NETWORK.py

@wastaken7 wastaken7 requested a review from Audionut February 25, 2026 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant