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

[RFC]pms: refact multiproperty features #320

Open
wants to merge 1 commit into
base: 14.0
Choose a base branch
from

Conversation

DarioLodeiros
Copy link
Member

This pull request includes a significant update to the Property Management System (PMS) module, focusing on version updates, migration scripts, and refactoring of property-related logic.

Migration Scripts:

  • Added a new migration script pms/migrations/14.0.3.49.0/post-migration.py to update the list_price field of product.template and the amount field of pms.board.service.line based on the highest values from ir.pms.property records.

Refactoring Property-Related Logic:

  • Removed the get_active_property_ids method from pms/models/res_users.py and updated security rules in pms/security/pms_security.xml to use user.pms_property_ids.ids instead. [1] [2] [3] [4] [5] [6] [7]
  • Refactored pms/models/ir_pms_property.py to include computed fields model_name and field_name, and added logic to handle creation of ir.pms.property records with model_name and field_name. [1] [2] [3]
  • Removed default property logic and related methods from various models, including pms/models/account_move.py, pms/models/pms_board_service_line.py, pms/models/pms_folio.py, pms/models/pms_reservation.py, and pms/models/pms_room.py. [1] [2] [3] [4] [5] [6] [7] [8]
  • Refactored pms/models/product_template.py to remove computed and inverse methods for list_price and added a one-to-many relationship for property_daily_limits. [1] [2] [3]

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