Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[assets_management] creazione e configurazione cespiti con utente multicompany #3651

Open
2 of 3 tasks
odooNextev opened this issue Oct 6, 2023 · 1 comment · Fixed by #3614 or #4046
Open
2 of 3 tasks

[assets_management] creazione e configurazione cespiti con utente multicompany #3651

odooNextev opened this issue Oct 6, 2023 · 1 comment · Fixed by #3614 or #4046
Labels
16.0 bug hotfix needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue. required

Comments

@odooNextev
Copy link
Contributor

odooNextev commented Oct 6, 2023

Module

assets_management

Describe the bug

Attualmente ci sono diversi errori nell'app dei cespiti in un ambiente multi azienda con gli utenti che possono accedere a più aziende:

  • Il primo è comune a praticamente tutti i modelli dell'app perchè durante la creazione degli oggetti viene impostata di default l'azienda predefinita dell'utente, non considerando quella attiva in quel momento per l'utente e questo è dovuto ad un'impostazione errata della funzione che recupera i default per i campi company_id: https://github.com/OCA/l10n-italy/blob/14.0/assets_management/models/asset_category.py#L16
  • Oltretutto quando si crea una categoria di cespiti (Assets> Asset Configuration > Assets category) con un utente che in quel momento ne ha abilitate più di una, i campi dei conti e dei sezionali non vengono filtrati per l'azienda impostata nel form, ma per le aziende abilitate.
  • Anche quando si crea un cespite (Assets> Asset Management > Assets) in un ambiente multi azienda con un utente che ne ha abilitate più di una, i campi category_id e depreciation_ids non vengono filtrati per l'azienda impostata nel form, ma per le aziende abilitate.

To Reproduce

Affected versions:

Steps to reproduce the behavior:

  1. accedere ad un DB multi azienda con un utente che è abilitato a più di un'azienda
  2. abilitare più aziende
    image
    My company (Chicago) --> abilitata
    My company (San Francisco) --> abilitata ed attivata
  3. creare una categoria di cespiti (Assets> Asset Configuration > Assets category)
    image
    image
    Come si può vedere l'azienda proposta di default è quella predefinita dell'utente e non quella attiva.
  4. aprire la tendina del sezionale e come si può notare ci sono sia quelli di un'azienda che dell'altra
    image
  5. disabilitando e lasciando solo un'azienda, i sezionali ed i conti vengono filtrati per essa, quando invece dovrebbero essere filtrati per l'azienda impostata nel form permettendo quindi di mischiare i dati delle aziende
    image
    image

Expected behavior
Bisognerebbe impostare i campi company_id in sola lettura e recuperati dall'azienda attiva in quel momento per l'utente corrente per evitare confusione iniziale.
Poi si dovrebbe filtrare i vari campi (sezionali, conti, categorie e tag) per l'azienda definita nel form che di conseguenza è quella attivata per l'utente in quel momento.

Additional context
E' un comportamento simile a quello descritto nella issue #3650 per le configurazioni delle Ri.Ba

@odooNextev odooNextev added the bug label Oct 6, 2023
@odooNextev odooNextev changed the title [assets_management] creazione configurazioni Ri.Ba con utente multicompany [assets_management] creazione e configurazione cespiti con utente multicompany Oct 11, 2023
@francesco-ooops francesco-ooops linked a pull request Dec 19, 2023 that will close this issue
@SirAionTech
Copy link
Contributor

Per 14.0 va aggiunta anche #4046.

@francesco-ooops francesco-ooops added needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue. labels Nov 8, 2024
@tafaRU tafaRU added the hotfix label Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
16.0 bug hotfix needs porting This issue has already been resolved for some version no stale Use this label to prevent the automated stale action from closing this PR/Issue. required
Projects
None yet
4 participants