Describe your changes

Author: John75SunCity

.devcontainer/.vscode/settings.json

@@ -4,13 +4,6 @@
     // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
     "version": "0.2.0",
     "configurations": [
-        {
-            "name": "Python Debugger: Current File",
-            "type": "debugpy",
-            "request": "launch",
-            "program": "${file}",
-            "console": "integratedTerminal"
-        },
         {
             "name": "Python Debugger: Current File",
             "type": "debugpy",

.devcontainer/.vscode/README.md .vscode/README.md.devcontainer/.vscode/README.md renamed to .vscode/README.md

@@ -0,0 +1,6 @@
+{
+    "python.analysis.typeCheckingMode": "basic",
+    "python.analysis.extraPaths": [
+        "/workspaces/ssh-git-github.com-odoo-odoo.git-8.0"
+    ]
+}

.devcontainer/.vscode/launch.json .vscode/launch.json.devcontainer/.vscode/launch.json renamed to .vscode/launch.json

@@ -1,5 +1,4 @@
 from odoo import http
-from odoo.http import request
 import logging
 
 _logger = logging.getLogger(__name__)
@@ -11,31 +10,31 @@ INTERNAL_USAGE = 'internal'
 class MyController(http.Controller):
 
     def _get_partner_inventory(self, partner):
-        serials = request.env['stock.production.lot'].search([(CUSTOMER_FIELD, '=', partner.id)])
-        return request.env['stock.quant'].search([
+        serials = http.request.env['stock.production.lot'].search([(CUSTOMER_FIELD, '=', partner.id)])
+        return http.request.env['stock.quant'].search([
             ('lot_id', 'in', serials.ids),
             ('location_id.usage', '=', INTERNAL_USAGE)
         ])
 
     @http.route('/my/inventory/request_pickup', type='http', auth='user', methods=['POST'], csrf=True)
     def request_pickup(self, **kw):
-        user = request.env.user
+        user = http.request.env.user
         partner = user.partner_id
         _logger.info('Inventory accessed by partner %s', partner.id)
         try:
-            item_ids = [int(id) for id in request.httprequest.form.getlist('item_ids')]
+            item_ids = [int(id) for id in http.request.httprequest.form.getlist('item_ids')]
         except ValueError:
-            _logger.warning("User %s (ID: %s) provided invalid item_ids: %s", user.login, user.id, request.httprequest.form.getlist('item_ids'))
-            return request.redirect('/my/inventory?error=invalid_item_ids')
+            _logger.warning("User %s (ID: %s) provided invalid item_ids: %s", user.login, user.id, http.request.httprequest.form.getlist('item_ids'))
+            return http.request.redirect('/my/inventory?error=invalid_item_ids')
         _logger.info('Pickup requested for items %s by partner %s', item_ids, partner.id)
         # ...existing code...
 
     @http.route('/my/inventory', type='http', auth='user', website=True, methods=['GET'], csrf=True)
     def my_inventory(self, **kw):
-        user = request.env.user
+        user = http.request.env.user
         partner = user.partner_id
         quants = self._get_partner_inventory(partner)
-        return request.render(
+        return http.request.render(
             'records_management.inventory_template',
             {
                 'quants': quants,

.vscode/settings.json

@@ -0,0 +1,23 @@
+version: '2'
+services:
+  db:
+    image: postgres:10
+    environment:
+      - POSTGRES_DB=postgres
+      - POSTGRES_USER=odoo
+      - POSTGRES_PASSWORD=odoo
+    restart: always
+
+  odoo:
+    image: odoo:8.0
+    depends_on:
+      - db
+    ports:
+      - "8069:8069"
+    volumes:
+      - ./records_management:/mnt/extra-addons/records_management
+    environment:
+      - HOST=db
+      - USER=odoo
+      - PASSWORD=odoo
+    restart: always

HTTP Controller

@@ -1,40 +1,40 @@
 from odoo import http
-from odoo.http import request
 
 class InventoryPortal(http.Controller):
     @http.route('/my/inventory', type='http', auth='user', website=True)
-    def inventory(self, **kw):
+    def inventory(self):
         # Authorization check
-        if not request.env.user.has_group('records_management.group_inventory_access'):
-            return request.redirect('/web/login?error=access_denied')
-        partner = request.env.user.partner_id
+        if not http.request.env.user.has_group('records_management.group_inventory_access'):
+            return http.request.redirect('/web/login?error=access_denied')
+        partner = http.request.env.user.partner_id
         if not partner:
-            return request.redirect('/my/inventory?error=partner_not_found')
-        serials = request.env['stock.production.lot'].search([('customer_id', '=', partner.id)])
-        quants = request.env['stock.quant'].search([
+            return http.request.redirect('/my/inventory?error=partner_not_found')
+        serials = http.request.env['stock.production.lot'].search([('customer_id', '=', partner.id)])
+        quants = http.request.env['stock.quant'].search([
             ('lot_id', 'in', serials.ids), 
             ('location_id.usage', '=', 'internal')
         ])
-        return request.render('records_management.inventory_template', {'quants': quants})
+        return http.request.render('records_management.inventory_template', {'quants': quants})
 
     @http.route('/my/request_pickup', type='http', auth='user', website=True, methods=['POST'])
-    def request_pickup(self, **post):
+    def request_pickup(self):
         # Authorization check
-        if not request.env.user.has_group('records_management.group_inventory_access'):
-            return request.redirect('/web/login?error=access_denied')
-        partner = request.env.user.partner_id
+        if not http.request.env.user.has_group('records_management.group_inventory_access'):
+            return http.request.redirect('/web/login?error=access_denied')
+        partner = http.request.env.user.partner_id
         if not partner:
-            return request.redirect('/my/inventory?error=partner_not_found')
-        item_ids = [int(id) for id in request.httprequest.form.getlist('item_ids')]
+            return http.request.redirect('/my/inventory?error=partner_not_found')
+        item_ids = [int(id) for id in http.request.httprequest.form.getlist('item_ids')]
         if not item_ids:
-            return request.redirect('/my/inventory?error=no_items_selected')
-        items = request.env['stock.production.lot'].search([
+            return http.request.redirect('/my/inventory?error=no_items_selected')
+        items = http.request.env['stock.production.lot'].search([
             ('id', 'in', item_ids),
             ('customer_id', '=', partner.id)
         ])
         if items:
-            request.env['pickup.request'].create({
+            http.request.env['pickup.request'].create({
                 'customer_id': partner.id,
                 'item_ids': [(6, 0, items.ids)],
             })
+        return http.request.redirect('/my/inventory')
         return request.redirect('/my/inventory')

docker-compose.yml

@@ -1 +0,0 @@
-from . import scrm_records_management

records_management/controllers/portal.py

@@ -1,4 +1,4 @@
-from odoo import fields, models
+from odoo import fields, models  # type: ignore
 
 class StockProductionLot(models.Model):
     _inherit = 'stock.production.lot'