Feature/api v3/merge

[bglamadrid]

Addressing the newest iteration of the base API document. The following changes were made:

• Follow new specification for error messages
  • Introduce new model class AppError
  • Make the ExceptionsControllerAdvice use this new class on all its methods
• Rename all references of the word Sell and Sales to Order and Orders respectively
  • REST API endpoints
  • /data/sales -> /data/orders
  • /data/sell_statuses -> /data/order_statuses
  • Hardcoded Spring Security authorities
  • sales:(create|read|update|delete) -> orders:(...)
  • sell_statuses:(create|read|update|delete) -> order_statuses:(...)
  • SQL schema tables, columns, constraints, indexes and initial values
  • Entity table mappings included as well
  • Custom properties
  • trebol.api.able-to-edit-sales-after-being-processed
    • to trebol.api.able-to-edit-orders-after-being-processed
  • Class names, by package
  • org.trebol.api.
    • models.
    • SellPojo -> OrderPojo
    • SellDetailPojo -> OrderDetailPojo
    • SellStatusPojo -> OrderStatusPojo
    • services. (for both interfaces and implementations)
    • SalesProcessService -> OrdersProcessService
  • org.trebol.jpa.
    • entities. (and their QueryDSL counterparts)
    • Sell -> Order
    • SellDetail -> OrderDetail
    • SellStatus -> OrderStatus
    • repositories.
    • SalesRepository -> OrdersRepository
    • SellDetailsRepository -> OrderDetailsRepository
    • SellStatusesRepository -> OrderStatusesRepository
    • sortspecs.
    • SalesSortSpec -> OrdersSortSpec
    • SellStatusesSortSpec -> OrderStatusesSortSpec
    • services. (for both interfaces and implementations)
    • crud.
      • SalesCrudService -> OrdersCrudService
      • SellStatusesCrudService -> OrderStatusesCrudService
    • patch.
      • SalesPatchService -> OrdersPatchService
      • SellStatusesPatchService -> OrderStatusesPatchService
    • conversion.
      • SalesConverterService -> OrdersConverterService
      • SellStatusesConverterService -> OrderStatusesConverterService
    • predicates.
      • SalesPredicateService -> OrdersPredicateService
      • SellStatusesPredicateService -> OrderStatusesPredicateService
  • And all other constant and variable names that reference them
• remove CustomerPojo and SalespersonPojo classes
  • the OrderPojo class now relates to two instances of PersonPojo instead
• Change status codes for most /data endpoints
  • All POST requests return 201, except for three specific paths
  • /data/orders/confirmation
  • /data/orders/rejection
  • /data/orders/completion
  • All PUT,PATCH,DELETE,OPTIONS requests return 204

There is one thing left however. The auth-related resources that were removed from the specification will be still very much needed, and so this backend implementation will still have them. And so! They must be properly documented.

[sonarcloud[bot]]

Quality Gate passed

Issues
30 New issues
0 Accepted issues

Measures
0 Security Hotspots
83.9% Coverage on New Code
1.0% Duplication on New Code

See analysis details on SonarCloud