Skip to content

[FIX] deltatech_picking_transit: sudo la crearea celui de-al doilea transfer (tichet 8970) - #2628

Open
VoicuStefan2001 wants to merge 1 commit into
19.0from
19.0-fix-deltatech_picking_transit
Open

[FIX] deltatech_picking_transit: sudo la crearea celui de-al doilea transfer (tichet 8970)#2628
VoicuStefan2001 wants to merge 1 commit into
19.0from
19.0-fix-deltatech_picking_transit

Conversation

@VoicuStefan2001

Copy link
Copy Markdown
Collaborator

deltatech_picking_transit — sudo la crearea celui de-al doilea transfer (port 18→19)

Referință: tichet 8.970 · PR pereche pe 18.0: #2627

Context / de ce

La transferul în doi pași, al doilea transfer se creează în depozitul de destinație, în
numele operatorului care validează primul transfer. Dacă operatorul nu are drepturi pe
tipul de operație / locațiile depozitului de destinație (reguli de înregistrare,
multi-company), fluxul pică cu eroare de acces.

Ce s-a schimbat

models/stock_picking.py (identic cu PR-ul de 18.0):

  • button_validate (fluxul automat): căutările de stock.warehouse și
    stock.picking.type se fac acum cu sudo().
  • create_second_transfer_wizard: argumentele picking_type_id /
    final_dest_location_id sunt ridicate la sudo(), iar al doilea picking se creează cu
    sudo() — astfel action_confirm(), message_post și write-urile ulterioare pe el
    trec de restricțiile operatorului. Acoperă atât fluxul automat cât și wizard-ul manual.
  • copy_move_lines: mutările se copiază cu move.sudo().copy(...).

__manifest__.py: bump versiune 19.0.0.0.919.0.0.0.10.

static/description/index.html: regenerat de hook-ul pre-commit tb-skin.

Testare

  • Modulul nu are teste active (fișierul de test e integral comentat).
  • Pe 18.0, modulul a fost actualizat cu --test-enable pe baza locală datus fără erori;
    pe 19.0 verificat sintactic + pre-commit (ruff, ruff-format, pylint-odoo) verde.

Note / riscuri

  • În wizard-ul manual, dropdown-ul de tip de operație rămâne filtrat de drepturile
    operatorului — dacă operatorul nu vede deloc tipul de operație, nu îl poate selecta
    manual. Fluxul automat (auto_second_transfer) e acoperit complet.
  • sudo() ocolește și regulile multi-company la căutarea depozitului după partener —
    domeniul e specific (partenerul depozitului), deci riscul de potrivire greșită e mic.

Generated with Claude Code

🤖 Generated with Claude Code

…ransfer (tichet 8970)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@VoicuStefan2001
VoicuStefan2001 force-pushed the 19.0-fix-deltatech_picking_transit branch from 61d39f9 to 76aab47 Compare July 7, 2026 09:19
@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.11%. Comparing base (565b10e) to head (76aab47).

Files with missing lines Patch % Lines
deltatech_picking_transit/models/stock_picking.py 0.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             19.0    #2628      +/-   ##
==========================================
- Coverage   73.13%   73.11%   -0.03%     
==========================================
  Files         733      733              
  Lines       13784    13786       +2     
  Branches     2471     2471              
==========================================
- Hits        10081    10079       -2     
- Misses       2886     2889       +3     
- Partials      817      818       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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