fix: Remove deleted filter and primary key from insert

- Remove deleted=0 filter from existence check (allow soft-deleted updates) - Remove primary key from insert payload to avoid conflicts - Cleaner approach for upsert logic Address CodeRabbit review feedback
fix: Use primary key only check and atomic insert in saveValue
2026-05-26 09:15:28 -04:00 · 2026-04-15 17:08:20 +00:00 · 2026-04-15 16:01:48 +00:00 · 2026-04-15 15:29:43 +00:00 · 2026-04-15 15:10:41 +00:00
75 changed files with 1188 additions and 3579 deletions
--- a/.env.example
+++ b/.env.example
@@ -16,9 +16,6 @@ CI_ENVIRONMENT = production
 # Configure with comma-separated list of domains/subdomains:
 #   app.allowedHostnames = 'yourdomain.com,www.yourdomain.com'
 #
-# Or via environment variable (useful for Docker/Compose):
-#   ALLOWED_HOSTNAMES=yourdomain.com,www.yourdomain.com
-#
 # For local development:
 #   app.allowedHostnames = 'localhost'
 #
--- a/.github/scripts/get-version.sh
+++ b/.github/scripts/get-version.sh
@@ -1,94 +0,0 @@
-#!/bin/bash
-set -euo pipefail
-
-# Shared version tag generation script for GitHub Actions workflows
-# Usage: ./get-version.sh [FORMAT] [SHA_LENGTH]
-#
-# Formats:
-#   docker-tag  - Docker image tag (default)
-#   archive     - Archive filename suffix
-#   all         - Output all version variables for GITHUB_OUTPUT
-#
-# Environment variables:
-#   GITHUB_REF        - Git ref (e.g., refs/heads/master, refs/pull/123/merge)
-#   GITHUB_SHA        - Git commit SHA
-#   GITHUB_EVENT_NAME - Event that triggered workflow (push, pull_request, etc.)
-#   GITHUB_EVENT_PATH - Path to event JSON (for PR number extraction)
-#   GITHUB_OUTPUT     - Path to GITHUB_OUTPUT file (when format=all)
-
-# Ensure we're in a git repository with source files
-cd "${GITHUB_WORKSPACE:-.}"
-
-# Get version from App.php
-VERSION=$(grep "application_version" app/Config/App.php | sed "s/.*= '\(.*\)';/\1/g")
-
-# Standardize SHA length (default: 7 chars)
-SHA_LENGTH="${2:-7}"
-SHA="${GITHUB_SHA:0:$SHA_LENGTH}"
-
-# Initialize variables
-IMAGE_TAG=""
-BRANCH=""
-
-# Detect event type and generate appropriate tag
-if [[ "$GITHUB_EVENT_NAME" == "pull_request" || "$GITHUB_EVENT_NAME" == "pull_request_review" ]]; then
-  # Extract PR number from event JSON
-  if [[ -f "${GITHUB_EVENT_PATH:-}" ]]; then
-    PR_NUMBER=$(jq -r '.pull_request.number // .number // empty' < "$GITHUB_EVENT_PATH" 2>/dev/null || true)
-    
-    if [[ -n "$PR_NUMBER" ]]; then
-      # PR-based tag (for PR deployments)
-      IMAGE_TAG="pr-${PR_NUMBER}-${SHA}"
-      BRANCH="pr-${PR_NUMBER}"
-    fi
-  fi
-  
-  # Fallback if we couldn't extract PR number
-  if [[ -z "$IMAGE_TAG" ]]; then
-    # Try to extract from GITHUB_REF
-    PR_NUMBER=$(echo "$GITHUB_REF" | grep -oP 'pull/\K[0-9]+' || true)
-    if [[ -n "$PR_NUMBER" ]]; then
-      IMAGE_TAG="pr-${PR_NUMBER}-${SHA}"
-      BRANCH="pr-${PR_NUMBER}"
-    else
-      # Last resort: use SHA only
-      IMAGE_TAG="${SHA}"
-      BRANCH="unknown"
-    fi
-  fi
-else
-  # Branch-based tag (for push events)
-  BRANCH="${GITHUB_REF#refs/heads/}"
-  BRANCH=$(echo "$BRANCH" | sed 's/feature\///' | tr '/' '_')
-  
-  if [[ "$BRANCH" == "master" ]]; then
-    # Master builds: use version as tag
-    IMAGE_TAG="${VERSION}"
-  else
-    # Feature branch builds: version + branch + sha
-    IMAGE_TAG="${VERSION}-${BRANCH}-${SHA}"
-  fi
-fi
-
-# Output format based on first argument
-case "${1:-docker-tag}" in
-  docker-tag)
-    echo "$IMAGE_TAG"
-    ;;
-  archive)
-    echo "${VERSION}.${SHA}"
-    ;;
-  all)
-    {
-      echo "version=${VERSION}"
-      echo "version-tag=${IMAGE_TAG}"
-      echo "short-sha=${SHA}"
-      echo "branch=${BRANCH}"
-    } >> "$GITHUB_OUTPUT"
-    echo "::debug::version=${VERSION}, version-tag=${IMAGE_TAG}, short-sha=${SHA}, branch=${BRANCH}"
-    ;;
-  *)
-    echo "::error::Unknown format: $1" >&2
-    exit 1
-    ;;
-esac
--- a/.github/workflows/build-release.yml
+++ b/.github/workflows/build-release.yml
@@ -22,7 +22,6 @@ jobs:
      version: ${{ steps.version.outputs.version }}
      version-tag: ${{ steps.version.outputs.version-tag }}
      short-sha: ${{ steps.version.outputs.short-sha }}
-      branch: ${{ steps.version.outputs.branch }}

    steps:
      - name: Checkout
@@ -76,13 +75,16 @@ jobs:
      - name: Get version info
        id: version
        run: |
-          chmod +x .github/scripts/get-version.sh
-          .github/scripts/get-version.sh all 7
+          VERSION=$(grep "application_version" app/Config/App.php | sed "s/.*= '\(.*\)';/\1/g")
+          BRANCH=$(echo "${GITHUB_REF#refs/heads/}" | sed 's/feature\///' | tr '/' '_')
+          TAG=$(echo "${GITHUB_TAG:-$BRANCH}" | tr '/' '_')
+          SHORT_SHA=$(git rev-parse --short=6 HEAD)
+          echo "version=$VERSION" >> $GITHUB_OUTPUT
+          echo "version-tag=$VERSION-$BRANCH-$SHORT_SHA" >> $GITHUB_OUTPUT
+          echo "short-sha=$SHORT_SHA" >> $GITHUB_OUTPUT
+          echo "branch=$BRANCH" >> $GITHUB_OUTPUT
        env:
-          GITHUB_EVENT_PATH: ${{ github.event_path }}
-          GITHUB_SHA: ${{ github.sha }}
-          GITHUB_REF: ${{ github.ref }}
-          GITHUB_EVENT_NAME: ${{ github.event_name }}
+          GITHUB_TAG: ${{ github.ref_name }}

      - name: Create .env file
        run: |
@@ -121,14 +123,13 @@ jobs:
            .
            !.git
            !node_modules
-          include-hidden-files: true
          retention-days: 1

  docker:
    name: Build Docker Image
    runs-on: ubuntu-22.04
    needs: build
-    if: github.event_name == 'push' || (github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository)
+    if: github.event_name == 'push'

    steps:
      - name: Download build context
@@ -152,14 +153,14 @@ jobs:
      - name: Determine Docker tags
        id: tags
        run: |
-          TAG="${{ needs.build.outputs.version-tag }}"
-          if [ "${{ github.event_name }}" = "pull_request" ]; then
-            echo "tags=${{ secrets.DOCKER_USERNAME }}/opensourcepos:${TAG}" >> $GITHUB_OUTPUT
-          elif [ "${{ needs.build.outputs.branch }}" = "master" ]; then
-            echo "tags=${{ secrets.DOCKER_USERNAME }}/opensourcepos:${TAG},${{ secrets.DOCKER_USERNAME }}/opensourcepos:latest" >> $GITHUB_OUTPUT
+          BRANCH=$(echo "${GITHUB_REF#refs/heads/}" | tr '/' '_')
+          if [ "$BRANCH" = "master" ]; then
+            echo "tags=${{ secrets.DOCKER_USERNAME }}/opensourcepos:${{ needs.build.outputs.version-tag }},${{ secrets.DOCKER_USERNAME }}/opensourcepos:master" >> $GITHUB_OUTPUT
          else
-            echo "tags=${{ secrets.DOCKER_USERNAME }}/opensourcepos:${TAG}" >> $GITHUB_OUTPUT
+            echo "tags=${{ secrets.DOCKER_USERNAME }}/opensourcepos:${{ needs.build.outputs.version-tag }}" >> $GITHUB_OUTPUT
          fi
+        env:
+          GITHUB_REF: ${{ github.ref }}

      - name: Build and push Docker images
        uses: docker/build-push-action@v5
@@ -192,7 +193,7 @@ jobs:
        id: version
        run: |
          VERSION="${{ needs.build.outputs.version }}"
-          SHORT_SHA="${{ needs.build.outputs.short-sha }}"
+          SHORT_SHA=$(git rev-parse --short=6 HEAD)
          echo "version=$VERSION" >> $GITHUB_OUTPUT
          echo "short-sha=$SHORT_SHA" >> $GITHUB_OUTPUT

--- a/.github/workflows/deploy-core.yml
+++ b/.github/workflows/deploy-core.yml
@@ -1,219 +0,0 @@
-name: Deploy Core
-
-on:
-  workflow_call:
-    inputs:
-      image_tag:
-        description: 'Docker image tag to deploy'
-        type: string
-        required: true
-      sha:
-        description: 'Git commit SHA to deploy'
-        type: string
-        required: true
-      description:
-        description: 'Deployment description'
-        type: string
-        required: true
-      pr_number:
-        description: 'Pull request number (optional)'
-        type: string
-        required: false
-    outputs:
-      deployment_id:
-        description: 'GitHub deployment ID'
-        value: ${{ jobs.deploy.outputs.deployment_id }}
-      status:
-        description: 'Deployment status (success/failure)'
-        value: ${{ jobs.deploy.outputs.status }}
-
-concurrency:
-  group: deploy-staging
-  cancel-in-progress: false
-
-permissions:
-  contents: read
-  deployments: write
-
-jobs:
-  deploy:
-    name: Deploy to staging
-    runs-on: ubuntu-latest
-    
-    environment:
-      name: staging
-      url: ${{ vars.DEPLOY_URL || 'https://dev.opensourcepos.org' }}
-      deployment: false
-    
-    outputs:
-      deployment_id: ${{ steps.deployment.outputs.deployment_id }}
-      status: ${{ steps.webhook.outputs.status }}
-    
-    steps:
-      - name: Create GitHub Deployment
-        id: deployment
-        env:
-          GH_TOKEN: ${{ github.token }}
-          IMAGE_TAG: ${{ inputs.image_tag }}
-          REF_SHA: ${{ inputs.sha }}
-          DESCRIPTION: ${{ inputs.description }}
-        run: |
-          set -euo pipefail
-          
-          DEPLOYMENT_ID=$(gh api "repos/${GITHUB_REPOSITORY}/deployments" \
-            -X POST \
-            -f ref="${REF_SHA}" \
-            -f environment="staging" \
-            -f description="${DESCRIPTION}" \
-            -F auto_merge=false \
-            -F required_contexts[] \
-            --jq '.id')
-          
-          if [ -z "$DEPLOYMENT_ID" ]; then
-            echo "::error::Failed to create deployment"
-            exit 1
-          fi
-          
-          echo "deployment_id=$DEPLOYMENT_ID" >> "$GITHUB_OUTPUT"
-          echo "Created deployment: $DEPLOYMENT_ID"
-      
-      - name: Set deployment status to in_progress
-        env:
-          GH_TOKEN: ${{ github.token }}
-        run: |
-          set -euo pipefail
-          
-          gh api "repos/${GITHUB_REPOSITORY}/deployments/${{ steps.deployment.outputs.deployment_id }}/statuses" \
-            -X POST \
-            -f state="in_progress" \
-            -f description="Deployment in progress..." \
-            -f log_url="${GITHUB_SERVER_URL}/${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_ID}"
-      
-      - name: Trigger deployment webhook
-        id: webhook
-        env:
-          DEPLOY_WEBHOOK_URL: ${{ secrets.DEPLOY_WEBHOOK_URL }}
-          DEPLOY_WEBHOOK_SECRET: ${{ secrets.DEPLOY_WEBHOOK_SECRET }}
-          DOCKER_REPO_NAME: ${{ secrets.DOCKER_REPO_NAME }}
-          IMAGE_TAG: ${{ inputs.image_tag }}
-          REF_SHA: ${{ inputs.sha }}
-          DEPLOYMENT_ID: ${{ steps.deployment.outputs.deployment_id }}
-          PR_NUMBER: ${{ inputs.pr_number }}
-        run: |
-          set -euo pipefail
-          
-          if [ -z "$DEPLOY_WEBHOOK_URL" ]; then
-            echo "::error::DEPLOY_WEBHOOK_URL secret is not configured"
-            echo "Please add the DEPLOY_WEBHOOK_URL secret in your repository settings"
-            echo "status=failure" >> "$GITHUB_OUTPUT"
-            exit 1
-          fi
-          
-          REPO_NAME="${DOCKER_REPO_NAME:-opensourcepos/opensourcepos}"
-          REPO_NAMESPACE="${REPO_NAME%%/*}"
-          REPO_SHORT_NAME="${REPO_NAME#*/}"
-          PUSHED_AT=$(date +%s)
-          
-          if [ -n "$PR_NUMBER" ]; then
-            PAYLOAD=$(jq -n \
-              --arg callback_url "${GITHUB_SERVER_URL}/${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_ID}" \
-              --argjson pushed_at "$PUSHED_AT" \
-              --arg pusher "$GITHUB_ACTOR" \
-              --arg tag "$IMAGE_TAG" \
-              --arg repo_name "$REPO_NAME" \
-              --arg name "$REPO_SHORT_NAME" \
-              --arg namespace "$REPO_NAMESPACE" \
-              --arg repo_url "https://hub.docker.com/r/${REPO_NAME}/" \
-              --arg deployment_id "$DEPLOYMENT_ID" \
-              --arg repository "$GITHUB_REPOSITORY" \
-              --arg sha "$REF_SHA" \
-              --arg run_id "$GITHUB_RUN_ID" \
-              --arg actor "$GITHUB_ACTOR" \
-              --argjson pr_number "$PR_NUMBER" \
-              '{
-                callback_url: $callback_url,
-                push_data: {pushed_at: $pushed_at, pusher: $pusher, tag: $tag},
-                repository: {repo_name: $repo_name, name: $name, namespace: $namespace, repo_url: $repo_url, status: "Active"},
-                github_deployment: {id: $deployment_id, environment: "staging", repository: $repository, sha: $sha, run_id: $run_id, actor: $actor, pull_request: $pr_number}
-              }')
-          else
-            PAYLOAD=$(jq -n \
-              --arg callback_url "${GITHUB_SERVER_URL}/${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_ID}" \
-              --argjson pushed_at "$PUSHED_AT" \
-              --arg pusher "$GITHUB_ACTOR" \
-              --arg tag "$IMAGE_TAG" \
-              --arg repo_name "$REPO_NAME" \
-              --arg name "$REPO_SHORT_NAME" \
-              --arg namespace "$REPO_NAMESPACE" \
-              --arg repo_url "https://hub.docker.com/r/${REPO_NAME}/" \
-              --arg deployment_id "$DEPLOYMENT_ID" \
-              --arg repository "$GITHUB_REPOSITORY" \
-              --arg sha "$REF_SHA" \
-              --arg run_id "$GITHUB_RUN_ID" \
-              --arg actor "$GITHUB_ACTOR" \
-              '{
-                callback_url: $callback_url,
-                push_data: {pushed_at: $pushed_at, pusher: $pusher, tag: $tag},
-                repository: {repo_name: $repo_name, name: $name, namespace: $namespace, repo_url: $repo_url, status: "Active"},
-                github_deployment: {id: $deployment_id, environment: "staging", repository: $repository, sha: $sha, run_id: $run_id, actor: $actor}
-              }')
-          fi
-          
-          echo "Sending webhook..."
-          echo "Image: ${IMAGE_TAG}"
-          echo "Environment: staging"
-          
-          HEADERS=(-H "Content-Type: application/json")
-          
-          if [ -n "$DEPLOY_WEBHOOK_SECRET" ]; then
-            SIGNATURE=$(printf '%s' "$PAYLOAD" | openssl dgst -sha256 -hmac "$DEPLOY_WEBHOOK_SECRET" | sed 's/.*= //')
-            HEADERS+=(-H "X-Hub-Signature-256: sha256=$SIGNATURE")
-            echo "Using HMAC-SHA256 signature verification"
-          else
-            echo "::warning::DEPLOY_WEBHOOK_SECRET not set - webhook calls will not be signed"
-            echo "For security, configure DEPLOY_WEBHOOK_SECRET in your repository settings"
-          fi
-          
-          HTTP_CODE=$(curl -sS --connect-timeout 10 --max-time 120 \
-            -o response.txt -w "%{http_code}" \
-            -X POST \
-            "${HEADERS[@]}" \
-            -d "$PAYLOAD" \
-            "$DEPLOY_WEBHOOK_URL") || HTTP_CODE="000"
-          
-          echo "Response code: $HTTP_CODE"
-          if [ -s response.txt ]; then
-            cat response.txt
-          fi
-          
-          if [ "$HTTP_CODE" -ge 200 ] && [ "$HTTP_CODE" -lt 300 ]; then
-            echo "status=success" >> "$GITHUB_OUTPUT"
-          else
-            echo "status=failure" >> "$GITHUB_OUTPUT"
-          fi
-      
-      - name: Set deployment status
-        if: always()
-        env:
-          GH_TOKEN: ${{ github.token }}
-          IMAGE_TAG: ${{ inputs.image_tag }}
-        run: |
-          set -euo pipefail
-          
-          STATE="${{ steps.webhook.outputs.status }}"
-          
-          if [ "$STATE" = "success" ]; then
-            DESCRIPTION=$(jq -nr --arg tag "$IMAGE_TAG" \
-              '"Deployed image \($tag) to staging"')
-            
-            gh api "repos/${GITHUB_REPOSITORY}/deployments/${{ steps.deployment.outputs.deployment_id }}/statuses" \
-              -X POST \
-              -f state="success" \
-              -f description="$DESCRIPTION"
-          else
-            gh api "repos/${GITHUB_REPOSITORY}/deployments/${{ steps.deployment.outputs.deployment_id }}/statuses" \
-              -X POST \
-              -f state="failure" \
-              -f description="Deployment failed"
-            exit 1
-          fi
--- a/.github/workflows/deploy-pr.yml
+++ b/.github/workflows/deploy-pr.yml
@@ -1,82 +0,0 @@
-name: PR Deploy
-
-on:
-  pull_request_review:
-    types: [submitted]
-
-concurrency:
-  group: staging-deploy
-  cancel-in-progress: false
-
-permissions:
-  contents: read
-  deployments: write
-  pull-requests: write
-
-jobs:
-  prepare:
-    name: Prepare deployment
-    runs-on: ubuntu-latest
-    if: >
-      github.event.review.state == 'approved' &&
-      github.event.pull_request.head.repo.full_name == github.repository
-    outputs:
-      image_tag: ${{ steps.image.outputs.tag }}
-      sha: ${{ github.event.pull_request.head.sha }}
-      pr_number: ${{ github.event.pull_request.number }}
-    
-    steps:
-      - name: Checkout PR
-        uses: actions/checkout@v4
-        with:
-          ref: ${{ github.event.pull_request.head.sha }}
-      
-      - name: Get image tag
-        id: image
-        run: |
-          chmod +x .github/scripts/get-version.sh
-          IMAGE_TAG=$(.github/scripts/get-version.sh docker-tag 7)
-          echo "tag=$IMAGE_TAG" >> "$GITHUB_OUTPUT"
-        env:
-          GITHUB_EVENT_PATH: ${{ github.event_path }}
-          GITHUB_SHA: ${{ github.event.pull_request.head.sha }}
-          GITHUB_REF: ${{ github.ref }}
-          GITHUB_EVENT_NAME: ${{ github.event_name }}
-
-  deploy:
-    name: Deploy to staging
-    needs: prepare
-    uses: ./.github/workflows/deploy-core.yml
-    with:
-      image_tag: ${{ needs.prepare.outputs.image_tag }}
-      sha: ${{ needs.prepare.outputs.sha }}
-      description: Deploy PR #${{ needs.prepare.outputs.pr_number }} to staging
-      pr_number: ${{ needs.prepare.outputs.pr_number }}
-    secrets: inherit
-
-  comment:
-    name: Comment deployment status
-    needs: [prepare, deploy]
-    if: always()
-    runs-on: ubuntu-latest
-    env:
-      GH_TOKEN: ${{ github.token }}
-      IMAGE_TAG: ${{ needs.prepare.outputs.image_tag }}
-      PR_NUMBER: ${{ needs.prepare.outputs.pr_number }}
-      REF_SHA: ${{ needs.prepare.outputs.sha }}
-      STATUS: ${{ needs.deploy.outputs.status }}
-    
-    steps:
-      - name: Comment on PR
-        run: |
-          if [ "$STATUS" = "success" ]; then
-            BODY=$(jq -nr --arg tag "$IMAGE_TAG" --arg sha "$REF_SHA" --arg url "${GITHUB_SERVER_URL}/${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_ID}" \
-              '"✅ **Staging deployment completed**\n\n🔗 **URL**: https://dev.opensourcepos.org\n📦 **Image Tag**: `\($tag)`\n🔨 **Commit**: \($sha)\n\nView logs: \($url)"')
-          else
-            BODY=$(jq -nr --arg url "${GITHUB_SERVER_URL}/${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_ID}" \
-              '"❌ **Staging deployment failed**\n\nCheck the [workflow logs](\($url)) for details."')
-          fi
-          
-          gh api "repos/${GITHUB_REPOSITORY}/issues/${PR_NUMBER}/comments" \
-            -X POST \
-            -f body="$BODY"
--- a/.github/workflows/deploy.yml
+++ b/.github/workflows/deploy.yml
@@ -1,23 +0,0 @@
-name: Deploy
-
-on:
-  workflow_dispatch:
-    inputs:
-      image_tag:
-        description: 'Docker image tag to deploy (e.g., v3.4.0, latest)'
-        required: true
-        default: 'latest'
-
-permissions:
-  contents: read
-  deployments: write
-
-jobs:
-  deploy:
-    name: Deploy to staging
-    uses: ./.github/workflows/deploy-core.yml
-    with:
-      image_tag: ${{ inputs.image_tag }}
-      sha: ${{ github.sha }}
-      description: Deploy image ${{ inputs.image_tag }}
-    secrets: inherit
--- a/.github/workflows/install-script-test.yml
+++ b/.github/workflows/install-script-test.yml
@@ -1,131 +0,0 @@
-name: Install Script Test
-
-on:
-  push:
-    paths:
-      - 'scripts/install-ubuntu.sh'
-      - '.github/workflows/install-script-test.yml'
-  pull_request:
-    paths:
-      - 'scripts/install-ubuntu.sh'
-      - '.github/workflows/install-script-test.yml'
-  workflow_dispatch:
-
-concurrency:
-  group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
-  cancel-in-progress: true
-
-permissions:
-  contents: read
-
-jobs:
-  install-test:
-    name: Test Install Script (${{ matrix.scenario }})
-    runs-on: ubuntu-22.04
-    timeout-minutes: 30
-
-    strategy:
-      fail-fast: false
-      matrix:
-        include:
-          - scenario: default
-            db_pass: ''
-          - scenario: custom-password
-            db_pass: 'TestPass123!'
-
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Make install script executable
-        run: chmod +x scripts/install-ubuntu.sh
-
-      - name: Run install script
-        env:
-          DB_PASS: ${{ matrix.db_pass }}
-        run: |
-          set -o pipefail
-          echo "Running install script with scenario: ${{ matrix.scenario }}"
-          sudo -E bash scripts/install-ubuntu.sh 2>&1 | tee install-output.log
-          echo "Install completed successfully"
-
-      - name: Wait for services to stabilize
-        run: sleep 10
-
-      - name: Verify Apache is running
-        run: |
-          echo "Checking Apache status..."
-          sudo systemctl status apache2 --no-pager
-          sudo systemctl is-active apache2
-
-      - name: Verify MariaDB is running
-        run: |
-          echo "Checking MariaDB status..."
-          sudo systemctl status mariadb --no-pager
-          sudo systemctl is-active mariadb
-
-      - name: Verify Apache HTTP response
-        run: |
-          echo "Testing HTTP response on port 80..."
-          HTTP_CODE=$(curl -s -o /dev/null -w "%{http_code}" http://localhost/)
-          echo "HTTP Response Code: $HTTP_CODE"
-          if [ "$HTTP_CODE" = "200" ] || [ "$HTTP_CODE" = "302" ]; then
-            echo "Apache is responding correctly"
-          elif [ "$HTTP_CODE" = "500" ]; then
-            echo "HTTP 500 - Application error. Checking .env configuration..."
-            sudo cat /var/www/ospos/.env 2>/dev/null | grep -E "database\.default\.(hostname|database|username|password)|encryption\.key|CI_ENVIRONMENT" | head -10
-            sudo cat /var/www/ospos/writable/logs/*.log 2>/dev/null | tail -20 || true
-            curl -s http://localhost/ | head -50
-            exit 1
-          else
-            echo "Unexpected HTTP code: $HTTP_CODE"
-            exit 1
-          fi
-
-      - name: Verify OSPOS login page
-        run: |
-          echo "Checking OSPOS login page..."
-          curl -s http://localhost/ | grep -qi "login\|password\|username" && echo "Login page content found" || {
-            echo "Login page verification failed"
-            curl -s http://localhost/ | head -50
-            exit 1
-          }
-
-      - name: Verify database exists
-        env:
-          DB_PASS: ${{ matrix.db_pass != '' && matrix.db_pass || '' }}
-        run: |
-          echo "Verifying database..."
-          
-          # Extract the generated password from install output if using default
-          if [ -z "${{ matrix.db_pass }}" ]; then
-            GENERATED_PASS=$(grep -oP 'Database Password: \K[^\s]+' install-output.log || true)
-            if [ -n "$GENERATED_PASS" ]; then
-              DB_PASS="$GENERATED_PASS"
-            fi
-          fi
-          
-          # Check database exists
-          sudo mysql -u root -e "SHOW DATABASES LIKE 'ospos';" | grep -q ospos && echo "Database 'ospos' exists" || {
-            echo "Database 'ospos' not found"
-            sudo mysql -u root -e "SHOW DATABASES;"
-            exit 1
-          }
-          
-          # Check tables exist
-          TABLE_COUNT=$(sudo mysql -u root ospos -e "SHOW TABLES;" | wc -l)
-          echo "Found $TABLE_COUNT tables in database"
-          if [ "$TABLE_COUNT" -gt 5 ]; then
-            echo "Database tables verified"
-          else
-            echo "Not enough tables found"
-            exit 1
-          fi
-
-      - name: Upload install log
-        uses: actions/upload-artifact@v4
-        if: always()
-        with:
-          name: install-log-${{ matrix.scenario }}
-          path: install-output.log
-          retention-days: 7
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,4 +1,5 @@
-[unreleased]: https://github.com/opensourcepos/opensourcepos/compare/3.4.1...HEAD
+[unreleased]: https://github.com/opensourcepos/opensourcepos/compare/3.4.0...HEAD
+[3.4.2]: https://github.com/opensourcepos/opensourcepos/compare/3.4.1...3.4.2
 [3.4.1]: https://github.com/opensourcepos/opensourcepos/compare/3.4.0...3.4.1
 [3.4.0]: https://github.com/opensourcepos/opensourcepos/compare/3.3.9...3.4.0
 [3.3.9]: https://github.com/opensourcepos/opensourcepos/compare/3.3.8...3.3.9
@@ -33,36 +34,10 @@ All notable changes to this project will be documented in this file.

 ## [Unreleased]

-## [3.4.1] - 2025-06-05
- Feature: PSR-12 Compliant Indentation by @objecttothis in ([#4196](https://github.com/opensourcepos/opensourcepos/pull/4196))
- Add .env to dist zip by @jekkos in ([#4199](https://github.com/opensourcepos/opensourcepos/pull/4199))
- Add CI4 coding standards linter ([#3708](https://github.com/opensourcepos/opensourcepos/issues/3708)) by @jekkos in ([#4198](https://github.com/opensourcepos/opensourcepos/pull/4198))
- Bump canvg from 3.0.10 to 3.0.11 by @dependabot in ([#4189](https://github.com/opensourcepos/opensourcepos/pull/4189))
- Bump jspdf and jspdf-autotable by @dependabot in ([#4190](https://github.com/opensourcepos/opensourcepos/pull/4190))
- Feature bump ci to 4.6.0 by @objecttothis in ([#4197](https://github.com/opensourcepos/opensourcepos/pull/4197))
- Add Kurdish language option to UI by @BudsieBuds in ([#4210](https://github.com/opensourcepos/opensourcepos/pull/4210))
- Convert language ku to ckb by @BudsieBuds in ([#4211](https://github.com/opensourcepos/opensourcepos/pull/4211))
- Fix PHP 8.4 errors by @BudsieBuds in ([#4215](https://github.com/opensourcepos/opensourcepos/pull/4215))
- Add default bootstrap to themes by @BudsieBuds in ([#4219](https://github.com/opensourcepos/opensourcepos/pull/4219))
- Update language names by @BudsieBuds in ([#4218](https://github.com/opensourcepos/opensourcepos/pull/4218))
- Update install docs by @BudsieBuds in ([#4217](https://github.com/opensourcepos/opensourcepos/pull/4217))
- Convert menu icons to SVG by @BudsieBuds in ([#4220](https://github.com/opensourcepos/opensourcepos/pull/4220))
- Enhance license handling by @BudsieBuds in ([#4223](https://github.com/opensourcepos/opensourcepos/pull/4223))
- Fix datetime rendering ([#4226](https://github.com/opensourcepos/opensourcepos/issues/4226)) by @jekkos in ([#4227](https://github.com/opensourcepos/opensourcepos/pull/4227))
- Fix datetime rendering by @jekkos in ([#4228](https://github.com/opensourcepos/opensourcepos/pull/4228))
- Fix null error when sending by email a receipt of a sale that has no invoice by @diego-ramos in ([#4229](https://github.com/opensourcepos/opensourcepos/pull/4229))
- Update Receivings.php to save form. by @odiea in ([#4231](https://github.com/opensourcepos/opensourcepos/pull/4231))
- Update Cashups.php for ajax cashup total to work. by @odiea in ([#4238](https://github.com/opensourcepos/opensourcepos/pull/4238))
- Coding style updates for PSR-12 compliance & improved readability by @BudsieBuds in ([#4204](https://github.com/opensourcepos/opensourcepos/pull/4204))
- Fix Codeigniter disallowed characters error with payment types that have accents by @diego-ramos in ([#4232](https://github.com/opensourcepos/opensourcepos/pull/4232))
- Fixed broken escape string for success & warning messages by @Franchovy in ([#4253](https://github.com/opensourcepos/opensourcepos/pull/4253))
- Bugfix constraint migration fix by @objecttothis in ([#4230](https://github.com/opensourcepos/opensourcepos/pull/4230))
- Fix item number lookup in sales/receivings ([#4212](https://github.com/opensourcepos/opensourcepos/issues/4212)) by @jekkos in ([#4250](https://github.com/opensourcepos/opensourcepos/pull/4250))
-
-## [3.4.0] - 2025-03-23
+## [3.4.0] - 2025-02-06

 - Translation updates (Spanish, Indonesian, Swedish, Urdu, Chinese, Thai, French, Dutch)
- PHP `8.x` support
+- PHP 8.x support
 - Security fixes (XSS, SQLi)
 - Migration to Gulp as buildsystem
 - Decimal validation fix
--- a/CODE_OF_CONDUCT.md
+++ b/CODE_OF_CONDUCT.md
@@ -1,85 +1,98 @@
-[comment]: # (Contributor Covenant 2.1 - from https://www.contributor-covenant.org/version/2/1/code_of_conduct/code_of_conduct.md)
-
-# Contributor Covenant Code of Conduct
-
-## Our Pledge
-
-We as members, contributors, and leaders pledge to make participation in our community a harassment-free experience for everyone, regardless of age, body size, visible or invisible disability, ethnicity, sex characteristics, gender identity and expression, level of experience, education, socio-economic status, nationality, personal appearance, race, caste, color, religion, or sexual identity and orientation.
-
-We pledge to act and interact in ways that contribute to an open, welcoming, diverse, inclusive, and healthy community.
-
-## Our Standards
-
-Examples of behavior that contributes to a positive environment for our community include:
+Contributor Covenant Code of Conduct
+Our Pledge
+We as members, contributors, and leaders pledge to make participation in our
+community a harassment-free experience for everyone, regardless of age, body
+size, visible or invisible disability, ethnicity, sex characteristics, gender
+identity and expression, level of experience, education, socio-economic status,
+nationality, personal appearance, race, caste, color, religion, or sexual
+identity and orientation.
+We pledge to act and interact in ways that contribute to an open, welcoming,
+diverse, inclusive, and healthy community.
+Our Standards
+Examples of behavior that contributes to a positive environment for our
+community include:

 * Demonstrating empathy and kindness toward other people
 * Being respectful of differing opinions, viewpoints, and experiences
 * Giving and gracefully accepting constructive feedback
-* Accepting responsibility and apologizing to those affected by our mistakes, and learning from the experience
-* Focusing on what is best not just for us as individuals, but for the overall community
+* Accepting responsibility and apologizing to those affected by our mistakes,
+and learning from the experience
+* Focusing on what is best not just for us as individuals, but for the overall
+community

 Examples of unacceptable behavior include:

-* The use of sexualized language or imagery, and sexual attention or advances of any kind
+* The use of sexualized language or imagery, and sexual attention or advances of
+any kind
 * Trolling, insulting or derogatory comments, and personal or political attacks
 * Public or private harassment
-* Publishing others' private information, such as a physical or email address, without their explicit permission
-* Other conduct which could reasonably be considered inappropriate in a professional setting
+* Publishing others’ private information, such as a physical or email address,
+without their explicit permission
+* Other conduct which could reasonably be considered inappropriate in a
+professional setting

-## Enforcement Responsibilities
+Enforcement Responsibilities
+Community leaders are responsible for clarifying and enforcing our standards of
+acceptable behavior and will take appropriate and fair corrective action in
+response to any behavior that they deem inappropriate, threatening, offensive,
+or harmful.
+Community leaders have the right and responsibility to remove, edit, or reject
+comments, commits, code, wiki edits, issues, and other contributions that are
+not aligned to this Code of Conduct, and will communicate reasons for moderation
+decisions when appropriate.
+Scope
+This Code of Conduct applies within all community spaces, and also applies when
+an individual is officially representing the community in public spaces.
+Examples of representing our community include using an official email address,
+posting via an official social media account, or acting as an appointed
+representative at an online or offline event.
+Enforcement
+Instances of abusive, harassing, or otherwise unacceptable behavior may be
+reported to the community leaders responsible for enforcement at
+[INSERT CONTACT METHOD].
+All complaints will be reviewed and investigated promptly and fairly.
+All community leaders are obligated to respect the privacy and security of the
+reporter of any incident.
+Enforcement Guidelines
+Community leaders will follow these Community Impact Guidelines in determining
+the consequences for any action they deem in violation of this Code of Conduct:
+1. Correction
+Community Impact: Use of inappropriate language or other behavior deemed
+unprofessional or unwelcome in the community.
+Consequence: A private, written warning from community leaders, providing
+clarity around the nature of the violation and an explanation of why the
+behavior was inappropriate. A public apology may be requested.
+2. Warning
+Community Impact: A violation through a single incident or series of
+actions.
+Consequence: A warning with consequences for continued behavior. No
+interaction with the people involved, including unsolicited interaction with
+those enforcing the Code of Conduct, for a specified period of time. This
+includes avoiding interactions in community spaces as well as external channels
+like social media. Violating these terms may lead to a temporary or permanent
+ban.
+3. Temporary Ban
+Community Impact: A serious violation of community standards, including
+sustained inappropriate behavior.
+Consequence: A temporary ban from any sort of interaction or public
+communication with the community for a specified period of time. No public or
+private interaction with the people involved, including unsolicited interaction
+with those enforcing the Code of Conduct, is allowed during this period.
+Violating these terms may lead to a permanent ban.
+4. Permanent Ban
+Community Impact: Demonstrating a pattern of violation of community
+standards, including sustained inappropriate behavior, harassment of an
+individual, or aggression toward or disparagement of classes of individuals.
+Consequence: A permanent ban from any sort of public interaction within the
+community.
+Attribution
+This Code of Conduct is adapted from the Contributor Covenant,
+version 2.1, available at
+https://www.contributor-covenant.org/version/2/1/code_of_conduct.html.
+Community Impact Guidelines were inspired by
+Mozilla’s code of conduct enforcement ladder.
+For answers to common questions about this code of conduct, see the FAQ at
+https://www.contributor-covenant.org/faq. Translations are available at
+https://www.contributor-covenant.org/translations.

-Community leaders are responsible for clarifying and enforcing our standards of acceptable behavior and will take appropriate and fair corrective action in response to any behavior that they deem inappropriate, threatening, offensive, or harmful.

-Community leaders have the right and responsibility to remove, edit, or reject comments, commits, code, wiki edits, issues, and other contributions that are not aligned to this Code of Conduct, and will communicate reasons for moderation decisions when appropriate.
-
-## Scope
-
-This Code of Conduct applies within all community spaces, and also applies when an individual is officially representing the community in public spaces. Examples of representing our community include using an official e-mail address, posting via an official social media account, or acting as an appointed representative at an online or offline event.
-
-## Enforcement
-
-Instances of abusive, harassing, or otherwise unacceptable behavior may be reported to the community leaders responsible for enforcement at [INSERT CONTACT METHOD]. All complaints will be reviewed and investigated promptly and fairly.
-
-All community leaders are obligated to respect the privacy and security of the reporter of any incident.
-
-## Enforcement Guidelines
-
-Community leaders will follow these Community Impact Guidelines in determining the consequences for any action they deem in violation of this Code of Conduct:
-
-### 1. Correction
-
-**Community Impact**: Use of inappropriate language or other behavior deemed unprofessional or unwelcome in the community.
-
-**Consequence**: A private, written warning from community leaders, providing clarity around the nature of the violation and an explanation of why the behavior was inappropriate. A public apology may be requested.
-
-### 2. Warning
-
-**Community Impact**: A violation through a single incident or series of actions.
-
-**Consequence**: A warning with consequences for continued behavior. No interaction with the people involved, including unsolicited interaction with those enforcing the Code of Conduct, for a specified period of time. This includes avoiding interactions in community spaces as well as external channels like social media. Violating these terms may lead to a temporary or permanent ban.
-
-### 3. Temporary Ban
-
-**Community Impact**: A serious violation of community standards, including sustained inappropriate behavior.
-
-**Consequence**: A temporary ban from any sort of interaction or public communication with the community for a specified period of time. No public or private interaction with the people involved, including unsolicited interaction with those enforcing the Code of Conduct, is allowed during this period. Violating these terms may lead to a permanent ban.
-
-### 4. Permanent Ban
-
-**Community Impact**: Demonstrating a pattern of violation of community standards, including sustained inappropriate behavior, harassment of an individual, or aggression toward or disparagement of classes of individuals.
-
-**Consequence**: A permanent ban from any sort of public interaction within the community.
-
-## Attribution
-
-This Code of Conduct is adapted from the [Contributor Covenant][homepage], version 2.1, available at [https://www.contributor-covenant.org/version/2/1/code_of_conduct.html][v2.1].
-
-Community Impact Guidelines were inspired by [Mozilla's code of conduct enforcement ladder][Mozilla CoC].
-
-For answers to common questions about this code of conduct, see the FAQ at [https://www.contributor-covenant.org/faq][FAQ]. Translations are available at [https://www.contributor-covenant.org/translations][translations].
-
-[homepage]: https://www.contributor-covenant.org
-[v2.1]: https://www.contributor-covenant.org/version/2/1/code_of_conduct.html
-[Mozilla CoC]: https://github.com/mozilla/diversity
-[FAQ]: https://www.contributor-covenant.org/faq
-[translations]: https://www.contributor-covenant.org/translations
--- a/3
+++ b/3
@@ -13,8 +13,7 @@ RUN echo "date.timezone = \"\${PHP_TIMEZONE}\"" > /usr/local/etc/php/conf.d/time

 WORKDIR /app
 COPY --chown=www-data:www-data . /app
-RUN chmod 750 /app/writable/logs /app/writable/uploads /app/writable/cache /app/public/uploads /app/public/uploads/item_pics \
-    && chmod 640 /app/writable/uploads/importCustomers.csv \
+RUN chmod 770 /app/writable/uploads /app/writable/logs /app/writable/cache \
    && ln -s /app/*[^public] /var/www \
    && rm -rf /var/www/html \
    && ln -nsf /app/public /var/www/html
--- a/INSTALL.md
+++ b/INSTALL.md
@@ -102,73 +102,5 @@ Do **not** use below command on live deployments unless you want to tear everyth

 ## Cloud install

-### Recommended: DigitalOcean
-
-Sign up through [our referral link](https://m.do.co/c/ac38c262507b) to get a [**$100, 60-day credit**](https://m.do.co/c/ac38c262507b).
-
-1. Create an Ubuntu 20.04+ or 22.04+ droplet
-2. SSH into your server: `ssh root@<your-droplet-ip>`
-3. Run the one-line installer:
-   ```bash
-   curl -sSL https://opensourcepos.org/install | sudo bash
-   ```
-
-The installer will:
- Install Apache, MariaDB, PHP 8.2 and required extensions
- Download the **latest stable release** of OSPOS from GitHub
- Create a database with secure random password
- Configure OSPOS and Apache
- **Set up SSL/TLS certificates** (interactive prompt or environment variables)
- Display login credentials after completion
-
-**Interactive Mode (Recommended for first-time users):**
-
-When run without environment variables, the installer will prompt you:
-1. Whether to configure SSL (recommended for production)
-2. Your domain name (e.g., `pos.example.com`)
-3. Your email for Let's Encrypt (for production SSL)
-
-```bash
-curl -sSL https://opensourcepos.org/install | sudo bash
-# Script will ask:
-# - Configure SSL? (y/n)
-# - Domain name: pos.example.com
-# - Email for Let's Encrypt: admin@example.com
-```
-
-**Non-Interactive Mode (for automation):**
-
-```bash
-# Development (no SSL)
-curl -sSL https://opensourcepos.org/install | APACHE_SERVER_NAME=localhost sudo -E bash
-
-# Production with Let's Encrypt SSL
-curl -sSL https://opensourcepos.org/install | APACHE_SERVER_NAME=pos.example.com SSL_EMAIL=admin@example.com sudo -E bash
-
-# Custom database password
-curl -sSL https://opensourcepos.org/install | DB_PASS=securepassword APACHE_SERVER_NAME=pos.example.com SSL_EMAIL=admin@example.com sudo -E bash
-```
-
-**Environment variables:**
- `DB_HOST` - Database host (default: localhost)
- `DB_NAME` - Database name (default: ospos)
- `DB_USER` - Database user (default: ospos)
- `DB_PASS` - Database password (default: auto-generated)
- `MYSQL_ROOT_PASS` - MariaDB root password (default: empty/no password)
- `OSPOS_DIR` - Installation directory (default: /var/www/ospos)
- `OSPOS_VERSION` - OSPOS version to install (default: latest stable release)
- `PHP_VERSION` - PHP version (default: 8.2)
- `APACHE_SERVER_NAME` - Server hostname (default: localhost, or set interactively)
- `SSL_EMAIL` - Email for Let's Encrypt. When set, enables production SSL with auto-renewal
- `SSL_DOMAIN` - Alternative to `APACHE_SERVER_NAME` for SSL certificate domain
-
-> **Testing:** This installer is tested with each commit via our CI workflow. A fresh Ubuntu container is spawned, the script runs to completion, and basic sanity checks verify the installation. For production deployments, we recommend testing on a staging server first. If you encounter issues, please [open an issue](https://github.com/opensourcepos/opensourcepos/issues/new?template=bug_report.yml) with your server version and error output.
-
-> **Note:** If the short URL is unavailable, use the direct GitHub URL:
-> ```bash
-> curl -sSL https://raw.githubusercontent.com/opensourcepos/opensourcepos/master/scripts/install-ubuntu.sh | sudo bash
-> ```
-
-For other cloud providers or manual installation, see the [detailed installation guide](https://github.com/opensourcepos/opensourcepos/wiki/Getting-Started-installations) in the wiki.
-
-**Important:** Change the default password after first login!
+If you choose DigitalOcean:
+[Through this link](https://m.do.co/c/ac38c262507b), you will get a [**free $100, 60-day credit**](https://m.do.co/c/ac38c262507b). [Check the wiki](https://github.com/opensourcepos/opensourcepos/wiki/Getting-Started-installations) for further instructions on how to install the necessary components.
--- a/SECURITY.md
+++ b/SECURITY.md
@@ -5,9 +5,8 @@
  - [Supported Versions](#supported-versions)
  - [Security Advisories](#security-advisories)
  - [Reporting a Vulnerability](#reporting-a-vulnerability)
-  - [Disclosure Process](#disclosure-process)

-<!-- END doctoc generated TOC please keep comment here to allow update -->
+<!-- END doctoc generated TOC please keep comment here to allow auto update -->

 # Security Policy

@@ -22,116 +21,26 @@ We release patches for security vulnerabilities.

 ## Security Advisories

-For a complete list of published and draft security advisories with CVE details, see our [GitHub Security Advisories page](https://github.com/opensourcepos/opensourcepos/security/advisories).
+The following security vulnerabilities have been published:
+
+### High Severity
+
+| CVE | Vulnerability | CVSS | Published | Fixed In | Credit |
+|-----|--------------|------|-----------|----------|--------|
+| [CVE-2025-68434](https://github.com/opensourcepos/opensourcepos/security/advisories/GHSA-wjm4-hfwg-5w5r) | CSRF leading to Admin Creation | 8.8 | 2025-12-17 | 3.4.2 | @Nixon-H, @jekkos |
+| [CVE-2025-68147](https://github.com/opensourcepos/opensourcepos/security/advisories/GHSA-xgr7-7pvw-fpmh) | Stored XSS in Return Policy | 8.1 | 2025-12-17 | 3.4.2 | @Nixon-H, @jekkos |
+| [CVE-2025-66924](https://github.com/opensourcepos/opensourcepos/security/advisories/GHSA-gv8j-f6gq-g59m) | Stored XSS in Item Kits | 7.2 | 2026-03-04 | 3.4.2 | @hungnqdz, @omkaryepre |
+
+### Medium Severity
+
+| CVE | Vulnerability | CVSS | Published | Fixed In | Credit |
+|-----|--------------|------|-----------|----------|--------|
+| [CVE-2025-68658](https://github.com/opensourcepos/opensourcepos/security/advisories/GHSA-32r8-8r9r-9chw) | Stored XSS in Company Name | 4.3 | 2026-01-13 | 3.4.2 | @hungnqdz |
+
+For a complete list including draft advisories, see our [GitHub Security Advisories page](https://github.com/opensourcepos/opensourcepos/security/advisories).

 ## Reporting a Vulnerability

-**Option 1: GitHub Security Advisory (Preferred)**
+Please report (suspected) security vulnerabilities to **[jeroen@steganos.dev](mailto:jeroen@steganos.dev)**. 

-1. Create a draft security advisory directly on GitHub:
-   - Go to https://github.com/opensourcepos/opensourcepos/security/advisories
-   - Click "New draft security advisory"
-   - Fill in the vulnerability details using our [template below](#vulnerability-template)
-   - Submit as **draft** (not published)
-
-2. Notify us for triage:
-   - Send an email to **[jeroen@steganos.dev](mailto:jeroen@steganos.dev)** with:
-     - Subject: `[GHSA] Brief description of vulnerability`
-     - Link to the draft advisory
-     - Brief summary
-
-**Option 2: Email Report**
-
-Send vulnerability details to **[jeroen@steganos.dev](mailto:jeroen@steganos.dev)**.
-
-You will receive a response within 48 hours. Confirmed vulnerabilities will be patched within a few days depending on complexity.
-
-## Disclosure Process
-
-### Timeline
-
-| Step | Timeline | Action |
-|------|----------|--------|
-| 1. Report received | Day 0 | We acknowledge within 48 hours |
-| 2. Triage & confirmation | Day 1-3 | We validate the vulnerability |
-| 3. Fix development | Day 3-7 | We develop and test the fix |
-| 4. Patch release | Day 7-10 | We release a security patch |
-| 5. CVE request | Day 7-14 | We request CVE from GitHub (if applicable) |
-| 6. Advisory published | Day 14 | We publish the advisory with credit |
-| 7. Public disclosure | Day 14+ | Full disclosure after patch release |
-
-### CVE Process
-
-**We request CVE identifiers through GitHub's security advisory system.** This is the preferred and easiest method:
-
-1. After we confirm and fix the vulnerability, we'll request a CVE through GitHub
-2. GitHub coordinates with MITRE on our behalf
-3. The CVE is automatically linked to the advisory
-4. You'll be credited as the reporter in the published advisory
-
-**Already have a CVE?** If you've already obtained a CVE from another source (e.g., VulDB, CVE.MITRE.ORG), please include it in your report or advisory. We'll update our advisory to reference the existing CVE.
-
-### No Bug Bounty Program
-
-**Important:** Open Source Point of Sale does not offer a bug bounty program.
-
- All security research and vulnerability triage is done on a **voluntary basis** in our free time
- We do not offer monetary rewards for vulnerability reports
- We do credit reporters in published advisories (unless anonymity is requested)
- We greatly appreciate the security research community's efforts to help improve project security
-
-### Security Best Practices for Researchers
-
- **Do not** access, modify, or delete data that doesn't belong to you
- **Do not** perform denial of service attacks
- **Do not** publicly disclose vulnerabilities before we've had time to fix them
- **Do** provide sufficient information to reproduce the vulnerability
- **Do** allow us reasonable time to fix before public disclosure
- **Do** report through official channels (GitHub advisories or email)
-
-### Vulnerability Template
-
-When creating a draft advisory, please include:
-
-```
-## Summary
-[Brief description of the vulnerability]
-
-## Impact
- **Confidentiality:** [High/Medium/Low - what data can be exposed]
- **Integrity:** [High/Medium/Low - what can be modified]
- **Availability:** [High/Medium/Low - service disruption potential]
- **Privilege Required:** [None/Low/High - authentication level needed]
- **CVSS v3.1:** [Score] ([Vector string])
-
-## Details
-[Technical details about the vulnerability]
-
-**Affected Code:**
-```php
-// Path to affected file and vulnerable code
-```
-
-**Attack Vector:**
-[How an attacker can exploit this]
-
-## Proof of Concept
-```bash
-# Steps to reproduce
-```
-
-## Patch
-[Suggested fix or approach]
-
-## Affected Versions
- OpenSourcePOS X.Y.Z and earlier
-
-## Credit
-[Your GitHub username or preferred name]
-```
-
---
-
-**Thank you to all security researchers who have contributed to making Open Source Point of Sale more secure.** Your voluntary efforts help protect thousands of users worldwide and contribute to a safer, more trustworthy free and open-source software ecosystem. We deeply appreciate your responsible disclosure and the time you invest in improving our project.
-
-If you've reported a vulnerability and would like to discuss CVE coordination or have questions about the process, please reach out to us at [jeroen@steganos.dev](mailto:jeroen@steganos.dev).
+You will receive a response from us within 48 hours. If the issue is confirmed, we will release a patch as soon as possible depending on complexity but historically within a few days.
--- a/app/Config/App.php
+++ b/app/Config/App.php
@@ -58,9 +58,9 @@ class App extends BaseConfig
     * Allowed Hostnames in the Site URL other than the hostname in the baseURL.
     * If you want to accept multiple Hostnames, set this.
     *
-     * Or via environment variable (useful for Docker/Compose):
-     *   ALLOWED_HOSTNAMES=example.com,www.example.com
-     * 
+     * E.g.,
+     * When your site URL ($baseURL) is 'http://example.com/', and your site
+     * also accepts 'http://media.example.com/' and 'http://accounts.example.com/':
     *     ['media.example.com', 'accounts.example.com']
     *
     * @var list<string>
@@ -286,11 +286,7 @@ class App extends BaseConfig

        // Solution for CodeIgniter 4 limitation: arrays cannot be set from .env
        // See: https://github.com/codeigniter4/CodeIgniter4/issues/7311
-        // Support both: app.allowedHostnames (from .env) and ALLOWED_HOSTNAMES (from environment/Docker)
-        $envAllowedHostnames = getenv('ALLOWED_HOSTNAMES');
-        if ($envAllowedHostnames === false || trim($envAllowedHostnames) === '') {
-            $envAllowedHostnames = getenv('app.allowedHostnames');
-        }
+        $envAllowedHostnames = getenv('app.allowedHostnames');
        if ($envAllowedHostnames !== false && trim($envAllowedHostnames) !== '') {
            $this->allowedHostnames = array_values(array_filter(
                array_map('trim', explode(',', $envAllowedHostnames)),
@@ -331,7 +327,7 @@ class App extends BaseConfig
            $errorMessage =
                'Security: allowedHostnames is not configured. ' .
                'Host header injection protection is disabled. ' .
-                'Set app.allowedHostnames in your .env file or ALLOWED_HOSTNAMES environment variable. ' .
+                'Set app.allowedHostnames in your .env file. ' .
                'Example: app.allowedHostnames = "example.com,www.example.com" ' .
                'Received Host: ' . $httpHost;

--- a/app/Config/Mimes.php
+++ b/app/Config/Mimes.php
@@ -486,9 +486,10 @@ class Mimes
    /**
     * Attempts to determine the best mime type for the given file extension.
     *
-     * @return string|null The mime type found, or none if unable to determine.
+     * @param string $extension
+     * @return array|string|null The mime type found, or none if unable to determine.
     */
-    public static function guessTypeFromExtension(string $extension)
+    public static function guessTypeFromExtension(string $extension): array|string|null
    {
        $extension = trim(strtolower($extension), '. ');

@@ -506,7 +507,7 @@ class Mimes
     *
     * @return string|null The extension determined, or null if unable to match.
     */
-    public static function guessExtensionFromType(string $type, ?string $proposedExtension = null)
+    public static function guessExtensionFromType(string $type, ?string $proposedExtension = null): ?string
    {
        $type = trim(strtolower($type), '. ');

@@ -522,7 +523,7 @@ class Mimes
        }

        // Reverse check the mime type list if no extension was proposed.
-        // This search is order sensitive!
+        // This search is order-sensitive!
        foreach (static::$mimes as $ext => $types) {
            if (in_array($type, (array) $types, true)) {
                return $ext;
--- a/app/Config/OSPOS.php
+++ b/app/Config/OSPOS.php
@@ -5,6 +5,7 @@ namespace Config;
 use App\Models\Appconfig;
 use CodeIgniter\Cache\CacheInterface;
 use CodeIgniter\Config\BaseConfig;
+use CodeIgniter\Database\Exceptions\DatabaseException;

 /**
 * This class holds the configuration options stored from the database so that on launch those settings can be cached
@@ -40,16 +41,13 @@ class OSPOS extends BaseConfig
                    $this->settings[$app_config->key] = $app_config->value;
                }
                $this->cache->save('settings', encode_array($this->settings));
-            } catch (\Exception $e) {
+            } catch (DatabaseException $e) {
                // Database table doesn't exist yet (migrations haven't run)
-                // or database connection failed. Return empty settings to
-                // allow migration page to display. Catches mysqli_sql_exception
-                // which is not a subclass of DatabaseException.
+                // Return empty settings to allow migration page to display
                $this->settings = [
                    'language' => 'english',
                    'language_code' => 'en',
-                    'company' => 'Home',
-                    'barcode_type' => 'Code39'
+                    'company' => 'Home'
                ];
            }
        }
--- a/app/Config/Session.php
+++ b/app/Config/Session.php
@@ -3,6 +3,7 @@
 namespace Config;

 use CodeIgniter\Config\BaseConfig;
+use CodeIgniter\Database\Exceptions\DatabaseException;
 use CodeIgniter\Session\Handlers\BaseHandler;
 use CodeIgniter\Session\Handlers\DatabaseHandler;
 use CodeIgniter\Session\Handlers\FileHandler;
@@ -138,11 +139,7 @@ class Session extends BaseConfig
                    $this->driver = FileHandler::class;
                    $this->savePath = WRITEPATH . 'session';
                }
-            } catch (\Exception $e) {
-                // Database not available yet (e.g. fresh install before migrations).
-                // Fall back to file-based sessions so the login/migration page
-                // can still be served. Catches mysqli_sql_exception which is
-                // not a subclass of DatabaseException but is a RuntimeException.
+            } catch (DatabaseException $e) {
                $this->driver = FileHandler::class;
                $this->savePath = WRITEPATH . 'session';
            }
--- a/app/Controllers/BaseController.php
+++ b/app/Controllers/BaseController.php
@@ -28,9 +28,12 @@ abstract class BaseController extends Controller
    // protected $session;

    /**
+     * @param RequestInterface $request
+     * @param ResponseInterface $response
+     * @param LoggerInterface $logger
     * @return void
     */
-    public function initController(RequestInterface $request, ResponseInterface $response, LoggerInterface $logger)
+    public function initController(RequestInterface $request, ResponseInterface $response, LoggerInterface $logger): void
    {
        // Load here all helpers you want to be available in your controllers that extend BaseController.
        // Caution: Do not put the this below the parent::initController() call below.
--- a/app/Controllers/Config.php
+++ b/app/Controllers/Config.php
@@ -82,7 +82,7 @@ class Config extends Secure_Controller
        $npmDev = false;
        $license = [];

-        $license[$i]['title'] = 'Open Source Point of Sale ' . config('App')->application_version;
+        $license[$i]['title'] = 'Open Source Point Of Sale ' . config('App')->application_version;

        if (file_exists('license/LICENSE')) {
            $license[$i]['text'] = file_get_contents('license/LICENSE', false, null, 0, 3000);
@@ -221,7 +221,6 @@ class Config extends Secure_Controller
     */
    public function getIndex(): string
    {
-        $data['config'] = $this->config;
        $data['stock_locations'] = $this->stock_location->get_all()->getResultArray();
        $data['dinner_tables'] = $this->dinner_table->get_all()->getResultArray();
        $data['customer_rewards'] = $this->customer_rewards->get_all()->getResultArray();
@@ -232,8 +231,6 @@ class Config extends Secure_Controller
        $data['line_sequence_options'] = $this->sale_lib->get_line_sequence_options();
        $data['register_mode_options'] = $this->sale_lib->get_register_mode_options();
        $data['invoice_type_options'] = $this->sale_lib->get_invoice_type_options();
-        $data['keyboardShortcutOptions'] = $this->sale_lib->getKeyShortcutsOptions();
-        $data['keyboardShortcuts'] = $this->sale_lib->getKeyShortcuts();
        $data['rounding_options'] = rounding_mode::get_rounding_options();
        $data['tax_code_options'] = $this->tax_lib->get_tax_code_options();
        $data['tax_category_options'] = $this->tax_lib->get_tax_category_options();
@@ -401,9 +398,6 @@ class Config extends Secure_Controller

        $this->module->set_show_office_group($this->request->getPost('show_office_group') != null);

-        $this->db->transStart();
-
-        $attributeSuccess = true;
        if ($batchSaveData['category_dropdown']) {
            $definitionData['definition_name'] = 'ospos_category';
            $definitionData['definition_flags'] = 0;
@@ -411,16 +405,12 @@ class Config extends Secure_Controller
            $definitionData['definition_id'] = CATEGORY_DEFINITION_ID;
            $definitionData['deleted'] = 0;

-            $attributeSuccess = $this->attribute->saveDefinition($definitionData, CATEGORY_DEFINITION_ID);
+            $this->attribute->saveDefinition($definitionData, CATEGORY_DEFINITION_ID);
        } elseif ($batchSaveData['category_dropdown'] == NO_DEFINITION_ID) {
-            $attributeSuccess = $this->attribute->deleteDefinition(CATEGORY_DEFINITION_ID);
+            $this->attribute->deleteDefinition(CATEGORY_DEFINITION_ID);
        }

-        $success = $attributeSuccess && $this->appconfig->batch_save($batchSaveData);
-
-        $this->db->transComplete();
-
-        $success = $success && $this->db->transStatus();
+        $success = $this->appconfig->batch_save($batchSaveData);

        return $this->response->setJSON(['success' => $success, 'message' => lang('Config.saved_' . ($success ? '' : 'un') . 'successfully')]);
    }
@@ -433,35 +423,32 @@ class Config extends Secure_Controller
     */
    public function postCheckNumberLocale(): ResponseInterface
    {
-        $numberLocale = $this->request->getPost('number_locale');
-        $saveNumberLocale = $this->request->getPost('save_number_locale');
-        $postedCurrencySymbol = $this->request->getPost('currency_symbol');
-        $postedCurrencyCode = $this->request->getPost('currency_code');
+        $number_locale = $this->request->getPost('number_locale');
+        $save_number_locale = $this->request->getPost('save_number_locale');

-        $fmt = new NumberFormatter($numberLocale, NumberFormatter::CURRENCY);
-
-        // Use posted values if provided, otherwise fall back to locale defaults
-        $currencySymbol = $postedCurrencySymbol !== '' ? $postedCurrencySymbol : $fmt->getSymbol(NumberFormatter::CURRENCY_SYMBOL);
-        $currencyCode = $postedCurrencyCode !== '' ? $postedCurrencyCode : $fmt->getTextAttribute(NumberFormatter::CURRENCY_CODE);
-
-        // Update saved locale if it changed
-        if ($numberLocale !== $saveNumberLocale) {
-            $saveNumberLocale = $numberLocale;
+        $fmt = new NumberFormatter($number_locale, NumberFormatter::CURRENCY);
+        if ($number_locale != $save_number_locale) {
+            $currency_symbol = $fmt->getSymbol(NumberFormatter::CURRENCY_SYMBOL);
+            $currency_code = $fmt->getTextAttribute(NumberFormatter::CURRENCY_CODE);
+            $save_number_locale = $number_locale;
+        } else {
+            $currency_symbol = empty($this->request->getPost('currency_symbol')) ? $fmt->getSymbol(NumberFormatter::CURRENCY_SYMBOL) : $this->request->getPost('currency_symbol');
+            $currency_code = empty($this->request->getPost('currency_code')) ? $fmt->getTextAttribute(NumberFormatter::CURRENCY_CODE) : $this->request->getPost('currency_code');
        }

        if ($this->request->getPost('thousands_separator') == 'false') {
            $fmt->setTextAttribute(NumberFormatter::GROUPING_SEPARATOR_SYMBOL, '');
        }

-        $fmt->setSymbol(NumberFormatter::CURRENCY_SYMBOL, $currencySymbol);
-        $numberLocaleExample = $fmt->format(1234567890.12300);
+        $fmt->setSymbol(NumberFormatter::CURRENCY_SYMBOL, $currency_symbol);
+        $number_local_example = $fmt->format(1234567890.12300);

        return $this->response->setJSON([
-            'success'               => $numberLocaleExample != false,
-            'save_number_locale'    => $saveNumberLocale,
-            'number_locale_example' => $numberLocaleExample,
-            'currency_symbol'       => $currencySymbol,
-            'currency_code'         => $currencyCode,
+            'success'               => $number_local_example != false,
+            'save_number_locale'    => $save_number_locale,
+            'number_locale_example' => $number_local_example,
+            'currency_symbol'       => $currency_symbol,
+            'currency_code'         => $currency_code,
        ]);
    }

@@ -924,9 +911,7 @@ class Config extends Secure_Controller
    public function postSaveReceipt(): ResponseInterface
    {
        $batch_save_data = [
-            'receipt_template'              => Sale_lib::isValidReceiptTemplate($this->request->getPost('receipt_template'))
-                ? $this->request->getPost('receipt_template')
-                : 'receipt_default',
+            'receipt_template'              => $this->request->getPost('receipt_template'),
            'receipt_font_size'             => $this->request->getPost('receipt_font_size', FILTER_SANITIZE_NUMBER_INT),
            'print_delay_autoreturn'        => $this->request->getPost('print_delay_autoreturn', FILTER_SANITIZE_NUMBER_INT),
            'email_receipt_check_behaviour' => $this->request->getPost('email_receipt_check_behaviour'),
@@ -951,44 +936,6 @@ class Config extends Secure_Controller
        return $this->response->setJSON(['success' => $success, 'message' => lang('Config.saved_' . ($success ? '' : 'un') . 'successfully')]);
    }

-    /**
-     * Saves keyboard shortcut bindings.
-     *
-     * @return ResponseInterface
-     * @noinspection PhpUnused
-     */
-    public function postSaveShortcuts(): ResponseInterface
-    {
-        $allowedShortcuts = array_keys($this->sale_lib->getKeyShortcutsOptions());
-        $currentShortcuts = $this->sale_lib->getKeyShortcuts();
-        $batchSaveData = [];
-
-        foreach ($currentShortcuts as $name => $shortcut) {
-            $postedValue = trim((string)$this->request->getPost('key_' . $name));
-
-            if (!in_array($postedValue, $allowedShortcuts, true)) {
-                $postedValue = $shortcut['value'];
-            }
-
-            $batchSaveData['key_' . $name] = $postedValue;
-        }
-
-        $duplicateValues = array_filter(array_count_values($batchSaveData), static fn(int $count): bool => $count > 1);
-        if (!empty($duplicateValues)) {
-            return $this->response->setJSON([
-                'success' => false,
-                'message' => lang('Config.shortcuts_duplicate_bindings')
-            ]);
-        }
-
-        $success = $this->appconfig->batch_save($batchSaveData);
-
-        return $this->response->setJSON([
-            'success' => $success,
-            'message' => lang('Config.saved_' . ($success ? '' : 'un') . 'successfully')
-        ]);
-    }
-
    /**
     * Saves invoice configuration. Used in app/Views/configs/invoice_config.php.
     *
--- a/app/Controllers/Home.php
+++ b/app/Controllers/Home.php
@@ -43,7 +43,7 @@ class Home extends Secure_Controller
    public function getChangePassword(int $employeeId = NEW_ENTRY): ResponseInterface|string
    {
        $loggedInEmployee = $this->employee->get_logged_in_employee_info();
-        $currentPersonId = (int) $loggedInEmployee->person_id;
+        $currentPersonId = $loggedInEmployee->person_id;

        $employeeId = $employeeId === NEW_ENTRY ? $currentPersonId : $employeeId;

@@ -68,11 +68,10 @@ class Home extends Secure_Controller
    public function postSave(int $employeeId = NEW_ENTRY): ResponseInterface
    {
        $currentUser = $this->employee->get_logged_in_employee_info();
-        $currentPersonId = (int) $currentUser->person_id;

-        $employeeId = $employeeId === NEW_ENTRY ? $currentPersonId : $employeeId;
+        $employeeId = $employeeId === NEW_ENTRY ? $currentUser->person_id : $employeeId;

-        if (!$this->employee->isAdmin($currentPersonId) && $employeeId !== $currentPersonId) {
+        if (!$this->employee->isAdmin($currentUser->person_id) && $employeeId !== $currentUser->person_id) {
            return $this->response->setStatusCode(403)->setJSON([
                'success' => false,
                'message' => lang('Employees.unauthorized_modify')
--- a/app/Controllers/Items.php
+++ b/app/Controllers/Items.php
@@ -154,23 +154,8 @@ class Items extends Secure_Controller
    {
        helper('file');

-        // Security: Sanitize filename to prevent path traversal
-        // Use basename() to strip directory components and prevent '../' attacks
-        $pic_filename = basename(rawurldecode($pic_filename));
-        $file_extension = strtolower(pathinfo($pic_filename, PATHINFO_EXTENSION));
-
-        // Validate file extension against system-configured allowed image types
-        // Handle both legacy pipe-separated and current comma-separated formats
-        // Fallback to types that GD library can process for thumbnail generation
-        $allowed_types = $this->config['image_allowed_types'] ?? 'jpg,jpeg,gif,png,webp,bmp,tif,tiff';
-        $allowed_extensions = strpos($allowed_types, '|') !== false
-            ? explode('|', $allowed_types)
-            : explode(',', $allowed_types);
-
-        if (!in_array($file_extension, $allowed_extensions, true)) {
-            return $this->response->setStatusCode(400)->setBody('Invalid file type');
-        }
-
+        $pic_filename = rawurldecode($pic_filename);
+        $file_extension = pathinfo($pic_filename, PATHINFO_EXTENSION);
        $images = glob("./uploads/item_pics/$pic_filename");
        $base_path = './uploads/item_pics/' . pathinfo($pic_filename, PATHINFO_FILENAME);

@@ -497,9 +482,9 @@ class Items extends Secure_Controller
        foreach ($result as &$item) {
            if (isset($item['item_number']) && empty($item['item_number']) && $this->config['barcode_generate_if_empty']) {
                if (isset($item['item_id'])) {
-                    $save_item = ['item_number' => $item['item_number']];
-                    $this->item->save_value($save_item, $item['item_id']);
-                }
+                     $save_item = ['item_number' => $item['item_number'], 'item_id' => $item['item_id']];
+                     $this->item->saveValue($save_item);
+                 }
            }
        }
        $data['items'] = $result;
@@ -678,7 +663,12 @@ class Items extends Secure_Controller

        $employee_id = $this->employee->get_logged_in_employee_info()->person_id;

-        if ($this->item->save_value($item_data, $item_id)) {
+        // For updates, include item_id in data array
+        if ($item_id !== NEW_ENTRY) {
+            $item_data['item_id'] = $item_id;
+        }
+
+        if ($this->item->saveValue($item_data)) {
            $success = true;
            $new_item = false;

@@ -841,8 +831,8 @@ class Items extends Secure_Controller
     */
    public function getRemoveLogo($item_id): ResponseInterface
    {
-        $item_data = ['pic_filename' => null];
-        $result = $this->item->save_value($item_data, $item_id);
+        $item_data = ['pic_filename' => null, 'item_id' => $item_id];
+        $result = $this->item->saveValue($item_data);

        return $this->response->setJSON(['success' => $result]);
    }
@@ -1054,21 +1044,15 @@ class Items extends Secure_Controller
                            return $value !== null && strlen($value);
                        });

-                        if (!$isFailedRow && $this->item->save_value($itemData, $itemId)) {
-                            if (!$this->save_tax_data($row, $itemData)) {
-                                $isFailedRow = true;
-                            }
-                            if (!$this->save_inventory_quantities($row, $itemData, $allowedStockLocations, $employeeId)) {
-                                $isFailedRow = true;
-                            }
+                        if (!$isFailedRow && $this->item->saveValue($itemData)) {
+                            $this->save_tax_data($row, $itemData);
+                            $this->save_inventory_quantities($row, $itemData, $allowedStockLocations, $employeeId);
                            $csvAttributeValues = $this->extractAttributeData($row);
-                            if (!$this->attribute->saveCSVRowAttributeData($csvAttributeValues, $itemData, $attributeData)) {
-                                $isFailedRow = true;
-                            }
+                            $isFailedRow = !$this->attribute->saveCSVRowAttributeData($csvAttributeValues, $itemData, $attributeData);
                            if ($isFailedRow) {
                                $failedRow = $key + 2;
                                $failCodes[] = $failedRow;
-                                log_message('error', "CSV Item import failed on line $failedRow while saving item.");
+                                log_message('error', "CSV Item import failed on line $failedRow while saving attributes.");
                                continue;
                            }

@@ -1258,15 +1242,13 @@ class Items extends Secure_Controller
     * @param array $item_data
     * @param array $allowed_locations
     * @param int $employee_id
-     * @return bool Returns true on success, false on failure
     * @throws ReflectionException
     */
-    private function save_inventory_quantities(array $row, array $item_data, array $allowed_locations, int $employee_id): bool
+    private function save_inventory_quantities(array $row, array $item_data, array $allowed_locations, int $employee_id): void
    {
        // Quantities & Inventory Section
        $comment = lang('Items.inventory_CSV_import_quantity');
        $is_update = (bool)$row['Id'];
-        $success = true;

        foreach ($allowed_locations as $location_id => $location_name) {
            $item_quantity_data = ['item_id' => $item_data['item_id'], 'location_id' => $location_id];
@@ -1280,22 +1262,20 @@ class Items extends Secure_Controller

            if (!empty($row["location_$location_name"]) || $row["location_$location_name"] === '0') {
                $item_quantity_data['quantity'] = $row["location_$location_name"];
-                $success &= $this->item_quantity->save_value($item_quantity_data, $item_data['item_id'], $location_id);
+                $this->item_quantity->save_value($item_quantity_data, $item_data['item_id'], $location_id);

                $csv_data['trans_inventory'] = $row["location_$location_name"];
-                $success &= (bool)$this->inventory->insert($csv_data, false);
+                $this->inventory->insert($csv_data, false);
            } elseif ($is_update) {
-                continue;
+                return;
            } else {
                $item_quantity_data['quantity'] = 0;
-                $success &= $this->item_quantity->save_value($item_quantity_data, $item_data['item_id'], $location_id);
+                $this->item_quantity->save_value($item_quantity_data, $item_data['item_id'], $location_id);

                $csv_data['trans_inventory'] = 0;
-                $success &= (bool)$this->inventory->insert($csv_data, false);
+                $this->inventory->insert($csv_data, false);
            }
        }
-
-        return (bool)$success;
    }

    /**
@@ -1303,9 +1283,8 @@ class Items extends Secure_Controller
     *
     * @param array $row
     * @param array $item_data
-     * @return bool Returns true on success, false on failure
     */
-    private function save_tax_data(array $row, array $item_data): bool
+    private function save_tax_data(array $row, array $item_data): void
    {
        $items_taxes_data = [];

@@ -1317,11 +1296,9 @@ class Items extends Secure_Controller
            $items_taxes_data[] = ['name' => $row['Tax 2 Name'], 'percent' => $row['Tax 2 Percent']];
        }

-        if (!empty($items_taxes_data)) {
-            return $this->item_taxes->save_value($items_taxes_data, $item_data['item_id']);
+        if (isset($items_taxes_data)) {
+            $this->item_taxes->save_value($items_taxes_data, $item_data['item_id']);
        }
-
-        return true;
    }

    /**
@@ -1340,8 +1317,8 @@ class Items extends Secure_Controller
                $images = glob(FCPATH . "uploads/item_pics/$item->pic_filename.*");
                if (sizeof($images) > 0) {
                    $new_pic_filename = pathinfo($images[0], PATHINFO_BASENAME);
-                    $item_data = ['pic_filename' => $new_pic_filename];
-                    $this->item->save_value($item_data, $item->item_id);
+                    $item_data = ['pic_filename' => $new_pic_filename, 'item_id' => $item->item_id];
+                    $this->item->saveValue($item_data);
                }
            }
        }
--- a/app/Controllers/Reports.php
+++ b/app/Controllers/Reports.php
@@ -1246,15 +1246,13 @@ class Reports extends Secure_Controller
    public function get_payment_type(): array
    {
        return [
-            'all'           => lang('Common.none_selected_text'),
-            'cash'          => lang('Sales.cash'),
-            'due'           => lang('Sales.due'),
-            'check'         => lang('Sales.check'),
-            'credit'        => lang('Sales.credit'),
-            'debit'         => lang('Sales.debit'),
-            'bank_transfer' => lang('Sales.bank_transfer'),
-            'wallet'        => lang('Sales.wallet'),
-            'invoices'      => lang('Sales.invoice')
+            'all'      => lang('Common.none_selected_text'),
+            'cash'     => lang('Sales.cash'),
+            'due'      => lang('Sales.due'),
+            'check'    => lang('Sales.check'),
+            'credit'   => lang('Sales.credit'),
+            'debit'    => lang('Sales.debit'),
+            'invoices' => lang('Sales.invoice')
        ];
    }

--- a/app/Controllers/Sales.php
+++ b/app/Controllers/Sales.php
@@ -93,8 +93,6 @@ class Sales extends Secure_Controller
                'only_check'        => lang('Sales.check_filter'),
                'only_creditcard'   => lang('Sales.credit_filter'),
                'only_debit'        => lang('Sales.debit'),
-                'only_bank_transfer'=> lang('Sales.bank_transfer'),
-                'only_wallet'       => lang('Sales.wallet'),
                'only_invoices'     => lang('Sales.invoice_filter'),
                'selected_customer' => lang('Sales.selected_customer')
            ];
@@ -158,8 +156,6 @@ class Sales extends Secure_Controller
            'selected_customer' => false,
            'only_creditcard'   => false,
            'only_debit'        => false,
-            'only_bank_transfer'=> false,
-            'only_wallet'       => false,
            'only_invoices'     => $this->config['invoice_enable'] && $this->request->getGet('only_invoices', FILTER_SANITIZE_NUMBER_INT),
            'is_valid_receipt'  => $this->sale->is_valid_receipt($search)
        ];
@@ -908,14 +904,6 @@ class Sales extends Secure_Controller
                return $this->_reload($data);
            } else {
                $data['barcode'] = $this->barcode_lib->generate_receipt_barcode($data['sale_id']);
-
-                // Validate receipt template to prevent path traversal
-                $receipt_template = $this->config['receipt_template'] ?? '';
-                if (!Sale_lib::isValidReceiptTemplate($receipt_template)) {
-                    $receipt_template = 'receipt_default';
-                }
-                $data['receipt_template_view'] = $receipt_template;
-
                $this->sale_lib->clear_all();
                return view('sales/receipt', $data);
            }
@@ -949,10 +937,7 @@ class Sales extends Secure_Controller
                new Token_customer((array)$sale_data)
            ];
            $text = $this->token_lib->render($text, $tokens);
-            $sale_data['mimetype'] = $this->email_lib->getLogoMimeType();
-
-            // Build img_tag for email views that need it (receipt_email.php)
-            $sale_data['img_tag'] = $this->email_lib->buildLogoImgTag();
+            $sale_data['mimetype'] = mime_content_type(FCPATH . 'uploads/' . $this->config['company_logo']);

            // Generate email attachment: invoice in PDF format
            $view = Services::renderer();
@@ -989,7 +974,13 @@ class Sales extends Secure_Controller

        if (!empty($sale_data['customer_email'])) {
            $sale_data['barcode'] = $this->barcode_lib->generate_receipt_barcode($sale_data['sale_id']);
-            $sale_data['img_tag'] = $this->email_lib->buildLogoImgTag();
+            $sale_data['img_tag'] = '';
+
+            $logo_path = FCPATH . 'uploads/' . $this->config['company_logo'];
+            if (!empty($this->config['company_logo']) && file_exists($logo_path)) {
+                $logo_data = base64_encode(file_get_contents($logo_path));
+                $sale_data['img_tag'] = '<img id="image" src="data:image/png;base64,' . $logo_data . '" alt="company_logo">';
+            }

            $to = $sale_data['customer_email'];
            $subject = lang('Sales.receipt');
@@ -1171,13 +1162,6 @@ class Sales extends Secure_Controller
        }
        $data['invoice_view'] = $invoice_type;

-        // Validate receipt template to prevent path traversal
-        $receipt_template = $this->config['receipt_template'] ?? '';
-        if (!Sale_lib::isValidReceiptTemplate($receipt_template)) {
-            $receipt_template = 'receipt_default';
-        }
-        $data['receipt_template_view'] = $receipt_template;
-
        return $data;
    }

@@ -1272,7 +1256,6 @@ class Sales extends Secure_Controller

        $data['quote_number'] = $this->sale_lib->get_quote_number();
        $data['work_order_number'] = $this->sale_lib->get_work_order_number();
-        $data['keyboardShortcuts'] = $this->sale_lib->getKeyShortcuts();

        // TODO: the if/else set below should be converted to a switch
        if ($this->sale_lib->get_mode() == 'sale_invoice') {    // TODO: Duplicated code.
@@ -1661,9 +1644,7 @@ class Sales extends Secure_Controller
     */
    public function getSalesKeyboardHelp(): string
    {
-        return view('sales/help', [
-            'keyboardShortcuts' => $this->sale_lib->getKeyShortcuts()
-        ]);
+        return view('sales/help');
    }

    /**
--- a/app/Database/Dockerfile
+++ b/app/Database/Dockerfile
@@ -1,5 +1,5 @@
 FROM alpine:3.14
-LABEL maintainer="jekkos"
+MAINTAINER jekkos

 ADD database.sql /docker-entrypoint-initdb.d/database.sql
 VOLUME /docker-entrypoint-initdb.d
--- a/app/Database/Migrations/20260506000000_AddShortcutKeys.php
+++ b/app/Database/Migrations/20260506000000_AddShortcutKeys.php
@@ -1,46 +0,0 @@
-<?php
-
-namespace App\Database\Migrations;
-
-use CodeIgniter\Database\Migration;
-
-class AddShortcutKeys extends Migration
-{
-    public function up(): void
-    {
-        $shortcutValues = [
-            ['key' => 'key_cancel', 'value' => '27 | ESC'],
-            ['key' => 'key_items', 'value' => '49 | ALT + 1'],
-            ['key' => 'key_customers', 'value' => '50 | ALT + 2'],
-            ['key' => 'key_suspend', 'value' => '51 | ALT + 3'],
-            ['key' => 'key_suspended', 'value' => '52 | ALT + 4'],
-            ['key' => 'key_amount', 'value' => '53 | ALT + 5'],
-            ['key' => 'key_payment', 'value' => '54 | ALT + 6'],
-            ['key' => 'key_complete', 'value' => '55 | ALT + 7'],
-            ['key' => 'key_finish', 'value' => '56 | ALT + 8'],
-            ['key' => 'key_help', 'value' => '57 | ALT + 9'],
-        ];
-
-        $this->db->table('app_config')->ignore(true)->insertBatch($shortcutValues);
-    }
-
-    public function down(): void
-    {
-        $shortcutKeys = [
-            'key_cancel',
-            'key_items',
-            'key_customers',
-            'key_suspend',
-            'key_suspended',
-            'key_amount',
-            'key_payment',
-            'key_complete',
-            'key_finish',
-            'key_help',
-        ];
-
-        $this->db->table('app_config')
-            ->whereIn('key', $shortcutKeys)
-            ->delete();
-    }
-}
--- a/app/Helpers/locale_helper.php
+++ b/app/Helpers/locale_helper.php
@@ -22,7 +22,7 @@ function current_language_code(bool $load_system_language = false): string
        }
    }

-    return $config['language_code'] ?? DEFAULT_LANGUAGE_CODE;
+    return $config->language_code ?? DEFAULT_LANGUAGE_CODE;
 }

 /**
@@ -43,7 +43,7 @@ function current_language(bool $load_system_language = false): string
        }
    }

-    return $config['language'] ?? DEFAULT_LANGUAGE;
+    return $config->language ?? DEFAULT_LANGUAGE_CODE;
 }

 /**
@@ -272,9 +272,6 @@ function get_payment_options(): array
        $payments[lang('Sales.upi')] = lang('Sales.upi');
    }

-    $payments[lang('Sales.bank_transfer')] = lang('Sales.bank_transfer');
-    $payments[lang('Sales.wallet')]        = lang('Sales.wallet');
-
    return $payments;
 }

--- a/app/Language/en-GB/Sales.php
+++ b/app/Language/en-GB/Sales.php
@@ -9,7 +9,6 @@ return [
    "amount_due"                       => "Amount Due",
    "amount_tendered"                  => "Amount Tendered",
    "authorized_signature"             => "Authorised Signature",
-    "bank_transfer"                    => "Bank Transfer",
    "cancel_sale"                      => "Cancel",
    "cash"                             => "Cash",
    "cash_1"                           => "",
@@ -224,7 +223,6 @@ return [
    "update"                           => "Update",
    "upi"                              => "UPI",
    "visa"                             => "",
-    "wallet"                           => "Wallet",
    "wholesale"                        => "",
    "work_order"                       => "Work Order",
    "work_order_number"                => "Work Order Number",
--- a/app/Language/en/Config.php
+++ b/app/Language/en/Config.php
@@ -302,10 +302,6 @@ return [
    "suggestions_layout"                        => "Search Suggestions Layout",
    "suggestions_second_column"                 => "Column 2",
    "suggestions_third_column"                  => "Column 3",
-    "shortcuts"                                 => "Shortcuts",
-    "shortcuts_configuration"                   => "Sales Keyboard Shortcut Configuration",
-    "shortcuts_duplicate_bindings"              => "Shortcut bindings must be unique.",
-    "shortcuts_save_error"                       => "Unable to save shortcut settings.",
    "system_conf"                               => "Setup & Conf",
    "system_info"                               => "System Info",
    "table"                                     => "Table",
--- a/app/Language/en/Sales.php
+++ b/app/Language/en/Sales.php
@@ -9,7 +9,6 @@ return [
    "amount_due"                       => "Amount Due",
    "amount_tendered"                  => "Amount Tendered",
    "authorized_signature"             => "Authorized Signature",
-    "bank_transfer"                    => "Bank Transfer",
    "cancel_sale"                      => "Cancel",
    "cash"                             => "Cash",
    "cash_1"                           => "",
@@ -224,7 +223,6 @@ return [
    "update"                           => "Update",
    "upi"                              => "UPI",
    "visa"                             => "",
-    "wallet"                           => "Wallet",
    "wholesale"                        => "",
    "work_order"                       => "Work Order",
    "work_order_number"                => "Work Order Number",
--- a/app/Language/es-ES/Items.php
+++ b/app/Language/es-ES/Items.php
@@ -26,7 +26,7 @@ return [
    "cost_price_required"                => "Precio al Por Mayor es un campo requerido.",
    "count"                              => "Actualizar Inventario",
    "csv_import_failed"                  => "Falló la importación de Hoja de Cálculo",
-    "csv_import_invalid_location"        => "Se encontraron ubicaciones de stock no válidas: {0}. Solo se permiten ubicaciones de stock válidas.",
+    "csv_import_invalid_location"        => "Ubicación(es) de stock inválida(s) encontrada(s): {0}. Solo ubicaciones de stock válidas son permitidas.",
    "csv_import_nodata_wrongformat"      => "El archivo subido no tiene datos o el formato es incorrecto.",
    "csv_import_partially_failed"        => "Hubo {0} falla(s) en la importación de producto(s) en la(s) línea(s): {1}. Ninguna fila ha sido importada.",
    "csv_import_success"                 => "Se importaron los articulos exitosamente.",
--- a/app/Language/es-ES/Sales.php
+++ b/app/Language/es-ES/Sales.php
@@ -9,7 +9,6 @@ return [
    "amount_due"                       => "Monto Adeudado",
    "amount_tendered"                  => "Cantidad Recibida",
    "authorized_signature"             => "Firma Autorizada",
-    "bank_transfer"                    => "Transferencia Bancaria",
    "cancel_sale"                      => "Cancelar Venta",
    "cash"                             => "Efectivo",
    "cash_1"                           => "1",
@@ -223,7 +222,6 @@ return [
    "update"                           => "Editar",
    "upi"                              => "PIN UPI",
    "visa"                             => "Tarjeta Visa",
-    "wallet"                           => "Monedero",
    "wholesale"                        => "Precio al por mayor",
    "work_order"                       => "Orden trabajo",
    "work_order_number"                => "Numero Orden Trabajo",
--- a/app/Language/es-MX/Sales.php
+++ b/app/Language/es-MX/Sales.php
@@ -9,7 +9,6 @@ return [
    "amount_due"                       => "Monto de adeudo",
    "amount_tendered"                  => "Cantidad Recibida",
    "authorized_signature"             => "Firma Autorizada",
-    "bank_transfer"                    => "Transferencia Bancaria",
    "cancel_sale"                      => "Cancelar",
    "cash"                             => "Efectivo",
    "cash_1"                           => "",
@@ -223,7 +222,6 @@ return [
    "update"                           => "Actualizar",
    "upi"                              => "UPI",
    "visa"                             => "",
-    "wallet"                           => "Monedero",
    "wholesale"                        => "",
    "work_order"                       => "Orden de trabajo",
    "work_order_number"                => "Número de orden de trabajo",
--- a/app/Language/fr/Sales.php
+++ b/app/Language/fr/Sales.php
@@ -9,7 +9,6 @@ return [
    "amount_due"                       => "Montant à Payer",
    "amount_tendered"                  => "Montant Présenté",
    "authorized_signature"             => "Signature autorisée",
-    "bank_transfer"                    => "Virement Bancaire",
    "cancel_sale"                      => "Annuler la Vente",
    "cash"                             => "Espèce",
    "cash_1"                           => "",
@@ -223,7 +222,6 @@ return [
    "update"                           => "Éditer",
    "upi"                              => "UPI",
    "visa"                             => "",
-    "wallet"                           => "Portefeuille",
    "wholesale"                        => "",
    "work_order"                       => "Commande de travail",
    "work_order_number"                => "Numéro de commande",
--- a/app/Language/ka/Calendar.php
+++ b/app/Language/ka/Calendar.php
@@ -38,7 +38,7 @@ return [
    "february"  => "",
    "march"     => "",
    "april"     => "",
-    "may"       => "",
+    "mayl"      => "",
    "june"      => "",
    "july"      => "",
    "august"    => "",
@@ -46,4 +46,4 @@ return [
    "october"   => "",
    "november"  => "",
    "december"  => "",
-];
+];
--- a/app/Language/lo/Calendar.php
+++ b/app/Language/lo/Calendar.php
@@ -38,7 +38,7 @@ return [
    "february"  => "",
    "march"     => "",
    "april"     => "",
-    "may"       => "",
+    "mayl"      => "",
    "june"      => "",
    "july"      => "",
    "august"    => "",
@@ -46,4 +46,4 @@ return [
    "october"   => "",
    "november"  => "",
    "december"  => "",
-];
+];
--- a/app/Language/ml/Calendar.php
+++ b/app/Language/ml/Calendar.php
@@ -38,7 +38,7 @@ return [
    "february"  => "ഫെബ്രുവരി",
    "march"     => "മാർച്ച്",
    "april"     => "ഏപ്രിൽ",
-    "may"       => "മേയ്",
+    "mayl"      => "മേയ്",
    "june"      => "ജൂൺ",
    "july"      => "ജൂലൈ",
    "august"    => "ആഗസ്റ്റ്",
@@ -46,4 +46,4 @@ return [
    "october"   => "ഒക്ടോബർ",
    "november"  => "നവംബർ",
    "december"  => "ഡിസംബർ",
-];
+];
--- a/app/Language/nb/Calendar.php
+++ b/app/Language/nb/Calendar.php
@@ -38,7 +38,7 @@ return [
    "february"  => "Februar",
    "march"     => "Mars",
    "april"     => "April",
-    "may"       => "Mai",
+    "mayl"      => "Mai",
    "june"      => "Juni",
    "july"      => "Juli",
    "august"    => "August",
@@ -46,4 +46,4 @@ return [
    "october"   => "Oktober",
    "november"  => "November",
    "december"  => "Desember",
-];
+];
--- a/app/Language/th/Bootstrap_tables.php
+++ b/app/Language/th/Bootstrap_tables.php
@@ -1,12 +1,12 @@
 <?php

 return [
-    'all' => "ทั้งหมด",
-    'columns' => "คอลัมน์",
-    'hide_show_pagination' => "ซ่อน/แสดง รายการหน้า",
-    'loading' => "กำลังดำเนินการ รอสักครู่ ...",
-    'page_from_to' => "แสดง {0} ถึง {1} จาก {2} รายการ",
-    'refresh' => "Refresh ข้อมูล",
-    'rows_per_page' => "{0} รายการ/หน้า",
-    'toggle' => "ซ่อน/แสดง",
+    "all"                  => "ทั้งหมด",
+    "columns"              => "คอลัมน์",
+    "hide_show_pagination" => "ซ่อน/แสดง รายการหน้า",
+    "loading"              => "กำลังดำเนินการ รอสักครู่",
+    "page_from_to"         => "แสดง {0} ถึง {1} จาก {2} รายการ",
+    "refresh"              => "Refresh ข้อมูล",
+    "rows_per_page"        => "{0} รายการ/หน้า",
+    "toggle"               => "ซ่อน/แสดง",
 ];
--- a/app/Language/th/Login.php
+++ b/app/Language/th/Login.php
@@ -9,9 +9,7 @@ return [
    "login"                         => "ลงชื่อเข้าใช้",
    "logout"                        => "ออกจากระบบ",
    "migration_needed"              => "การย้ายฐานข้อมูลไปยัง {0} จะเริ่มต้นหลังจากเข้าสู่ระบบ",
-    "migration_required" => "จําเป็นต้องมีการปรับปรุงฐานข้อมูล",
-    "migration_auth_message" => "ผู้ดูแลระบบจำเป็นต้องมีสิทธิ์ในการปรับปรุงฐานข้อมูลเวอร์ชั่น {0} กรุณาเข้าระบบเพื่อดำเนินการต่อ",
-    "migration_complete_redirect" => "ทำการปรับปรุงฐานข้อมูลเรียบร้อย กำลังดำเนินการไปหน้าเข้าสู่ระบบ ...",
+    "migration_required"            => "",
    "migration_auth_message"        => "",
    "migration_initializing"        => "",
    "migration_running"             => "",
@@ -19,6 +17,7 @@ return [
    "migration_complete_login"      => "",
    "migration_failed"              => "",
    "migration_error_connection"    => "",
+    "migration_complete_redirect"   => "",
    "password"                      => "รหัสผ่าน",
    "required_username"             => "จำเป็นต้องระบุชื่อผู้ใช้งาน",
    "username"                      => "ชื่อผู้ใช้",
--- a/app/Language/th/Sales.php
+++ b/app/Language/th/Sales.php
@@ -1,232 +1,232 @@
 <?php

 return [
-    'customers_available_points' => "คะแนนที่มี",
-    'rewards_package' => "คะแนนสะสม",
-    'rewards_remaining_balance' => "คะแนนสะสมคงเหลือ ",
-    'account_number' => "บัญชี #",
-    'add_payment' => "เพิ่มบิล",
-    'amount_due' => "ยอดค้างชำระ",
-    'amount_tendered' => "ชำระเข้ามา",
-    'authorized_signature' => "ลายเซ็นผู้มีอำนาจ",
-    'cancel_sale' => "ยกเลิกการขาย",
-    'cash' => "เงินสด",
-    'cash_1' => "",
-    'cash_2' => "",
-    'cash_3' => "",
-    'cash_4' => "",
-    'cash_adjustment' => "การปรับเงินสดขาย",
-    'cash_deposit' => "ฝากเงินสด",
-    'cash_filter' => "เงินสด",
-    'change_due' => "เงินทอน",
-    'change_price' => "เปลี่ยนราคาขาย",
-    'check' => "โอนเงิน/พร้อมเพย์/เช็ค",
-    'check_balance' => "เช็คยอดคงเหลือ",
-    'check_filter' => "ตรวจสอบ",
-    'close' => "",
-    'comment' => "หมายเหตุ",
-    'comments' => "หมายเหตุ",
-    'company_name' => "",
-    'complete' => "",
-    'complete_sale' => "จบการขาย",
-    'confirm_cancel_sale' => "แน่ใจหรือไม่ที่จะล้างการขายนี้? ทุกรายการจะถูกลบทั้งหมด",
-    'confirm_delete' => "โปรดยืนยันการลบรายการขายที่เลือกไว้ ?",
-    'confirm_restore' => "คุณแน่ใจหรือไม่ว่าต้องการยกเลิกการขายที่เลือกไว้?",
-    'credit' => "เครดิตการ์ด",
-    'credit_deposit' => "เงินฝากเครดิต",
-    'credit_filter' => "บัตรเครติด",
-    'current_table' => "",
-    'customer' => "ลูกค้า",
-    'customer_address' => "Customer Address",
-    'customer_discount' => "ส่วนลด",
-    'customer_email' => "Customer Email",
-    'customer_location' => "Customer Location",
-    'customer_mailchimp_status' => "สถานะของระบบส่งเมล์เมล์ชิม",
-    'customer_optional' => "(ต้องระบุวันที่ชำระเงิน)",
-    'customer_required' => "(ต้องระบุ)",
-    'customer_total' => "Total",
-    'customer_total_spent' => "",
-    'daily_sales' => "",
-    'date' => "วันที่ขาย",
-    'date_range' => "ระหว่างวันที่",
-    'date_required' => "กรุณากรอกวันที่ให้ถูกต้อง",
-    'date_type' => "กรุณากรอกข้อมูลในช่องวันที่",
-    'debit' => "บัตรประชารัฐ/เดบิตการ์ด",
-    'debit_filter' => "",
-    'delete' => "อนุญาตให้ลบ",
-    'delete_confirmation' => "แน่ใจหรือไม่ที่จะลบรายการขายนี้, ลบแล้วไม่สามารถเรียกกลับคืนใด้",
-    'delete_entire_sale' => "ลบการขายทั้งหมด",
-    'delete_successful' => "คุณลบการขายสำเร็จ",
-    'delete_unsuccessful' => "คุณลบการขายไม่สำเร็จ",
-    'description_abbrv' => "รายละเอียด",
-    'discard' => "ยกเลิก",
-    'discard_quote' => "",
-    'discount' => "ส่วนลด %",
-    'discount_included' => "% ส่วนลด",
-    'discount_short' => "%",
-    'due' => "วันครบกำหนด",
-    'due_filter' => "วันที่ครบกำหนด",
-    'edit' => "แก้ไข",
-    'edit_item' => "แก้ไขสินค้า",
-    'edit_sale' => "แก้ไขการขาย",
-    'email_receipt' => "อีเมลบิล",
-    'employee' => "พนักงาน",
-    'entry' => "การนำเข้า",
-    'error_editing_item' => "แก้ไขสินค้าล้มเหลว",
-    'negative_price_invalid' => "ราคาไม่สามารถเป็นค่าติดลบได้",
-    'negative_quantity_invalid' => "จำนวนไม่สามารถเป็นค่าติดลบได้",
-    'negative_discount_invalid' => "ส่วนลดไม่สามารถเป็นค่าติดลบได้",
-    'discount_percent_exceeds_100' => "ส่วนลดเปอร์เซ็นต์มีค่าได้ไม่เกิน 100%",
-    'discount_exceeds_item_total' => "ส่วนลดต้องไม่เกินจำนวนรายการขายทั้งหมด",
-    'negative_total_invalid' => "",
-    'find_or_scan_item' => "ค้นหาสินค้า",
-    'find_or_scan_item_or_receipt' => "ค้นหา หรือ แสกนรายการ หรือ ใบเสร็จ",
-    'giftcard' => "บัตรของขวัญ",
-    'giftcard_balance' => "ยอดคงเหลือบัตรของขวัญ",
-    'giftcard_filter' => "",
-    'giftcard_number' => "เลขที่บัตรของขวัญ",
-    'group_by_category' => "กลุ่มตามหมวดหมู่",
-    'group_by_type' => "กลุ่มตามประเภท",
-    'hsn' => "HSN",
-    'id' => "เลขที่ขาย",
-    'include_prices' => "รวมในราคา?",
-    'invoice' => "ใบแจ้งหนี้",
-    'invoice_confirm' => "ใบแจ้งหนี้นี้จะถูกส่งไปที่",
-    'invoice_enable' => "เลขที่ใบแจ้งหนี้",
-    'invoice_filter' => "ใบแจ้งหนี้",
-    'invoice_no_email' => "ลูกค้ารายนี้ไม่มีที่อยู่อีเมล",
-    'invoice_number' => "เลขใบแจ้งหนี้ #",
-    'invoice_number_duplicate' => "ใบแจ้งหนี้หมายเลข {0} จะต้องไม่ซ้ำกัน",
-    'invoice_sent' => "ส่งใบแจ้งหนี้ไปที่",
-    'invoice_total' => "ยอดรวมในใบแจ้งหนี้",
-    'invoice_type_custom_invoice' => "ใบแจ้งหนี้ที่กำหนดเอง (custom_invoice.php)",
-    'invoice_type_custom_tax_invoice' => "ใบกำกับภาษีที่กำหนดเอง (custom_tax_invoice.php)",
-    'invoice_type_invoice' => "ใบแจ้งหนี้ (invoice.php)",
-    'invoice_type_tax_invoice' => "ใบกำกับภาษี (tax_invoice.php)",
-    'invoice_unsent' => "ไม่สามารถส่งใบแจ้งหนี้ถึง",
-    'invoice_update' => "คำนวณใหม่",
-    'item_insufficient_of_stock' => "จำนวนสินค้าไม่เพียงพอ",
-    'item_name' => "ชื่อสินค้า",
-    'item_number' => "สินค้า #",
-    'item_out_of_stock' => "สินค้าจำหน่ายหมด",
-    'key_browser' => "ความช่วยเหลือ",
-    'key_cancel' => "ยกเลิกใบเสนอราคา/ใบแจ้งหนี้ /ใบการขาย นี้",
-    'key_customer_search' => "ค้นหาลูกค้า",
-    'key_finish_quote' => "จบใบเสนอราคา/ใบแจ้งหนี้โดยไม่ต้องชำระเงิน",
-    'key_finish_sale' => "เพิ่มการชำระเงินและใบแจ้งหนี้ /ใบรายการขาย",
-    'key_full' => "เปิดแบบเต็มหน้าจอ",
-    'key_function' => "ฟังก์ชั่น",
-    'key_help' => "คำสั่งลัดงานขาย",
-    'key_help_modal' => "เปิดหน้าต่างคำสั่งลัดงานขาย",
-    'key_in' => "ขยายเข้า",
-    'key_item_search' => "ค้นหารายการขาย",
-    'key_out' => "ขยายออก",
-    'key_payment' => "เพิ่มการชำระเงิน",
-    'key_print' => "พิมพ์หน้านี้",
-    'key_restore' => "คืนการแสดงผลแบบดั้งเดิม/ขยาย",
-    'key_search' => "ค้นหาตารางรายงาน",
-    'key_suspend' => "พักรายการขายปัจจุบัน",
-    'key_suspended' => "แสดงรายการขายที่พักไว้",
-    'key_system' => "ทางลัดระบบ",
-    'key_tendered' => "แก้ไขจำนวนเงินรับมา",
-    'key_title' => "ทางลัดคียบอร์ดงานขาย",
-    'mc' => "",
-    'mode' => "รูปแบบการลงทะเบียน",
-    'must_enter_numeric' => "จำนวนที่ถุกประมูลต้องใส่ข้อมุลที่เปนตัวเลข",
-    'must_enter_numeric_giftcard' => "เลขที่บัตรของขวัญ ต้องใส่ตัวเลขเท่านั้น",
-    'new_customer' => "ลูกค้าใหม่",
-    'new_item' => "สินค้าใหม่",
-    'no_description' => "ไม่ระบุรายละเอียด",
-    'no_filter' => "ทั้งหมด",
-    'no_items_in_cart' => "ไม่พบสินค้าในตระกร้า",
-    'no_sales_to_display' => "ไม่มีการขายที่จะแสดง",
-    'none_selected' => "คุณยังไม่ได้เลือกการขายที่จะลบ",
-    'nontaxed_ind' => " . ",
-    'not_authorized' => "การกระทำนี้ไม่ได้รับอนุญาต",
-    'one_or_multiple' => "การขาย",
-    'payment' => "รูปแบบชำระเงิน",
-    'payment_amount' => "จำนวน",
-    'payment_not_cover_total' => "จำนวนเงินที่ชำระต้องมากกว่าหรือเท่ากับยอดรวม",
-    'payment_type' => "ชำระโดย",
-    'payments' => "",
-    'payments_total' => "ยอดชำระแล้ว",
-    'price' => "ราคา",
-    'print_after_sale' => "พิมพ์บิลหลังการขาย",
-    'quantity' => "จำนวน",
-    'quantity_less_than_reorder_level' => "คำเตือน ถ้าจำนวนของไม่เพียงพอกับความต้องการหรือไม่ตรงกับยอดในบันชี ก็สามารถทำการขายได้ แต่ต้องเชคปริมานสินค้าคงคลัง",
-    'quantity_less_than_zero' => "คำเตือน: ถ้าจำนวนของไม่เพียงพอกับความต้องการหรือไม่ตรงกับยอดในบัญชี ก็สามารถทำการขายได้ แต่ต้องตรวจสอบปริมาญสินค้าคงคลังก่อน",
-    'quantity_of_items' => "ปริมาณของ {0} รายการ",
-    'quote' => "ใบเสนอราคา",
-    'quote_number' => "หมายเลขอ้างอิง",
-    'quote_number_duplicate' => "หมายเลขอ้างอิงต้องไม่ซ้ำกัน",
-    'quote_sent' => "ส่งการอ้างอิงถึง",
-    'quote_unsent' => "ส่งการอ้างอิงถึงผิดพลาด",
-    'receipt' => "บิลขาย",
-    'receipt_no_email' => "ลูกค้านี้ไม่มีที่อยู่อีเมล์",
-    'receipt_number' => "จุดขาย#",
-    'receipt_sent' => "ส่งใบเสร็จไปที่",
-    'receipt_unsent' => "ไม่สามารถส่งใบเสร็จไปที่",
-    'refund' => "ประเภทการยกเลิกการขาย",
-    'register' => "ลงทะเบียนขาย",
-    'remove_customer' => "ลบลูกค้า",
-    'remove_discount' => "",
-    'return' => "คืน",
-    'rewards' => "คะแนนสะสม",
-    'rewards_balance' => "คะแนนสะสมคงเหลือ",
-    'sale' => "ขาย",
-    'sale_by_invoice' => "การขายโดยใบแจ้งหนี้",
-    'sale_for_customer' => "ลูกค้า:",
-    'sale_time' => "เวลา",
-    'sales_tax' => "ภาษีการขาย",
-    'sales_total' => "",
-    'select_customer' => "เลือกลูกค้า (Optional)",
-    'send_invoice' => "ส่งใบแจ้งหนี้",
-    'send_quote' => "ส่งใบเสนอราคา",
-    'send_receipt' => "ส่งใบเสร็จ",
-    'send_work_order' => "ส่งคำสั่งงาน",
-    'serial' => "หมายเลขซีเรียล",
-    'service_charge' => "",
-    'show_due' => "",
-    'show_invoice' => "ใบแจ้งหนี้",
-    'show_receipt' => "ใบเสร็จ",
-    'start_typing_customer_name' => "เริ่มต้นพิมพ์ชื่อลูกค้า...",
-    'start_typing_item_name' => "เริ่มต้นพิมพ์ชื่อสินค้า หรือ สแกนบาร์โค๊ด...",
-    'stock' => "คลังสินค้า",
-    'stock_location' => "ที่เก็บ",
-    'sub_total' => "ยอดรวมย่อย",
-    'successfully_deleted' => "ลบการขายสมยูรณ์",
-    'successfully_restored' => "คุณกู้คืนสำเร็จแล้ว",
-    'successfully_suspended_sale' => "การขายของคุณถูกระงับเรียบร้อย",
-    'successfully_updated' => "อัพเดทการขายสมบูรณ์",
-    'suspend_sale' => "พักรายการ",
-    'suspended_doc_id' => "รหัสเอกสาร",
-    'suspended_sale_id' => "รหัสการขายที่ถูกพัก",
-    'suspended_sales' => "การขายที่พักไว้",
-    'table' => "โต๊ะ",
-    'takings' => "การขายประจำวัน",
-    'tax' => "ภาษี",
-    'tax_id' => "รหัสภาษี",
-    'tax_invoice' => "ใบกำกับภาษี",
-    'tax_percent' => "ภาษี %",
-    'taxed_ind' => "ภ",
-    'total' => "ยอดรวม",
-    'total_tax_exclusive' => "ยอดไม่รวมภาษี",
-    'transaction_failed' => "การดำเนินการขายล้มเหลว",
-    'unable_to_add_item' => "เพิ่มรายการไปยังการขายล้มเหลว",
-    'unsuccessfully_deleted' => "ลบการขายไม่สำเร็จ",
-    'unsuccessfully_restored' => "การคืนค่ารายการขายล้มเหลว",
-    'unsuccessfully_suspended_sale' => "การขายของคุณถูกระงับเรียบร้อย",
-    'unsuccessfully_updated' => "อัพเดทการขายไม่สมบูรณ์",
-    'unsuspend' => "ยกเลิกการระงับ",
-    'unsuspend_and_delete' => "ยกเลิกการระงับ และ ลบ",
-    'update' => "แก้ไข",
-    'upi' => "ยูพีไอ",
-    'visa' => "",
-    'wholesale' => "",
-    'work_order' => "คำสั่งงาน",
-    'work_order_number' => "หมายเลขคำสั่งงาน",
-    'work_order_number_duplicate' => "หมายเลขคำสั่งงานต้องไม่ซ้ำกัน",
-    'work_order_sent' => "คำสั่งงานส่งถึง",
-    'work_order_unsent' => "ส่งคำสั่งงานล้มเหลว",
-    'selected_customer' => "ลูกค้าที่เลือก",
+    "customers_available_points"       => "คะแนนที่มี",
+    "rewards_package"                  => "คะแนนสะสม",
+    "rewards_remaining_balance"        => "คะแนนสะสมคงเหลือ ",
+    "account_number"                   => "บัญชี #",
+    "add_payment"                      => "เพิ่มบิล",
+    "amount_due"                       => "ยอดค้างชำระ",
+    "amount_tendered"                  => "ชำระเข้ามา",
+    "authorized_signature"             => "ลายเซ็นผู้มีอำนาจ",
+    "cancel_sale"                      => "ยกเลิกการขาย",
+    "cash"                             => "เงินสด",
+    "cash_1"                           => "",
+    "cash_2"                           => "",
+    "cash_3"                           => "",
+    "cash_4"                           => "",
+    "cash_adjustment"                  => "การปรับเงินสดขาย",
+    "cash_deposit"                     => "ฝากเงินสด",
+    "cash_filter"                      => "เงินสด",
+    "change_due"                       => "เงินทอน",
+    "change_price"                     => "เปลี่ยนราคาขาย",
+    "check"                            => "โอนเงิน/พร้อมเพย์/เช็ค",
+    "check_balance"                    => "เช็คยอดคงเหลือ",
+    "check_filter"                     => "ตรวจสอบ",
+    "close"                            => "",
+    "comment"                          => "หมายเหตุ",
+    "comments"                         => "หมายเหตุ",
+    "company_name"                     => "",
+    "complete"                         => "",
+    "complete_sale"                    => "จบการขาย",
+    "confirm_cancel_sale"              => "แน่ใจหรือไม่ที่จะล้างการขายนี้? ทุกรายการจะถูกลบทั้งหมด",
+    "confirm_delete"                   => "โปรดยืนยันการลบรายการขายที่เลือกไว้ ?",
+    "confirm_restore"                  => "คุณแน่ใจหรือไม่ว่าต้องการยกเลิกการขายที่เลือกไว้?",
+    "credit"                           => "เครดิตการ์ด",
+    "credit_deposit"                   => "เงินฝากเครดิต",
+    "credit_filter"                    => "บัตรเครติด",
+    "current_table"                    => "",
+    "customer"                         => "ลูกค้า",
+    "customer_address"                 => "Customer Address",
+    "customer_discount"                => "ส่วนลด",
+    "customer_email"                   => "Customer Email",
+    "customer_location"                => "Customer Location",
+    "customer_mailchimp_status"        => "สถานะของระบบส่งเมล์เมล์ชิม",
+    "customer_optional"                => "(ต้องระบุวันที่ชำระเงิน)",
+    "customer_required"                => "(ต้องระบุ)",
+    "customer_total"                   => "Total",
+    "customer_total_spent"             => "",
+    "daily_sales"                      => "",
+    "date"                             => "วันที่ขาย",
+    "date_range"                       => "ระหว่างวันที่",
+    "date_required"                    => "กรุณากรอกวันที่ให้ถูกต้อง",
+    "date_type"                        => "กรุณากรอกข้อมูลในช่องวันที่",
+    "debit"                            => "บัตรประชารัฐ/เดบิตการ์ด",
+    "debit_filter"                     => "",
+    "delete"                           => "อนุญาตให้ลบ",
+    "delete_confirmation"              => "แน่ใจหรือไม่ที่จะลบรายการขายนี้, ลบแล้วไม่สามารถเรียกกลับคืนใด้",
+    "delete_entire_sale"               => "ลบการขายทั้งหมด",
+    "delete_successful"                => "คุณลบการขายสำเร็จ",
+    "delete_unsuccessful"              => "คุณลบการขายไม่สำเร็จ",
+    "description_abbrv"                => "รายละเอียด",
+    "discard"                          => "ยกเลิก",
+    "discard_quote"                    => "",
+    "discount"                         => "ส่วนลด %",
+    "discount_included"                => "% ส่วนลด",
+    "discount_short"                   => "%",
+    "due"                              => "วันครบกำหนด",
+    "due_filter"                       => "วันที่ครบกำหนด",
+    "edit"                             => "แก้ไข",
+    "edit_item"                        => "แก้ไขสินค้า",
+    "edit_sale"                        => "แก้ไขการขาย",
+    "email_receipt"                    => "อีเมลบิล",
+    "employee"                         => "พนักงาน",
+    "entry"                            => "การนำเข้า",
+    "error_editing_item"               => "แก้ไขสินค้าล้มเหลว",
+    "negative_price_invalid"           => "",
+    "negative_quantity_invalid"        => "",
+    "negative_discount_invalid"        => "",
+    "discount_percent_exceeds_100"     => "",
+    "discount_exceeds_item_total"      => "",
+    "negative_total_invalid"           => "",
+    "find_or_scan_item"                => "ค้นหาสินค้า",
+    "find_or_scan_item_or_receipt"     => "ค้นหา หรือ แสกนรายการ หรือ ใบเสร็จ",
+    "giftcard"                         => "บัตรของขวัญ",
+    "giftcard_balance"                 => "ยอดคงเหลือบัตรของขวัญ",
+    "giftcard_filter"                  => "",
+    "giftcard_number"                  => "เลขที่บัตรของขวัญ",
+    "group_by_category"                => "กลุ่มตามหมวดหมู่",
+    "group_by_type"                    => "กลุ่มตามประเภท",
+    "hsn"                              => "HSN",
+    "id"                               => "เลขที่ขาย",
+    "include_prices"                   => "รวมในราคา?",
+    "invoice"                          => "ใบแจ้งหนี้",
+    "invoice_confirm"                  => "ใบแจ้งหนี้นี้จะถูกส่งไปที่",
+    "invoice_enable"                   => "เลขที่ใบแจ้งหนี้",
+    "invoice_filter"                   => "ใบแจ้งหนี้",
+    "invoice_no_email"                 => "ลูกค้ารายนี้ไม่มีที่อยู่อีเมล",
+    "invoice_number"                   => "เลขใบแจ้งหนี้ #",
+    "invoice_number_duplicate"         => "ใบแจ้งหนี้หมายเลข {0} จะต้องไม่ซ้ำกัน",
+    "invoice_sent"                     => "ส่งใบแจ้งหนี้ไปที่",
+    "invoice_total"                    => "ยอดรวมในใบแจ้งหนี้",
+    "invoice_type_custom_invoice"      => "ใบแจ้งหนี้ที่กำหนดเอง (custom_invoice.php)",
+    "invoice_type_custom_tax_invoice"  => "ใบกำกับภาษีที่กำหนดเอง (custom_tax_invoice.php)",
+    "invoice_type_invoice"             => "ใบแจ้งหนี้ (invoice.php)",
+    "invoice_type_tax_invoice"         => "ใบกำกับภาษี (tax_invoice.php)",
+    "invoice_unsent"                   => "ไม่สามารถส่งใบแจ้งหนี้ถึง",
+    "invoice_update"                   => "คำนวณใหม่",
+    "item_insufficient_of_stock"       => "จำนวนสินค้าไม่เพียงพอ",
+    "item_name"                        => "ชื่อสินค้า",
+    "item_number"                      => "สินค้า #",
+    "item_out_of_stock"                => "สินค้าจำหน่ายหมด",
+    "key_browser"                      => "ความช่วยเหลือ",
+    "key_cancel"                       => "ยกเลิกใบเสนอราคา/ใบแจ้งหนี้ /ใบการขาย นี้",
+    "key_customer_search"              => "ค้นหาลูกค้า",
+    "key_finish_quote"                 => "จบใบเสนอราคา/ใบแจ้งหนี้โดยไม่ต้องชำระเงิน",
+    "key_finish_sale"                  => "เพิ่มการชำระเงินและใบแจ้งหนี้ /ใบรายการขาย",
+    "key_full"                         => "เปิดแบบเต็มหน้าจอ",
+    "key_function"                     => "ฟังก์ชั่น",
+    "key_help"                         => "คำสั่งลัดงานขาย",
+    "key_help_modal"                   => "เปิดหน้าต่างคำสั่งลัดงานขาย",
+    "key_in"                           => "ขยายเข้า",
+    "key_item_search"                  => "ค้นหารายการขาย",
+    "key_out"                          => "ขยายออก",
+    "key_payment"                      => "เพิ่มการชำระเงิน",
+    "key_print"                        => "พิมพ์หน้านี้",
+    "key_restore"                      => "คืนการแสดงผลแบบดั้งเดิม/ขยาย",
+    "key_search"                       => "ค้นหาตารางรายงาน",
+    "key_suspend"                      => "พักรายการขายปัจจุบัน",
+    "key_suspended"                    => "แสดงรายการขายที่พักไว้",
+    "key_system"                       => "ทางลัดระบบ",
+    "key_tendered"                     => "แก้ไขจำนวนเงินรับมา",
+    "key_title"                        => "ทางลัดคียบอร์ดงานขาย",
+    "mc"                               => "",
+    "mode"                             => "รูปแบบการลงทะเบียน",
+    "must_enter_numeric"               => "จำนวนที่ถุกประมูลต้องใส่ข้อมุลที่เปนตัวเลข",
+    "must_enter_numeric_giftcard"      => "เลขที่บัตรของขวัญ ต้องใส่ตัวเลขเท่านั้น",
+    "new_customer"                     => "ลูกค้าใหม่",
+    "new_item"                         => "สินค้าใหม่",
+    "no_description"                   => "ไม่ระบุรายละเอียด",
+    "no_filter"                        => "ทั้งหมด",
+    "no_items_in_cart"                 => "ไม่พบสินค้าในตระกร้า",
+    "no_sales_to_display"              => "ไม่มีการขายที่จะแสดง",
+    "none_selected"                    => "คุณยังไม่ได้เลือกการขายที่จะลบ",
+    "nontaxed_ind"                     => " . ",
+    "not_authorized"                   => "การกระทำนี้ไม่ได้รับอนุญาต",
+    "one_or_multiple"                  => "การขาย",
+    "payment"                          => "รูปแบบชำระเงิน",
+    "payment_amount"                   => "จำนวน",
+    "payment_not_cover_total"          => "จำนวนเงินที่ชำระต้องมากกว่าหรือเท่ากับยอดรวม",
+    "payment_type"                     => "ชำระโดย",
+    "payments"                         => "",
+    "payments_total"                   => "ยอดชำระแล้ว",
+    "price"                            => "ราคา",
+    "print_after_sale"                 => "พิมพ์บิลหลังการขาย",
+    "quantity"                         => "จำนวน",
+    "quantity_less_than_reorder_level" => "คำเตือน ถ้าจำนวนของไม่เพียงพอกับความต้องการหรือไม่ตรงกับยอดในบันชี ก็สามารถทำการขายได้ แต่ต้องเชคปริมานสินค้าคงคลัง",
+    "quantity_less_than_zero"          => "คำเตือน: ถ้าจำนวนของไม่เพียงพอกับความต้องการหรือไม่ตรงกับยอดในบัญชี ก็สามารถทำการขายได้ แต่ต้องตรวจสอบปริมาญสินค้าคงคลังก่อน",
+    "quantity_of_items"                => "ปริมาณของ {0} รายการ",
+    "quote"                            => "ใบเสนอราคา",
+    "quote_number"                     => "หมายเลขอ้างอิง",
+    "quote_number_duplicate"           => "หมายเลขอ้างอิงต้องไม่ซ้ำกัน",
+    "quote_sent"                       => "ส่งการอ้างอิงถึง",
+    "quote_unsent"                     => "ส่งการอ้างอิงถึงผิดพลาด",
+    "receipt"                          => "บิลขาย",
+    "receipt_no_email"                 => "ลูกค้านี้ไม่มีที่อยู่อีเมล์",
+    "receipt_number"                   => "จุดขาย#",
+    "receipt_sent"                     => "ส่งใบเสร็จไปที่",
+    "receipt_unsent"                   => "ไม่สามารถส่งใบเสร็จไปที่",
+    "refund"                           => "ประเภทการยกเลิกการขาย",
+    "register"                         => "ลงทะเบียนขาย",
+    "remove_customer"                  => "ลบลูกค้า",
+    "remove_discount"                  => "",
+    "return"                           => "คืน",
+    "rewards"                          => "คะแนนสะสม",
+    "rewards_balance"                  => "คะแนนสะสมคงเหลือ",
+    "sale"                             => "ขาย",
+    "sale_by_invoice"                  => "การขายโดยใบแจ้งหนี้",
+    "sale_for_customer"                => "ลูกค้า:",
+    "sale_time"                        => "เวลา",
+    "sales_tax"                        => "ภาษีการขาย",
+    "sales_total"                      => "",
+    "select_customer"                  => "เลือกลูกค้า (Optional)",
+    "send_invoice"                     => "ส่งใบแจ้งหนี้",
+    "send_quote"                       => "ส่งใบเสนอราคา",
+    "send_receipt"                     => "ส่งใบเสร็จ",
+    "send_work_order"                  => "ส่งคำสั่งงาน",
+    "serial"                           => "หมายเลขซีเรียล",
+    "service_charge"                   => "",
+    "show_due"                         => "",
+    "show_invoice"                     => "ใบแจ้งหนี้",
+    "show_receipt"                     => "ใบเสร็จ",
+    "start_typing_customer_name"       => "เริ่มต้นพิมพ์ชื่อลูกค้า...",
+    "start_typing_item_name"           => "เริ่มต้นพิมพ์ชื่อสินค้า หรือ สแกนบาร์โค๊ด...",
+    "stock"                            => "คลังสินค้า",
+    "stock_location"                   => "ที่เก็บ",
+    "sub_total"                        => "ยอดรวมย่อย",
+    "successfully_deleted"             => "ลบการขายสมยูรณ์",
+    "successfully_restored"            => "คุณกู้คืนสำเร็จแล้ว",
+    "successfully_suspended_sale"      => "การขายของคุณถูกระงับเรียบร้อย",
+    "successfully_updated"             => "อัพเดทการขายสมบูรณ์",
+    "suspend_sale"                     => "พักรายการ",
+    "suspended_doc_id"                 => "รหัสเอกสาร",
+    "suspended_sale_id"                => "รหัสการขายที่ถูกพัก",
+    "suspended_sales"                  => "การขายที่พักไว้",
+    "table"                            => "โต๊ะ",
+    "takings"                          => "การขายประจำวัน",
+    "tax"                              => "ภาษี",
+    "tax_id"                           => "รหัสภาษี",
+    "tax_invoice"                      => "ใบกำกับภาษี",
+    "tax_percent"                      => "ภาษี %",
+    "taxed_ind"                        => "ภ",
+    "total"                            => "ยอดรวม",
+    "total_tax_exclusive"              => "ยอดไม่รวมภาษี",
+    "transaction_failed"               => "การดำเนินการขายล้มเหลว",
+    "unable_to_add_item"               => "เพิ่มรายการไปยังการขายล้มเหลว",
+    "unsuccessfully_deleted"           => "ลบการขายไม่สำเร็จ",
+    "unsuccessfully_restored"          => "การคืนค่ารายการขายล้มเหลว",
+    "unsuccessfully_suspended_sale"    => "การขายของคุณถูกระงับเรียบร้อย",
+    "unsuccessfully_updated"           => "อัพเดทการขายไม่สมบูรณ์",
+    "unsuspend"                        => "ยกเลิกการระงับ",
+    "unsuspend_and_delete"             => "ยกเลิกการระงับ และ ลบ",
+    "update"                           => "แก้ไข",
+    "upi"                              => "ยูพีไอ",
+    "visa"                             => "",
+    "wholesale"                        => "",
+    "work_order"                       => "คำสั่งงาน",
+    "work_order_number"                => "หมายเลขคำสั่งงาน",
+    "work_order_number_duplicate"      => "หมายเลขคำสั่งงานต้องไม่ซ้ำกัน",
+    "work_order_sent"                  => "คำสั่งงานส่งถึง",
+    "work_order_unsent"                => "ส่งคำสั่งงานล้มเหลว",
+    "selected_customer"                => "ลูกค้าที่เลือก",
 ];
--- a/app/Libraries/Email_lib.php
+++ b/app/Libraries/Email_lib.php
@@ -82,40 +82,4 @@ class Email_lib

        return $result;
    }
-
-    /**
-     * Gets the mime type of the company logo file.
-     *
-     * @return string Mime type or empty string if logo doesn't exist
-     */
-    public function getLogoMimeType(): string
-    {
-        $logo_path = FCPATH . 'uploads/' . $this->config['company_logo'];
-
-        if (!empty($this->config['company_logo']) && file_exists($logo_path)) {
-            $mimeType = mime_content_type($logo_path);
-            return $mimeType !== false ? $mimeType : '';
-        }
-
-        return '';
-    }
-
-    /**
-     * Builds an img tag for the company logo to use in email templates.
-     *
-     * @return string HTML img tag with base64-encoded logo, or empty string if no logo
-     */
-    public function buildLogoImgTag(): string
-    {
-        $mimeType = $this->getLogoMimeType();
-
-        if ($mimeType === '') {
-            return '';
-        }
-
-        $logo_path = FCPATH . 'uploads/' . $this->config['company_logo'];
-        $logo_data = base64_encode(file_get_contents($logo_path));
-
-        return '<img id="image" src="data:' . $mimeType . ';base64,' . $logo_data . '" alt="company_logo">';
-    }
 }
--- a/app/Libraries/MY_Migration.php
+++ b/app/Libraries/MY_Migration.php
@@ -2,6 +2,7 @@

 namespace App\Libraries;

+use CodeIgniter\Database\Exceptions\DatabaseException;
 use CodeIgniter\Database\MigrationRunner;
 use Config\Database;
 use stdClass;
@@ -43,9 +44,7 @@ class MY_Migration extends MigrationRunner
                $result = $builder->get()->getRow();
                return $result ? $result->version : 0;
            }
-        } catch (\Exception $e) {
-            // Database not available yet (e.g. fresh install before schema).
-            // Catches mysqli_sql_exception which is not a DatabaseException.
+        } catch (DatabaseException $e) {
            return 0;
        }

@@ -77,9 +76,8 @@ class MY_Migration extends MigrationRunner
                $result = $builder->get()->getRow();
                return $result ? $result->version : false;
            }
-        } catch (\Exception $e) {
-            // Database not available yet (e.g. fresh install before schema).
-            // Catches mysqli_sql_exception which is not a DatabaseException.
+        } catch (DatabaseException $e) {
+            // Database doesn't exist yet or connection failed
        }

        return false;
--- a/app/Libraries/Sale_lib.php
+++ b/app/Libraries/Sale_lib.php
@@ -23,19 +23,6 @@ use ReflectionException;
 */
 class Sale_lib
 {
-    private const KEY_SHORTCUT_DEFAULTS = [
-        'cancel'    => ['value' => '27 | ESC', 'code' => 27, 'label' => 'ESC'],
-        'items'     => ['value' => '49 | ALT + 1', 'code' => 49, 'label' => 'ALT + 1'],
-        'customers' => ['value' => '50 | ALT + 2', 'code' => 50, 'label' => 'ALT + 2'],
-        'suspend'   => ['value' => '51 | ALT + 3', 'code' => 51, 'label' => 'ALT + 3'],
-        'suspended' => ['value' => '52 | ALT + 4', 'code' => 52, 'label' => 'ALT + 4'],
-        'amount'    => ['value' => '53 | ALT + 5', 'code' => 53, 'label' => 'ALT + 5'],
-        'payment'   => ['value' => '54 | ALT + 6', 'code' => 54, 'label' => 'ALT + 6'],
-        'complete'  => ['value' => '55 | ALT + 7', 'code' => 55, 'label' => 'ALT + 7'],
-        'finish'    => ['value' => '56 | ALT + 8', 'code' => 56, 'label' => 'ALT + 8'],
-        'help'      => ['value' => '57 | ALT + 9', 'code' => 57, 'label' => 'ALT + 9'],
-    ];
-
    private Attribute $attribute;
    private Customer $customer;
    private Dinner_table $dinner_table;
@@ -108,11 +95,6 @@ class Sale_lib
        'custom_tax_invoice'
    ];

-    private const ALLOWED_RECEIPT_TEMPLATES = [
-        'receipt_default',
-        'receipt_short'
-    ];
-
    public function get_invoice_type_options(): array
    {
        $invoice_types = [];
@@ -123,54 +105,11 @@ class Sale_lib
        return $invoice_types;
    }

-    /**
-     * Returns the available keyboard shortcut choices for the configuration screen.
-     *
-     * @return array<string, string>
-     */
-    public function getKeyShortcutsOptions(): array
-    {
-        $keyShortcuts = [];
-
-        foreach (self::KEY_SHORTCUT_DEFAULTS as $shortcut) {
-            $keyShortcuts[$shortcut['value']] = $shortcut['label'];
-        }
-
-        return $keyShortcuts;
-    }
-
-    /**
-     * Returns parsed shortcut bindings from app_config with sensible defaults.
-     *
-     * @return array<string, array{value:string,code:int,label:string}>
-     */
-    public function getKeyShortcuts(): array
-    {
-        $keyboardShortcuts = [];
-
-        foreach (self::KEY_SHORTCUT_DEFAULTS as $name => $default) {
-            $value = $this->config["key_$name"] ?? $default['value'];
-            $parts = array_map('trim', explode('|', $value, 2));
-            $keyboardShortcuts[$name] = [
-                'value' => $value,
-                'code'  => (int)($parts[0] ?? $default['code']),
-                'label' => $parts[1] ?? $default['label']
-            ];
-        }
-
-        return $keyboardShortcuts;
-    }
-
    public static function isValidInvoiceType(string $invoice_type): bool
    {
        return in_array($invoice_type, self::ALLOWED_INVOICE_TYPES, true);
    }

-    public static function isValidReceiptTemplate(string $receipt_template): bool
-    {
-        return in_array($receipt_template, self::ALLOWED_RECEIPT_TEMPLATES, true);
-    }
-
    /**
     * @return array
     */
--- a/app/Models/Attribute.php
+++ b/app/Models/Attribute.php
@@ -601,10 +601,6 @@ class Attribute extends Model
     */
    public function saveAttributeLink(int $itemId, int $definitionId, int $attributeId): bool
    {
-        if ($attributeId <= 0) {
-            return false;
-        }
-
        $normalizedItemId = empty($itemId) ? null : $itemId;
        $normalizedAttributeId = empty($attributeId) ? null : $attributeId;

--- a/app/Models/Item.php
+++ b/app/Models/Item.php
@@ -65,10 +65,8 @@ class Item extends Model
    public function exists(string $item_id, bool $ignore_deleted = false, bool $deleted = false): bool
    {
        $builder = $this->db->table('items');
-        $builder->groupStart();
        $builder->where('item_id', $item_id);
        $builder->orWhere('item_number', $item_id);
-        $builder->groupEnd();

        if (!$ignore_deleted) {
            $builder->where('deleted', $deleted);
@@ -391,10 +389,9 @@ class Item extends Model
    public function get_item_id(string $item_number, bool $ignore_deleted = false, bool $deleted = false): bool|int
    {
        $builder = $this->db->table('items');
-        $builder->groupStart();
+        $builder->join('suppliers', 'suppliers.person_id = items.supplier_id', 'left');
        $builder->where('item_number', $item_number);
        $builder->orWhere('item_id', $item_number);
-        $builder->groupEnd();

        if (!$ignore_deleted) {
            $builder->where('items.deleted', $deleted);
@@ -439,32 +436,62 @@ class Item extends Model

    /**
     * Inserts or updates an item
+     * 
+     * If the primary key (item_id) is present in the data array and the record exists,
+     * it will update the existing record. Otherwise, it will insert a new record.
+     * 
+     * @param array $data The item data to save (passed by reference to set item_id on insert)
+     * @return bool True on success, false on failure
     */
-    public function save_value(array &$item_data, int $item_id = NEW_ENTRY): bool    // TODO: need to bring this in line with parent or change the name
+    public function saveValue(array &$data): bool
    {
-        $builder = $this->db->table('items');
+        $primaryKey = $this->primaryKey;
+        $id = $data[$primaryKey] ?? NEW_ENTRY;

-        if ($item_id < 1 || !$this->exists($item_id, true)) {
-            if ($builder->insert($item_data)) {
-                $item_data['item_id'] = (int)$this->db->insertID();
-                if ($item_id < 1) {
-                    $builder = $this->db->table('items');
-                    $builder->where('item_id', $item_data['item_id']);
-                    $builder->update(['low_sell_item_id' => $item_data['item_id']]);
-                }
-
-                return true;
+        // If id > 0 and record exists by primary key only, update it
+        if ($id > 0) {
+            // Check existence strictly by primary key (regardless of soft-delete status)
+            $builder = $this->db->table('items');
+            $builder->where($primaryKey, $id);
+            $exists = $builder->countAllResults() > 0;
+            
+            if ($exists) {
+                // Remove primary key from data array for update
+                $updateData = $data;
+                unset($updateData[$primaryKey]);
+                
+                $builder = $this->db->table('items');
+                $builder->where($primaryKey, $id);
+                return $builder->update($updateData);
            }
-
-            return false;
-        } else {
-            $item_data['item_id'] = $item_id;
        }

+        // Insert new record with transaction for atomicity
+        $this->db->transBegin();
+        
+        // Remove primary key from insert payload if present
+        $insertData = $data;
+        unset($insertData[$primaryKey]);
+        
        $builder = $this->db->table('items');
-        $builder->where('item_id', $item_id);
-
-        return $builder->update($item_data);
+        $success = $builder->insert($insertData);
+        
+        if ($success) {
+            $data[$primaryKey] = (int)$this->db->insertID();
+            
+            // Update low_sell_item_id for new items
+            $builder = $this->db->table('items');
+            $builder->where($primaryKey, $data[$primaryKey]);
+            $success = $builder->update(['low_sell_item_id' => $data[$primaryKey]]);
+        }
+        
+        if ($success) {
+            $this->db->transCommit();
+            return true;
+        }
+        
+        $this->db->transRollback();
+        return false;
    }

    /**
@@ -1082,9 +1109,9 @@ class Item extends Model
        $total_quantity = $old_total_quantity + $items_received;
        $average_price = bcdiv(bcadd(bcmul((string)$items_received, (string)$new_price), bcmul((string)$old_total_quantity, (string)$old_price)), (string)$total_quantity);

-        $data = ['cost_price' => $average_price];
+        $data = ['cost_price' => $average_price, 'item_id' => $item_id];

-        return $this->save_value($data, $item_id);
+        return $this->saveValue($data);
    }

    /**
--- a/app/Models/Receiving.php
+++ b/app/Models/Receiving.php
@@ -294,9 +294,7 @@ class Receiving extends Model
            lang('Sales.check') => lang('Sales.check'),
            lang('Sales.debit') => lang('Sales.debit'),
            lang('Sales.credit') => lang('Sales.credit'),
-            lang('Sales.due') => lang('Sales.due'),
-            lang('Sales.bank_transfer') => lang('Sales.bank_transfer'),
-            lang('Sales.wallet') => lang('Sales.wallet')
+            lang('Sales.due') => lang('Sales.due')
        ];
    }

--- a/app/Models/Reports/Summary_sales_taxes.php
+++ b/app/Models/Reports/Summary_sales_taxes.php
@@ -33,16 +33,14 @@ class Summary_sales_taxes extends Summary_report
     * @param object $builder
     * @return void
     */
-    protected function _where(array $inputs, object &$builder): void
+    protected function _where(array $inputs, object &$builder): void    // TODO: hungarian notation
    {
        $builder->where('sales.sale_status', COMPLETED);

-        if (empty($this->config['date_or_time_format'])) {
-            $builder->where('DATE(sales.sale_time) >=', $inputs['start_date']);
-            $builder->where('DATE(sales.sale_time) <=', $inputs['end_date']);
+        if (empty($this->config['date_or_time_format'])) {    // TODO: Duplicated code
+            $builder->where('DATE(sales.sale_time) BETWEEN ' . $this->db->escape($inputs['start_date']) . ' AND ' . $this->db->escape($inputs['end_date']));
        } else {
-            $builder->where('sales.sale_time >=', $inputs['start_date']);
-            $builder->where('sales.sale_time <=', $inputs['end_date']);
+            $builder->where('sales.sale_time BETWEEN ' . $this->db->escape(rawurldecode($inputs['start_date'])) . ' AND ' . $this->db->escape(rawurldecode($inputs['end_date'])));
        }
    }

@@ -55,11 +53,9 @@ class Summary_sales_taxes extends Summary_report
        $builder = $this->db->table('sales_taxes');

        if (empty($this->config['date_or_time_format'])) {
-            $builder->where('DATE(sale_time) >=', $inputs['start_date']);
-            $builder->where('DATE(sale_time) <=', $inputs['end_date']);
+            $builder->where('DATE(sale_time) BETWEEN ' . $inputs['start_date'] . ' AND ' . $inputs['end_date']);
        } else {
-            $builder->where('sale_time >=', $inputs['start_date']);
-            $builder->where('sale_time <=', $inputs['end_date']);
+            $builder->where('sale_time BETWEEN ' . $this->db->escape(rawurldecode($inputs['start_date'])) . ' AND ' . $this->db->escape(rawurldecode($inputs['end_date'])));
        }

        $builder->select('reporting_authority, jurisdiction_name, tax_category, tax_rate, SUM(sale_tax_amount) AS tax');
--- a/app/Models/Sale.php
+++ b/app/Models/Sale.php
@@ -277,14 +277,6 @@ class Sale extends Model
            $builder->like('payment_type', lang('Sales.debit'));
        }

-        if ($filters['only_bank_transfer']) {
-            $builder->like('payment_type', lang('Sales.bank_transfer'));
-        }
-
-        if ($filters['only_wallet']) {
-            $builder->like('payment_type', lang('Sales.wallet'));
-        }
-
        $builder->groupBy('payment_type');

        $payments = $builder->get()->getResultArray();
@@ -1517,13 +1509,5 @@ class Sale extends Model
        if ($filters['only_check']) {
            $builder->like('payments.payment_type', lang('Sales.check'));
        }
-
-        if ($filters['only_bank_transfer']) {
-            $builder->like('payments.payment_type', lang('Sales.bank_transfer'));
-        }
-
-        if ($filters['only_wallet']) {
-            $builder->like('payments.payment_type', lang('Sales.wallet'));
-        }
    }
 }
--- a/app/Views/barcodes/barcode_sheet.php
+++ b/app/Views/barcodes/barcode_sheet.php
@@ -11,34 +11,31 @@ $barcode_lib = new Barcode_lib();

 <!doctype html>
 <html lang="<?= current_language_code() ?>">
-
-<head>
-    <meta charset="utf-8">
-    <title><?= esc(lang('Items.generate_barcodes')) ?></title>
-    <link rel="stylesheet" href="<?= esc(base_url('css/barcode_font.css'), 'url') ?>">
-    <style>
-        .barcode svg {
-            height: <?= (int) $barcode_config['barcode_height'] ?>px;
-            width: <?= (int) $barcode_config['barcode_width'] ?>px;
-        }
-    </style>
-</head>
-
-<body class="<?= esc('font_' . $barcode_lib->get_font_name($barcode_config['barcode_font']), 'attr') ?>" style="font-size: <?= (int) $barcode_config['barcode_font_size'] ?>px;">
-    <table style="border-spacing: <?= (int) $barcode_config['barcode_page_cellspacing'] ?>px; width: <?= (int) $barcode_config['barcode_page_width'] ?>%;">
-        <tr>
-            <?php
-                $count = 0;
-                foreach ($items as $item) {
-                    if ($count % $barcode_config['barcode_num_in_row'] == 0 && $count != 0) {
-                        echo '</tr><tr>';
+    <head>
+        <meta charset="utf-8">
+        <title><?= lang('Items.generate_barcodes') ?></title>
+        <link rel="stylesheet" href="<?= base_url() ?>css/barcode_font.css">
+        <style>
+            .barcode svg {
+                height: <?= $barcode_config['barcode_height'] ?>px;
+                width: <?= $barcode_config['barcode_width'] ?>px;
+            }
+        </style>
+    </head>
+    <body class=<?= 'font_' . $barcode_lib->get_font_name($barcode_config['barcode_font']) ?> style="font-size: <?= $barcode_config['barcode_font_size'] ?>px;">
+        <table style="border-spacing: <?= $barcode_config['barcode_page_cellspacing'] ?>; width: <?= $barcode_config['barcode_page_width'] ?>%;">
+            <tr>
+                <?php
+                    $count = 0;
+                    foreach ($items as $item) {
+                        if ($count % $barcode_config['barcode_num_in_row'] == 0 && $count != 0) {
+                            echo '</tr><tr>';
+                        }
+                        echo '<td>' . $barcode_lib->display_barcode($item, $barcode_config) . '</td>';
+                        $count++;
                    }
-                    echo '<td>' . $barcode_lib->display_barcode($item, $barcode_config) . '</td>';
-                    $count++;
-                }
-            ?>
-        </tr>
-    </table>
-</body>
-
+                ?>
+            </tr>
+        </table>
+    </body>
 </html>
--- a/app/Views/configs/barcode_config.php
+++ b/app/Views/configs/barcode_config.php
@@ -204,7 +204,6 @@
                <?= form_label(lang('Config.barcode_number_in_row'), 'barcode_num_in_row', ['class' => 'control-label col-xs-2 required']) ?>
                <div class="col-xs-2">
                    <?= form_input([
-                        'type'  => 'number',
                        'name'  => 'barcode_num_in_row',
                        'id'    => 'barcode_num_in_row',
                        'class' => 'form-control input-sm required',
@@ -218,9 +217,6 @@
                <div class="col-sm-2">
                    <div class="input-group">
                        <?= form_input([
-                            'type'  => 'number',
-                            'min'   => '0',
-                            'max'   => '100',
                            'name'  => 'barcode_page_width',
                            'id'    => 'barcode_page_width',
                            'class' => 'form-control input-sm required',
@@ -236,7 +232,6 @@
                <div class="col-sm-2">
                    <div class="input-group">
                        <?= form_input([
-                            'type'  => 'number',
                            'name'  => 'barcode_page_cellspacing',
                            'id'    => 'barcode_page_cellspacing',
                            'class' => 'form-control input-sm required',
--- a/app/Views/configs/email_config.php
+++ b/app/Views/configs/email_config.php
@@ -17,9 +17,9 @@
                    <?= form_dropdown(
                        'protocol',
                        [
-                            'mail'     => 'Mail',
-                            'sendmail' => 'Sendmail',
-                            'smtp'     => 'SMTP'
+                            'mail'     => 'mail',
+                            'sendmail' => 'sendmail',
+                            'smtp'     => 'smtp'
                        ],
                        $config['protocol'],
                        'class="form-control input-sm" id="protocol"'
@@ -55,7 +55,6 @@
                <?= form_label(lang('Config.email_smtp_port'), 'smtp_port', ['class' => 'control-label col-xs-2']) ?>
                <div class="col-xs-2">
                    <?= form_input([
-                        'type'  => 'number',
                        'name'  => 'smtp_port',
                        'id'    => 'smtp_port',
                        'class' => 'form-control input-sm',
@@ -84,7 +83,6 @@
                <?= form_label(lang('Config.email_smtp_timeout'), 'smtp_timeout', ['class' => 'control-label col-xs-2']) ?>
                <div class="col-xs-2">
                    <?= form_input([
-                        'type'  => 'number',
                        'name'  => 'smtp_timeout',
                        'id'    => 'smtp_timeout',
                        'class' => 'form-control input-sm',
--- a/app/Views/configs/info_config.php
+++ b/app/Views/configs/info_config.php
@@ -105,7 +105,6 @@
                            <span class="glyphicon glyphicon-phone-alt"></span>
                        </span>
                        <?= form_input([
-                            'type'  => 'tel',
                            'name'  => 'phone',
                            'id'    => 'phone',
                            'class' => 'form-control input-sm required',
@@ -123,7 +122,6 @@
                            <span class="glyphicon glyphicon-phone-alt"></span>
                        </span>
                        <?= form_input([
-                            'type'  => 'tel',
                            'name'  => 'fax',
                            'id'    => 'fax',
                            'class' => 'form-control input-sm',
--- a/app/Views/configs/manage.php
+++ b/app/Views/configs/manage.php
@@ -29,9 +29,6 @@
    <li role="presentation">
        <a data-toggle="tab" href="#invoice_tab" title="<?= lang('Config.invoice_configuration') ?>"><?= lang('Config.invoice') ?></a>
    </li>
-    <li role="presentation">
-        <a data-toggle="tab" href="#shortcuts_tab" title="<?= lang('Config.shortcuts_configuration') ?>"><?= lang('Config.shortcuts') ?></a>
-    </li>
    <li role="presentation">
        <a data-toggle="tab" href="#reward_tab" title="<?= lang('Config.reward_configuration') ?>"><?= lang('Config.reward') ?></a>
    </li>
@@ -68,9 +65,6 @@
    <div class="tab-pane" id="invoice_tab">
        <?= view('configs/invoice_config') ?>
    </div>
-    <div class="tab-pane" id="shortcuts_tab">
-        <?= view('configs/shortcuts_config') ?>
-    </div>
    <div class="tab-pane" id="reward_tab">
        <?= view('configs/reward_config') ?>
    </div>
--- a/app/Views/configs/shortcuts_config.php
+++ b/app/Views/configs/shortcuts_config.php
@@ -1,88 +0,0 @@
-<?php
-/**
- * @var array $config
- * @var array $keyboardShortcutOptions
- * @var array $keyboardShortcuts
- */
-
-$keyboardShortcuts ??= [];
-$keyboardShortcutOptions ??= [];
-$config ??= [];
-
-$shortcutLabels = [
-    'cancel'    => lang('Sales.key_cancel'),
-    'items'     => lang('Sales.key_item_search'),
-    'customers' => lang('Sales.key_customer_search'),
-    'suspend'   => lang('Sales.key_suspend'),
-    'suspended' => lang('Sales.key_suspended'),
-    'amount'    => lang('Sales.key_tendered'),
-    'payment'   => lang('Sales.key_payment'),
-    'complete'  => lang('Sales.key_finish_sale'),
-    'finish'    => lang('Sales.key_finish_quote'),
-    'help'      => lang('Sales.key_help_modal')
-];
-?>
-
-<?= form_open('config/saveShortcuts', ['id' => 'shortcuts_config_form', 'class' => 'form-horizontal']) ?>
-    <div id="config_wrapper">
-        <div class="row">
-            <fieldset id="config_info">
-                <div class="col-md-8">
-                    <div id="required_fields_message"><?= esc(lang('Common.fields_required_message')) ?></div>
-                    <ul id="shortcuts_error_message_box" class="error_message_box"></ul>
-
-                    <?php foreach ($shortcutLabels as $name => $label): ?>
-                        <div class="form-group form-group-sm">
-                            <?= form_label($label, 'key_' . $name, ['class' => 'control-label col-xs-3']) ?>
-                            <div class="col-xs-4">
-                                <?php $keyboardShortcutSelectedValue = $keyboardShortcuts[$name]['value'] ?? ''; ?>
-                                <?= form_dropdown(
-                                    'key_' . $name,
-                                    $keyboardShortcutOptions,
-                                    $keyboardShortcutSelectedValue,
-                                    'class="form-control input-sm"'
-                                ) ?>
-                            </div>
-                        </div>
-                    <?php endforeach; ?>
-
-                    <div class="col-xs-12 clearfix">
-                        <?= form_submit([
-                            'name'  => 'submit_shortcuts',
-                            'id'    => 'submit_shortcuts',
-                            'value' => lang('Common.submit'),
-                            'class' => 'btn btn-primary btn-sm pull-right'
-                        ]) ?>
-                    </div>
-                </div>
-            </fieldset>
-        </div>
-    </div>
-<?= form_close() ?>
-
-<script type="text/javascript">
-    $('#shortcuts_config_form').validate($.extend(form_support.handler, {
-        submitHandler: function(form) {
-            $(form).ajaxSubmit({
-                success: function(response) {
-                    $.notify({
-                        message: response.message
-                    }, {
-                        type: response.success ? 'success' : 'danger'
-                    });
-                },
-                error: function(xhr) {
-                    const rawMessage = xhr.responseJSON?.message ?? xhr.responseText ?? <?= json_encode(lang('Config.shortcuts_save_error')) ?>;
-                    $.notify({
-                        message: DOMPurify.sanitize(rawMessage)
-                    }, {
-                        type: 'danger'
-                    });
-                },
-                dataType: 'json'
-            });
-        },
-
-        errorLabelContainer: '#shortcuts_error_message_box'
-    }));
-</script>
--- a/app/Views/configs/system_info.php
+++ b/app/Views/configs/system_info.php
@@ -25,8 +25,8 @@ use Config\OSPOS;
    <div class="container">
        <div class="row">
            <div class="col-sm-2" style="text-align: left;"><br>
-                <p style="min-height: 17.7em; font-weight: bold;">General Info</p>
-                <p style="min-height: 12.2em; font-weight: bold;">User Setup</p><br>
+                <p style="min-height: 14.7em; font-weight: bold;">General Info</p>
+                <p style="min-height: 10.5em; font-weight: bold;">User Setup</p><br>
                <p style="font-weight: bold;">Permissions</p>
            </div>
            <div class="col-sm-8" id="issuetemplate" style="text-align: left;"><br>
@@ -42,7 +42,7 @@ use Config\OSPOS;
                    echo "&#187; OpenSSL: ", extension_loaded('openssl') ? '<span style="color: green;">Enabled &#x2713</span>' : '<span style="color: red;">Disabled &#x2717</span>', '<br>';
                    echo "&#187; MBString: ", extension_loaded('mbstring') ? '<span style="color: green;">Enabled &#x2713</span>' : '<span style="color: red;">Disabled &#x2717</span>', '<br>';
                    echo "&#187; Curl: ", extension_loaded('curl') ? '<span style="color: green;">Enabled &#x2713</span>' : '<span style="color: red;">Disabled &#x2717</span>', '<br>';
-                    echo "&#187; Json: ", extension_loaded('json') ? '<span style="color: green;">Enabled &#x2713</span>' : '<span style="color: red;">Disabled &#x2717</span>', '<br>';
+                    echo "&#187; Json: ", extension_loaded('json') ? '<span style="color: green;">Enabled &#x2713</span>' : '<span style="color: red;">Disabled &#x2717</span>', '<br><br>';
                    echo "&#187; Xml: ", extension_loaded('xml') ? '<span style="color: green;">Enabled &#x2713</span>' : '<span style="color: red;">Disabled &#x2717</span>', '<br><br>';
                ?>
                User Configuration:<br>
--- a/app/Views/configs/tax_config.php
+++ b/app/Views/configs/tax_config.php
@@ -51,10 +51,6 @@
                </div>
                <div class="col-xs-1 input-group">
                    <?= form_input([
-                        'type'  => 'number',
-                        'step'  => 'any',
-                        'min'   => '0',
-                        'max'   => '100',
                        'name'  => 'default_tax_1_rate',
                        'id'    => 'default_tax_1_rate',
                        'class' => 'form-control input-sm',
@@ -76,10 +72,6 @@
                </div>
                <div class="col-xs-1 input-group">
                    <?= form_input([
-                        'type'  => 'number',
-                        'step'  => 'any',
-                        'min'   => '0',
-                        'max'   => '100',
                        'name'  => 'default_tax_2_rate',
                        'id'    => 'default_tax_2_rate',
                        'class' => 'form-control input-sm',
--- a/app/Views/errors/html/debug.css
+++ b/app/Views/errors/html/debug.css
@@ -101,11 +101,9 @@ p.lead {
 }

 .tabs {
-    list-style: none;
-    list-style-position: inside;
-    margin: 0;
+    list-style: none inside none;
    padding: 0;
-    margin-bottom: -1px;
+    margin: 0 0 -1px;
 }
 .tabs li {
    display: inline;
--- a/app/Views/login.php
+++ b/app/Views/login.php
@@ -5,14 +5,9 @@
 * @var bool $is_new_install
 * @var string $latest_version
 * @var bool $gcaptcha_enabled
- * @var CodeIgniter\HTTP\IncomingRequest $request
 * @var array $config
 * @var $validation
 */
-
-use Config\Services;
-
-$request = Services::request();
 ?>

 <!doctype html>
@@ -159,6 +154,11 @@ $request = Services::request();
        </div>
    </footer>

+    <?php
+    use Config\Services;
+    $request = Services::request();
+    ?>
+
    <?php if (ENVIRONMENT == 'development' || get_cookie('debug') == 'true' || $request->getGet('debug') == 'true') : ?>
        <!-- inject:login:debug:js -->
        <!-- endinject -->
--- a/app/Views/partial/header.php
+++ b/app/Views/partial/header.php
@@ -12,16 +12,14 @@ $request = Services::request();
 ?>

 <!doctype html>
-<html lang="<?= current_language_code() ?>">
+<html lang="<?= $request->getLocale() ?>">

 <head>
    <meta charset="utf-8">
    <base href="<?= base_url() ?>">
    <title><?= esc($config['company']) . ' | ' . lang('Common.powered_by') . ' OSPOS ' . esc(config('App')->application_version) ?></title>
-    <meta name="robots" content="noindex, nofollow">
    <link rel="shortcut icon" type="image/x-icon" href="images/favicon.ico">
-    <?php $theme = (empty($config['theme']) ? 'flatly' : esc($config['theme'])); ?>
-    <link rel="stylesheet" href="resources/bootswatch/<?= "$theme" ?>/bootstrap.min.css">
+    <link rel="stylesheet" href="<?= 'resources/bootswatch/' . (empty($config['theme']) ? 'flatly' : esc($config['theme'])) . '/bootstrap.min.css' ?>">

    <?php if (ENVIRONMENT == 'development' || get_cookie('debug') == 'true' || $request->getGet('debug') == 'true') : ?>
        <!-- inject:debug:css -->
--- a/app/Views/sales/help.php
+++ b/app/Views/sales/help.php
@@ -1,24 +1,3 @@
-<?php
-/**
- * @var array $keyboardShortcuts
- */
-
-$keyboardShortcuts ??= [];
-
-$shortcut_labels = [
-    'cancel'    => lang('Sales.key_cancel'),
-    'items'     => lang('Sales.key_item_search'),
-    'customers' => lang('Sales.key_customer_search'),
-    'suspend'   => lang('Sales.key_suspend'),
-    'suspended' => lang('Sales.key_suspended'),
-    'amount'    => lang('Sales.key_tendered'),
-    'payment'   => lang('Sales.key_payment'),
-    'complete'  => lang('Sales.key_finish_sale'),
-    'finish'    => lang('Sales.key_finish_quote'),
-    'help'      => lang('Sales.key_help_modal')
-];
-?>
-
 <div class="container-fluid">

    <ul class="nav nav-tabs" id="SCTabs" data-toggle="tab">
@@ -36,13 +15,46 @@ $shortcut_labels = [
                    </tr>
                </thead>
                <tbody>
-                    <?php foreach ($shortcut_labels as $name => $label): ?>
-                        <?php $shortcut = $keyboardShortcuts[$name] ?? ['label' => '', 'code' => '']; ?>
-                        <tr>
-                            <td><code><?= esc($shortcut['label'] !== '' ? $shortcut['label'] : $shortcut['code']) ?></code></td>
-                            <td><?= esc($label) ?></td>
-                        </tr>
-                    <?php endforeach; ?>
+                    <tr>
+                        <td><code>ESC</code></td>
+                        <td><?= lang('Sales.key_cancel'); ?></td>
+                    </tr>
+                    <tr>
+                        <td><code>ALT + 1</code></td>
+                        <td><?= lang('Sales.key_item_search'); ?></td>
+                    </tr>
+                    <tr>
+                        <td><code>ALT + 2</code></td>
+                        <td><?= lang('Sales.key_customer_search'); ?></td>
+                    </tr>
+                    <tr>
+                        <td><code>ALT + 3</code></td>
+                        <td><?= lang('Sales.key_suspend'); ?></td>
+                    </tr>
+                    <tr>
+                        <td><code>ALT + 4</code></td>
+                        <td><?= lang('Sales.key_suspended'); ?></td>
+                    </tr>
+                    <tr>
+                        <td><code>ALT + 5</code></td>
+                        <td><?= lang('Sales.key_tendered'); ?></td>
+                    </tr>
+                    <tr>
+                        <td><code>ALT + 6</code></td>
+                        <td><?= lang('Sales.key_payment'); ?></td>
+                    </tr>
+                    <tr>
+                        <td><code>ALT + 7</code></td>
+                        <td><?= lang('Sales.key_finish_sale'); ?></td>
+                    </tr>
+                    <tr>
+                        <td><code>ALT + 8</code></td>
+                        <td><?= lang('Sales.key_finish_quote'); ?></td>
+                    </tr>
+                    <tr>
+                        <td><code>ALT + 9</code></td>
+                        <td><?= lang('Sales.key_help_modal'); ?></td>
+                    </tr>
                </tbody>
            </table>
        </div>
--- a/app/Views/sales/quote_email.php
+++ b/app/Views/sales/quote_email.php
@@ -15,7 +15,7 @@
 ?>

 <!doctype html>
-<html lang="<?= current_language_code() ?>">
+<html lang="<?= $this->request->getLocale() ?>">

 <head>
    <meta charset="utf-8">
--- a/app/Views/sales/receipt.php
+++ b/app/Views/sales/receipt.php
@@ -2,14 +2,11 @@
 /**
 * @var int $sale_id_num
 * @var bool $print_after_sale
- * @var string $receipt_template_view
 * @var array $config
 */

 use App\Models\Employee;

-$template = $receipt_template_view ?? 'receipt_default';
-
 ?>

 <?= view('partial/header') ?>
@@ -64,6 +61,6 @@ if (isset($error_message)) {
    <?php endif; ?>
 </div>

-<?= view('sales/' . $template) ?>
+<?= view('sales/' . $config['receipt_template']) ?>

 <?= view('partial/footer') ?>
--- a/app/Views/sales/register.php
+++ b/app/Views/sales/register.php
@@ -405,7 +405,6 @@ helper('url');
            <div id="payment_details">
                <?php if ($payments_cover_total) { // Show Complete sale button instead of Add Payment if there is no amount due left ?>
                    <?= form_open("$controller_name/addPayment", ['id' => 'add_payment_form', 'class' => 'form-horizontal']) ?>
-                        <input type="hidden" name="complete_after_payment" value="0">
                        <table class="sales_table_100">
                            <tr>
                                <td><?= lang(ucfirst($controller_name) . '.payment') ?></td>
@@ -446,7 +445,6 @@ helper('url');
                    ?>
                <?php } else { ?>
                    <?= form_open("$controller_name/addPayment", ['id' => 'add_payment_form', 'class' => 'form-horizontal']) ?>
-                        <input type="hidden" name="complete_after_payment" value="0">
                        <table class="sales_table_100">
                            <tr>
                                <td><?= lang(ucfirst($controller_name) . '.payment') ?></td>
@@ -567,21 +565,6 @@ helper('url');
 </div>

 <script type="text/javascript">
-    const keyboardShortcuts = <?= json_encode($keyboardShortcuts ?? []) ?>;
-    const paymentsCoverTotal = <?= json_encode((bool) $payments_cover_total) ?>;
-    const shortcutCodes = {
-        items: keyboardShortcuts?.items?.code ?? null,
-        customers: keyboardShortcuts?.customers?.code ?? null,
-        suspend: keyboardShortcuts?.suspend?.code ?? null,
-        suspended: keyboardShortcuts?.suspended?.code ?? null,
-        amount: keyboardShortcuts?.amount?.code ?? null,
-        payment: keyboardShortcuts?.payment?.code ?? null,
-        complete: keyboardShortcuts?.complete?.code ?? null,
-        finish: keyboardShortcuts?.finish?.code ?? null,
-        help: keyboardShortcuts?.help?.code ?? null,
-        cancel: keyboardShortcuts?.cancel?.code ?? null
-    };
-
    $(document).ready(function() {
        const redirect = function() {
            window.location.href = "<?= site_url('sales'); ?>";
@@ -767,7 +750,6 @@ helper('url');
        });

        $('#add_payment_button').click(function() {
-            $('#add_payment_form').find('input[name="complete_after_payment"]').val('0');
            $('#add_payment_form').submit();
        });

@@ -857,51 +839,43 @@ helper('url');
    }

    // Add Keyboard Shortcuts/Hotkeys to Sale Register
-    document.body.onkeyup = function(event) {
-        if ($(event.target).closest('.modal').length || $('.modal.in').length) {
-            return;
-        }
-        if (event.altKey) {
-            switch (event.keyCode) {
-                case shortcutCodes.items:
-                    $("#item").focus();
-                    $("#item").select();
-                    break;
-                case shortcutCodes.customers:
-                    $("#customer").focus();
-                    $("#customer").select();
-                    break;
-                case shortcutCodes.suspend:
-                    $("#suspend_sale_button").click();
-                    break;
-                case shortcutCodes.suspended:
-                    $("#show_suspended_sales_button").click();
-                    break;
-                case shortcutCodes.amount:
-                    $("#amount_tendered").focus();
-                    $("#amount_tendered").select();
-                    break;
-                case shortcutCodes.payment:
-                    $("#add_payment_button").click();
-                    break;
-                case shortcutCodes.complete:
-                    if (paymentsCoverTotal && $("#finish_sale_button").length) {
-                        $("#finish_sale_button").click();
-                    } else {
-                        $("#add_payment_button").click();
-                    }
-                    break;
-                case shortcutCodes.finish:
-                    $("#finish_invoice_quote_button").click();
-                    break;
-                case shortcutCodes.help:
-                    $("#show_keyboard_help").click();
-                    break;
-            }
+    document.body.onkeyup = function(e) {
+        switch (event.altKey && event.keyCode) {
+            case 49: // Alt + 1 Items Seach
+                $("#item").focus();
+                $("#item").select();
+                break;
+            case 50: // Alt + 2 Customers Search
+                $("#customer").focus();
+                $("#customer").select();
+                break;
+            case 51: // Alt + 3 Suspend Current Sale
+                $("#suspend_sale_button").click();
+                break;
+            case 52: // Alt + 4 Check Suspended
+                $("#show_suspended_sales_button").click();
+                break;
+            case 53: // Alt + 5 Edit Amount Tendered Value
+                $("#amount_tendered").focus();
+                $("#amount_tendered").select();
+                break;
+            case 54: // Alt + 6 Add Payment
+                $("#add_payment_button").click();
+                break;
+            case 55: // Alt + 7 Add Payment and Complete Sales/Invoice
+                $("#add_payment_button").click();
+                window.location.href = "<?= 'sales/complete' ?>";
+                break;
+            case 56: // Alt + 8 Finish Quote/Invoice without payment
+                $("#finish_invoice_quote_button").click();
+                break;
+            case 57: // Alt + 9 Open Shortcuts Help Modal
+                $("#show_keyboard_help").click();
+                break;
        }

        switch (event.keyCode) {
-            case shortcutCodes.cancel:
+            case 27: // ESC Cancel Current Sale
                $("#cancel_sale_button").click();
                break;
        }
--- a/app/Views/sales/work_order_email.php
+++ b/app/Views/sales/work_order_email.php
@@ -14,7 +14,7 @@
 ?>

 <!doctype html>
-<html lang="<?= current_language_code() ?>">
+<html lang="<?= $this->request->getLocale() ?>">

 <head>
    <meta charset="utf-8">
--- a/composer.json
+++ b/composer.json
@@ -2,19 +2,25 @@
    "name": "opensourcepos/opensourcepos",
    "description": "Open Source Point of Sale is a web based POS system written in the PHP language. It uses MySQL as backend and has a simple user interface",
    "license": "MIT",
-    "type": "project",
-    "keywords": [
-        "point-of-sale",
-        "POS"
-    ],
    "authors": [
        {
            "name": "jekkos"
        },
+        {
+            "name": "FrancescoUK"
+        },
        {
            "name": "objecttothis"
+        },
+        {
+            "name": "steveireland"
        }
    ],
+    "type": "project",
+    "keywords": [
+        "point-of-sale",
+        "POS"
+    ],
    "homepage": "https://opensourcepos.org",
    "support": {
        "issues": "https://github.com/opensourcepos/opensourcepos/issues",
@@ -25,8 +31,8 @@
        "matrix": "https://matrix.to/#/#opensourcepos_Lobby:gitter.im"
    },
    "require": {
-        "php": "^8.2",
        "ext-intl": "*",
+        "php": "^8.2",
        "codeigniter4/framework": "4.7.2",
        "dompdf/dompdf": "^2.0.3",
        "ezyang/htmlpurifier": "^4.17",
@@ -50,7 +56,7 @@
    },
    "autoload": {
        "psr-4": {
-            "App\\": "app/",
+            "App\\":  "app/",
            "CodeIgniter\\": "vendor/codeigniter4/framework/system/"
        },
        "exclude-from-classmap": [
@@ -67,8 +73,5 @@
    },
    "scripts": {
        "test": "phpunit"
-    },
-    "scripts-descriptions": {
-        "test": "Run unit tests"
    }
 }
--- a/composer.lock
+++ b/composer.lock
@@ -4,7 +4,7 @@
        "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies",
        "This file is @generated automatically"
    ],
-    "content-hash": "eabbc14aefdea4c933869069d6eadadb",
+    "content-hash": "e95f6e5e86d323370ddb0df57c4d3fb3",
    "packages": [
        {
            "name": "codeigniter4/framework",
@@ -5382,9 +5382,9 @@
    "prefer-stable": false,
    "prefer-lowest": false,
    "platform": {
-        "php": "^8.2",
-        "ext-intl": "*"
+        "ext-intl": "*",
+        "php": "^8.1"
    },
    "platform-dev": {},
-    "plugin-api-version": "2.9.0"
+    "plugin-api-version": "2.6.0"
 }
--- a/docker-compose.dev.yml
+++ b/docker-compose.dev.yml
@@ -46,7 +46,6 @@ services:
            - .:/app
        environment:
            - CI_ENVIRONMENT=development
-            - ALLOWED_HOSTNAMES=localhost
            - MYSQL_USERNAME=admin
            - MYSQL_PASSWORD=pointofsale
            - MYSQL_DB_NAME=ospos
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -16,7 +16,6 @@ services:
            - logs:/app/writable/logs
        environment:
            - CI_ENVIRONMENT=production
-            - ALLOWED_HOSTNAMES=localhost
            - FORCE_HTTPS=false
            - PHP_TIMEZONE=UTC
            - MYSQL_USERNAME=admin
--- a/gulpfile.js
+++ b/gulpfile.js
@@ -300,7 +300,6 @@ gulp.task('copy-menubar', function() {
 // Run all required tasks
 gulp.task('default',
    gulp.series('clean',
-        'update-licenses',
        'copy-bootswatch',
        'copy-bootswatch5',
        'copy-bootstrap',
--- a/package-lock.json
+++ b/package-lock.json
--- a/package.json
+++ b/package.json
@@ -1,33 +1,34 @@
 {
    "name": "@opensourcepos/opensourcepos",
    "version": "3.4.2",
-    "description": "Open Source Point of Sale is a web based point of sale system written in the PHP language. It uses MySQL as the data storage back-end and has a simple user interface.",
-    "keywords": [
-        "point-of-sale",
-        "POS"
-    ],
-    "homepage": "https://opensourcepos.org",
-    "bugs": {
-        "url": "https://github.com/opensourcepos/opensourcepos/issues"
-    },
-    "repository": {
-        "type": "git",
-        "url": "git+https://github.com/opensourcepos/opensourcepos.git"
-    },
+    "description": "pen Source Point of Sale is a web based point of sale system written in the PHP language. It uses MySQL as the data storage back-end and has a simple user interface.",
+    "main": "index.php",
    "license": "MIT",
-    "contributors": [
+    "authors": [
        "jekkos <jekkos - at - opensourcepos.org>",
-        "objecttothis <objecttothis - at - gmail.com>"
+        "FrancescoUK <francesco.lodolo.uk - at - gmail.com>",
+        "objecttothis <objecttothis - at - gmail.com>",
+        "SteveIreland <stevei - at - ruledomain.com>"
    ],
    "files": [
        "dist/opensourcepos.$version.tgz"
    ],
-    "type": "module",
-    "main": "index.php",
+    "publishConfig": {
+        "registry": "https://npm.pkg.github.com/"
+    },
+    "keywords": [
+        "point-of-sale",
+        "POS"
+    ],
+    "repository": {
+        "type": "git",
+        "url": "https://github.com/opensourcepos/opensourcepos"
+    },
    "scripts": {
        "build": "gulp default",
        "gulp": "gulp"
    },
+    "type": "module",
    "dependencies": {
        "bootstrap": "^3.4.1",
        "bootstrap-daterangepicker": "^2.1.27",
@@ -38,9 +39,9 @@
        "bootstrap-tagsinput-2021": "^0.8.6",
        "bootstrap-toggle": "^2.2.2",
        "bootstrap3-dialog": "github:nakupanda/bootstrap3-dialog#master",
-        "bootstrap5": "npm:bootstrap@^5.3.8",
+        "bootstrap5": "npm:bootstrap@^5.3.5",
        "bootswatch": "^3.4.1",
-        "bootswatch5": "npm:bootswatch@^5.3.8",
+        "bootswatch5": "npm:bootswatch@^5.3.5",
        "chartist": "^0.11.4",
        "chartist-plugin-axistitle": "^0.0.7",
        "chartist-plugin-barlabels": "^0.0.5",
@@ -48,7 +49,7 @@
        "chartist-plugin-tooltips": "^0.0.17",
        "clipboard": "^2.0.11",
        "coffeescript": "^2.7.0",
-        "dompurify": "^3.4.0",
+        "dompurify": "^3.3.2",
        "elegant-circles": "github:opensourcepos/elegant-circles#minified",
        "es6-promise": "^4.2.8",
        "file-saver": "^2.0.5",
@@ -63,26 +64,23 @@
        "tableexport.jquery.plugin": "^1.30.0"
    },
    "devDependencies": {
-        "gulp": "^5.0.1",
+        "gulp": "^5.0.0",
        "gulp-clean": "^0.4.0",
        "gulp-clean-css": "^4.3.0",
        "gulp-concat": "^2.6.1",
        "gulp-debug": "^5.0.1",
        "gulp-gzip": "^1.4.2",
-        "gulp-header": "^2.0.12",
+        "gulp-header": "^2.0.9",
        "gulp-inject": "^5.0.5",
-        "gulp-rename": "^2.1.0",
-        "gulp-rev": "^12.0.0",
+        "gulp-rename": "^2.0.0",
+        "gulp-rev": "^10.0.0",
        "gulp-run": "^1.7.1",
-        "gulp-tar": "^5.0.0",
+        "gulp-tar": "^4.0.0",
        "gulp-uglify": "^3.0.2",
        "gulp-zip": "^6.1.0",
-        "license-report": "^6.8.2",
-        "npm-check-updates": "^22.1.1",
+        "license-report": "^6.7.2",
+        "npm-check-updates": "^17.1.14",
        "readable-stream": "^4.4.2",
        "stream-series": "^0.1.1"
-    },
-    "publishConfig": {
-        "registry": "https://npm.pkg.github.com/"
    }
 }
--- a/public/index.php
+++ b/public/index.php
@@ -1,15 +1,12 @@
 <?php

-use CodeIgniter\Boot;
-use Config\Paths;
-
 /*
 *---------------------------------------------------------------
 * CHECK PHP VERSION
 *---------------------------------------------------------------
 */

-$minPhpVersion = '8.2'; // If you update this, don't forget to update `spark`.
+$minPhpVersion = '8.1'; // If you update this, don't forget to update `spark`.
 if (version_compare(PHP_VERSION, $minPhpVersion, '<')) {
    $message = sprintf(
        'Your PHP version must be %s or higher to run CodeIgniter. Current version: %s',
@@ -51,9 +48,9 @@ if (getcwd() . DIRECTORY_SEPARATOR !== FCPATH) {
 require FCPATH . '../app/Config/Paths.php';
 // ^^^ Change this line if you move your application folder

-$paths = new Paths();
+$paths = new Config\Paths();

 // LOAD THE FRAMEWORK BOOTSTRAP FILE
 require $paths->systemDirectory . '/Boot.php';

-exit(Boot::bootWeb($paths));
+exit(CodeIgniter\Boot::bootWeb($paths));
--- a/scripts/install-ubuntu.sh
+++ b/scripts/install-ubuntu.sh
@@ -1,381 +0,0 @@
-#!/bin/bash
-
-set -e
-
-COLOR_RED='\033[0;31m'
-COLOR_GREEN='\033[0;32m'
-COLOR_YELLOW='\033[1;33m'
-COLOR_BLUE='\033[0;34m'
-COLOR_RESET='\033[0m'
-
-echo -e "${COLOR_BLUE}╔══════════════════════════════════════════════════════════╗${COLOR_RESET}"
-echo -e "${COLOR_BLUE}║     Open Source Point of Sale - Ubuntu Installer        ║${COLOR_RESET}"
-echo -e "${COLOR_BLUE}║                    Version 3.4+                          ║${COLOR_RESET}"
-echo -e "${COLOR_BLUE}╚══════════════════════════════════════════════════════════╝${COLOR_RESET}"
-echo ""
-
-if [ "$EUID" -ne 0 ]; then
-    echo -e "${COLOR_RED}Please run this script as root or with sudo${COLOR_RESET}"
-    exit 1
-fi
-
-export DEBIAN_FRONTEND=noninteractive
-
-DB_HOST="${DB_HOST:-localhost}"
-DB_NAME="${DB_NAME:-ospos}"
-DB_USER="${DB_USER:-ospos}"
-DB_PASS="${DB_PASS:-$(openssl rand -base64 24)}"
-OSPOS_DIR="${OSPOS_DIR:-/var/www/ospos}"
-OSPOS_VERSION="${OSPOS_VERSION:-}"
-PHP_VERSION="${PHP_VERSION:-8.2}"
-APACHE_SERVER_NAME="${APACHE_SERVER_NAME:-}"
-SSL_EMAIL="${SSL_EMAIL:-}"
-SSL_DOMAIN="${SSL_DOMAIN:-}"
-MYSQL_ROOT_PASS="${MYSQL_ROOT_PASS:-}"
-
-# Validate database variables contain only safe characters (alphanumeric, underscore, hyphen, dot)
-validate_db_vars() {
-    local var_name="$1"
-    local var_value="$2"
-    local pattern='^[a-zA-Z0-9_\-\.]+$'
-    if [[ ! "$var_value" =~ $pattern ]]; then
-        echo -e "${COLOR_RED}Error: ${var_name} contains invalid characters. Only alphanumeric, underscore, hyphen, and dot are allowed.${COLOR_RESET}"
-        exit 1
-    fi
-}
-
-# Validate critical database variables
-validate_db_vars "DB_NAME" "$DB_NAME"
-validate_db_vars "DB_USER" "$DB_USER"
-validate_db_vars "DB_HOST" "$DB_HOST"
-
-# Check if running interactively
-INTERACTIVE=false
-if [ -t 0 ]; then
-    INTERACTIVE=true
-fi
-
-echo -e "${COLOR_YELLOW}Configuration:${COLOR_RESET}"
-echo -e "  Database Name: ${DB_NAME}"
-echo -e "  Database User: ${DB_USER}"
-echo -e "  Database Host: ${DB_HOST}"
-echo -e "  Install Directory: ${OSPOS_DIR}"
-echo -e "  PHP Version: ${PHP_VERSION}"
-if [ -n "$OSPOS_VERSION" ]; then
-    echo -e "  OSPOS Version: ${OSPOS_VERSION}"
-else
-    echo -e "  OSPOS Version: latest"
-fi
-if [ -n "$APACHE_SERVER_NAME" ]; then
-    echo -e "  Server Name: ${APACHE_SERVER_NAME}"
-fi
-echo ""
-
-if [ -d "$OSPOS_DIR" ]; then
-    echo -e "${COLOR_RED}Installation directory $OSPOS_DIR already exists${COLOR_RESET}"
-    echo -e "${COLOR_YELLOW}Remove it or set OSPOS_DIR environment variable${COLOR_RESET}"
-    exit 1
-fi
-
-echo -e "${COLOR_GREEN}[1/9] Updating system packages...${COLOR_RESET}"
-apt-get update -qq
-
-echo -e "${COLOR_GREEN}[2/9] Installing Apache, PHP, and dependencies...${COLOR_RESET}"
-# Add PHP repository for newer PHP versions if not available in default repos
-if ! apt-cache policy php${PHP_VERSION} 2>/dev/null | grep -q "Candidate:"; then
-    echo -e "${COLOR_YELLOW}PHP ${PHP_VERSION} not in default repos, adding ondrej/php PPA...${COLOR_RESET}"
-    apt-get install -y -qq software-properties-common
-    add-apt-repository -y ppa:ondrej/php
-    apt-get update -qq
-fi
-
-apt-get install -y -qq \
-    apache2 \
-    mariadb-server \
-    mariadb-client \
-    php${PHP_VERSION} \
-    php${PHP_VERSION}-mysql \
-    php${PHP_VERSION}-gd \
-    php${PHP_VERSION}-bcmath \
-    php${PHP_VERSION}-intl \
-    php${PHP_VERSION}-mbstring \
-    php${PHP_VERSION}-curl \
-    php${PHP_VERSION}-xml \
-    php${PHP_VERSION}-zip \
-    git \
-    curl \
-    unzip \
-    openssl
-
-echo -e "${COLOR_GREEN}[3/9] Starting MariaDB...${COLOR_RESET}"
-systemctl start mariadb
-systemctl enable mariadb
-
-if [ -z "$MYSQL_ROOT_PASS" ]; then
-    echo -e "${COLOR_BLUE}Securing MariaDB installation...${COLOR_RESET}"
-    mysql -e "ALTER USER 'root'@'localhost' IDENTIFIED BY '';"
-    mysql -e "FLUSH PRIVILEGES;"
-else
-    mysql -e "ALTER USER 'root'@'localhost' IDENTIFIED BY '${MYSQL_ROOT_PASS}';"
-fi
-
-echo -e "${COLOR_GREEN}[4/9] Creating database and user...${COLOR_RESET}"
-if [ -n "$MYSQL_ROOT_PASS" ]; then
-    mysql -u root -p"${MYSQL_ROOT_PASS}" <<EOF
-CREATE DATABASE IF NOT EXISTS ${DB_NAME} CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci;
-CREATE USER IF NOT EXISTS '${DB_USER}'@'${DB_HOST}' IDENTIFIED BY '${DB_PASS}';
-GRANT ALL PRIVILEGES ON ${DB_NAME}.* TO '${DB_USER}'@'${DB_HOST}';
-FLUSH PRIVILEGES;
-EOF
-else
-    mysql -u root <<EOF
-CREATE DATABASE IF NOT EXISTS ${DB_NAME} CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci;
-CREATE USER IF NOT EXISTS '${DB_USER}'@'${DB_HOST}' IDENTIFIED BY '${DB_PASS}';
-GRANT ALL PRIVILEGES ON ${DB_NAME}.* TO '${DB_USER}'@'${DB_HOST}';
-FLUSH PRIVILEGES;
-EOF
-fi
-
-echo -e "${COLOR_GREEN}[5/9] Downloading OSPOS...${COLOR_RESET}"
-mkdir -p "$(dirname "$OSPOS_DIR")"
-cd "$(dirname "$OSPOS_DIR")"
-
-if [ -z "$OSPOS_VERSION" ]; then
-    OSPOS_VERSION=$(curl -sS https://api.github.com/repos/opensourcepos/opensourcepos/releases/latest | grep '"tag_name":' | sed -E 's/.*"([^"]+)".*/\1/')
-    if [ -z "$OSPOS_VERSION" ]; then
-        echo -e "${COLOR_RED}Failed to get latest release version${COLOR_RESET}"
-        exit 1
-    fi
-fi
-
-echo -e "${COLOR_BLUE}Downloading OSPOS version ${OSPOS_VERSION}...${COLOR_RESET}"
-ASSET_URL=$(curl -sS "https://api.github.com/repos/opensourcepos/opensourcepos/releases/tags/${OSPOS_VERSION}" | grep '"browser_download_url"' | head -1 | sed -E 's/.*"([^"]+)".*/\1/')
-
-if [ -z "$ASSET_URL" ]; then
-    echo -e "${COLOR_RED}Failed to find release asset for ${OSPOS_VERSION}${COLOR_RESET}"
-    exit 1
-fi
-
-curl -sSL "$ASSET_URL" -o ospos.zip
-
-if [ ! -f ospos.zip ] || [ ! -s ospos.zip ]; then
-    echo -e "${COLOR_RED}Failed to download OSPOS release ${OSPOS_VERSION}${COLOR_RESET}"
-    rm -f ospos.zip
-    exit 1
-fi
-
-unzip -q ospos.zip -d ospos-temp
-mkdir -p "${OSPOS_DIR}"
-cp -r ospos-temp/. "${OSPOS_DIR}/"
-rm -rf ospos-temp ospos.zip
-
-echo -e "${COLOR_GREEN}Downloaded OSPOS ${OSPOS_VERSION}${COLOR_RESET}"
-
-echo -e "${COLOR_GREEN}[6/9] Setting up OSPOS...${COLOR_RESET}"
-cd "${OSPOS_DIR}"
-
-curl -sS https://getcomposer.org/installer | php -- --install-dir=/usr/local/bin --filename=composer 2>/dev/null
-
-if [ -f "composer.json" ]; then
-    echo -e "${COLOR_BLUE}Installing dependencies...${COLOR_RESET}"
-    composer install --no-dev --optimize-autoloader --no-interaction --quiet 2>/dev/null
-fi
-
-echo -e "${COLOR_GREEN}[7/9] Configuring OSPOS...${COLOR_RESET}"
-if [ -f ".env.example" ]; then
-    cp .env.example .env
-fi
-
-if [ -f ".env" ]; then
-    # Escape special characters in password for sed
-    ESCAPED_DB_PASS=$(printf '%s\n' "$DB_PASS" | sed 's/[&/\]/\\&/g')
-    
-    sed -i "s|database\.default\.hostname = 'localhost'|database.default.hostname = '${DB_HOST}'|" .env
-    sed -i "s|database\.default\.database = 'ospos'|database.default.database = '${DB_NAME}'|" .env
-    sed -i "s|database\.default\.username = 'admin'|database.default.username = '${DB_USER}'|" .env
-    sed -i "s|database\.default\.password = 'pointofsale'|database.default.password = '${ESCAPED_DB_PASS}'|" .env
-    sed -i "s|CI_ENVIRONMENT = development|CI_ENVIRONMENT = production|" .env
-    
-    if grep -q "encryption\.key = ''" .env; then
-        ENCRYPTION_KEY=$(openssl rand -base64 32)
-        ESCAPED_KEY=$(printf '%s\n' "$ENCRYPTION_KEY" | sed 's/[&/\]/\\&/g')
-        sed -i "s|encryption\.key = ''|encryption.key = '${ESCAPED_KEY}'|" .env
-    fi
-fi
-
-echo -e "${COLOR_GREEN}[8/9] Importing database schema...${COLOR_RESET}"
-if [ -n "$MYSQL_ROOT_PASS" ]; then
-    mysql -u root -p"${MYSQL_ROOT_PASS}" ${DB_NAME} < app/Database/database.sql
-else
-    mysql -u root ${DB_NAME} < app/Database/database.sql
-fi
-
-# Interactive SSL configuration
-if $INTERACTIVE && [ -z "$SSL_EMAIL" ] && [ -z "$APACHE_SERVER_NAME" ]; then
-    echo ""
-    echo -e "${COLOR_BLUE}╔══════════════════════════════════════════════════════════╗${COLOR_RESET}"
-    echo -e "${COLOR_BLUE}║            SSL/TLS Configuration                          ║${COLOR_RESET}"
-    echo -e "${COLOR_BLUE}╚══════════════════════════════════════════════════════════╝${COLOR_RESET}"
-    echo ""
-    echo -e "${COLOR_YELLOW}SSL provides secure HTTPS access to your OSPOS installation.${COLOR_RESET}"
-    echo -e "${COLOR_YELLOW}For production, we recommend Let's Encrypt (free SSL certificate).${COLOR_RESET}"
-    echo ""
-    
-    read -p "Configure SSL? (y/n) [n]: " CONFIGURE_SSL
-    CONFIGURE_SSL=${CONFIGURE_SSL:-n}
-    
-    if [[ "$CONFIGURE_SSL" =~ ^[Yy]$ ]]; then
-        read -p "Enter your domain name (e.g., pos.example.com): " SSL_DOMAIN
-        SSL_DOMAIN=${SSL_DOMAIN:-localhost}
-        APACHE_SERVER_NAME=$SSL_DOMAIN
-        
-        read -p "Enter your email for Let's Encrypt notifications: " SSL_EMAIL
-        
-        if [ -z "$SSL_EMAIL" ]; then
-            echo -e "${COLOR_YELLOW}No email provided. Using self-signed certificate (not recommended for production).${COLOR_RESET}"
-            SSL_TYPE="self-signed"
-        else
-            SSL_TYPE="letsencrypt"
-        fi
-    else
-        APACHE_SERVER_NAME="localhost"
-        SSL_TYPE="none"
-    fi
-fi
-
-# Set default server name if not provided
-if [ -z "$APACHE_SERVER_NAME" ]; then
-    APACHE_SERVER_NAME="localhost"
-fi
-
-# If SSL_EMAIL is set without SSL_DOMAIN, use APACHE_SERVER_NAME
-if [ -n "$SSL_EMAIL" ] && [ -z "$SSL_DOMAIN" ] && [ "$APACHE_SERVER_NAME" != "localhost" ]; then
-    SSL_DOMAIN="$APACHE_SERVER_NAME"
-fi
-
-echo -e "${COLOR_GREEN}[9/9] Configuring Apache...${COLOR_RESET}"
-cat > /etc/apache2/sites-available/ospos.conf <<EOF
-<VirtualHost *:80>
-    ServerName ${APACHE_SERVER_NAME}
-    DocumentRoot ${OSPOS_DIR}/public
-
-    <Directory ${OSPOS_DIR}/public>
-        Options -Indexes +FollowSymLinks
-        AllowOverride All
-        Require all granted
-    </Directory>
-
-    ErrorLog \${APACHE_LOG_DIR}/ospos_error.log
-    CustomLog \${APACHE_LOG_DIR}/ospos_access.log combined
-</VirtualHost>
-EOF
-
-a2enmod rewrite
-a2dissite 000-default.conf
-a2ensite ospos.conf
-
-chown -R www-data:www-data "${OSPOS_DIR}"
-chmod -R 750 "${OSPOS_DIR}/writable"
-
-systemctl restart apache2
-systemctl enable apache2
-
-# Configure SSL if requested
-if [ -n "$SSL_EMAIL" ] && [ -n "$SSL_DOMAIN" ]; then
-    # Let's Encrypt SSL
-    echo -e "${COLOR_BLUE}Installing Certbot for Let's Encrypt...${COLOR_RESET}"
-    apt-get install -y -qq certbot python3-certbot-apache
-    
-    echo -e "${COLOR_BLUE}Obtaining SSL certificate for ${SSL_DOMAIN}...${COLOR_RESET}"
-    certbot --apache -d ${SSL_DOMAIN} --non-interactive --agree-tos --email ${SSL_EMAIL} --redirect
-    
-    echo -e "${COLOR_BLUE}Setting up auto-renewal...${COLOR_RESET}"
-    systemctl enable certbot.timer
-    systemctl start certbot.timer
-    
-    PROTOCOL="https"
-    FINAL_URL="https://${SSL_DOMAIN}/"
-elif [ -n "$SSL_DOMAIN" ]; then
-    # Self-signed SSL
-    echo -e "${COLOR_BLUE}Generating self-signed SSL certificate...${COLOR_RESET}"
-    openssl req -x509 -nodes -days 365 -newkey rsa:2048 \
-        -keyout /etc/ssl/private/ospos-selfsigned.key \
-        -out /etc/ssl/certs/ospos-selfsigned.crt \
-        -subj "/C=US/ST=State/L=City/O=Organization/CN=${SSL_DOMAIN}" 2>/dev/null
-    
-    cat > /etc/apache2/sites-available/ospos-ssl.conf <<EOF
-<VirtualHost *:443>
-    ServerName ${SSL_DOMAIN}
-    DocumentRoot ${OSPOS_DIR}/public
-
-    SSLEngine on
-    SSLCertificateFile /etc/ssl/certs/ospos-selfsigned.crt
-    SSLCertificateKeyFile /etc/ssl/private/ospos-selfsigned.key
-
-    <Directory ${OSPOS_DIR}/public>
-        Options -Indexes +FollowSymLinks
-        AllowOverride All
-        Require all granted
-    </Directory>
-
-    ErrorLog \${APACHE_LOG_DIR}/ospos_ssl_error.log
-    CustomLog \${APACHE_LOG_DIR}/ospos_ssl_access.log combined
-</VirtualHost>
-EOF
-    
-    a2enmod ssl
-    a2ensite ospos-ssl.conf
-    
-    cat > /etc/apache2/sites-available/ospos.conf <<EOF
-<VirtualHost *:80>
-    ServerName ${SSL_DOMAIN}
-    Redirect permanent / https://${SSL_DOMAIN}/
-</VirtualHost>
-EOF
-    
-    a2dissite ospos.conf
-    a2ensite ospos.conf
-    
-    PROTOCOL="https"
-    FINAL_URL="https://${SSL_DOMAIN}/"
-    
-    echo -e "${COLOR_YELLOW}Note: Your browser will show a security warning for self-signed${COLOR_RESET}"
-    echo -e "${COLOR_YELLOW}      certificates. For production, re-run with an email for Let's Encrypt.${COLOR_RESET}"
-else
-    PROTOCOL="http"
-    FINAL_URL="http://${APACHE_SERVER_NAME}/"
-fi
-
-systemctl restart apache2
-
-# Configure allowed hostnames
-if [ -f "${OSPOS_DIR}/.env" ]; then
-    sed -i "s|app\.allowedHostnames = ''|app.allowedHostnames = '${APACHE_SERVER_NAME}'|" ${OSPOS_DIR}/.env
-fi
-
-echo ""
-echo -e "${COLOR_GREEN}╔══════════════════════════════════════════════════════════╗${COLOR_RESET}"
-echo -e "${COLOR_GREEN}║            Installation Complete!                         ║${COLOR_RESET}"
-echo -e "${COLOR_GREEN}╚══════════════════════════════════════════════════════════╝${COLOR_RESET}"
-echo ""
-echo -e "${COLOR_YELLOW}Database Credentials:${COLOR_RESET}"
-echo -e "  Database: ${DB_NAME}"
-echo -e "  Username: ${DB_USER}"
-echo -e "  Password: ${DB_PASS}"
-echo ""
-echo -e "${COLOR_YELLOW}Login Credentials:${COLOR_RESET}"
-echo -e "  URL:      ${FINAL_URL}"
-if [ -n "$SSL_EMAIL" ]; then
-    echo -e "  SSL:      Let's Encrypt (auto-renewal enabled)"
-elif [ -n "$SSL_DOMAIN" ]; then
-    echo -e "  SSL:      Self-signed certificate"
-else
-    echo -e "  SSL:      Not configured (HTTP only)"
-fi
-echo -e "  Username: admin"
-echo -e "  Password: pointofsale"
-echo ""
-echo -e "${COLOR_RED}IMPORTANT: Change the default password after first login!${COLOR_RESET}"
-echo ""
-echo -e "${COLOR_BLUE}Configuration file: ${OSPOS_DIR}/.env${COLOR_RESET}"
-echo ""
--- a/9
+++ b/9
@@ -10,9 +10,6 @@
 * the LICENSE file that was distributed with this source code.
 */

-use CodeIgniter\Boot;
-use Config\Paths;
-
 /*
 * --------------------------------------------------------------------
 * CODEIGNITER COMMAND-LINE TOOLS
@@ -38,7 +35,7 @@ if (str_starts_with(PHP_SAPI, 'cgi')) {
 *---------------------------------------------------------------
 */

-$minPhpVersion = '8.2'; // If you update this, don't forget to update `public/index.php`.
+$minPhpVersion = '8.1'; // If you update this, don't forget to update `public/index.php`.
 if (version_compare(PHP_VERSION, $minPhpVersion, '<')) {
    $message = sprintf(
        'Your PHP version must be %s or higher to run CodeIgniter. Current version: %s',
@@ -79,9 +76,9 @@ chdir(FCPATH);
 require FCPATH . '../app/Config/Paths.php';
 // ^^^ Change this line if you move your application folder

-$paths = new Paths();
+$paths = new Config\Paths();

 // LOAD THE FRAMEWORK BOOTSTRAP FILE
 require $paths->systemDirectory . '/Boot.php';

-exit(Boot::bootSpark($paths));
+exit(CodeIgniter\Boot::bootSpark($paths));
--- a/tests/Config/AppTest.php
+++ b/tests/Config/AppTest.php
@@ -18,7 +18,6 @@ class AppTest extends CIUnitTestCase
        // Clean up environment
        putenv('CI_ENVIRONMENT');
        putenv('app.allowedHostnames');
-        putenv('ALLOWED_HOSTNAMES');
        unset($_SERVER['HTTP_HOST']);
    }

@@ -282,106 +281,4 @@ class AppTest extends CIUnitTestCase
        putenv('app.allowedHostnames');
        putenv('CI_ENVIRONMENT');
    }
-
-    public function testAllowedHostnamesEnvVarParsedAsCommaSeparated(): void
-    {
-        // Set ALLOWED_HOSTNAMES environment variable
-        putenv('ALLOWED_HOSTNAMES=example.com,www.example.com,demo.example.com');
-
-        $_SERVER['HTTP_HOST'] = 'www.example.com';
-        $_SERVER['SCRIPT_NAME'] = '/index.php';
-        $_SERVER['HTTPS'] = null;
-
-        $app = new App();
-
-        // Constructor should parse comma-separated values
-        $this->assertEquals(['example.com', 'www.example.com', 'demo.example.com'], $app->allowedHostnames);
-        $this->assertStringContainsString('www.example.com', $app->baseURL);
-
-        // Clean up
-        putenv('ALLOWED_HOSTNAMES');
-    }
-
-    public function testAllowedHostnamesEnvVarTakesPrecedenceOverDotEnv(): void
-    {
-        // Set both environment variables
-        putenv('ALLOWED_HOSTNAMES=allowed1.com,allowed2.com');
-        putenv('app.allowedHostnames=dotenv1.com,dotenv2.com');
-
-        $_SERVER['HTTP_HOST'] = 'allowed1.com';
-        $_SERVER['SCRIPT_NAME'] = '/index.php';
-        $_SERVER['HTTPS'] = null;
-
-        $app = new App();
-
-        // ALLOWED_HOSTNAMES should take precedence
-        $this->assertEquals(['allowed1.com', 'allowed2.com'], $app->allowedHostnames);
-        $this->assertStringContainsString('allowed1.com', $app->baseURL);
-
-        // Clean up
-        putenv('ALLOWED_HOSTNAMES');
-        putenv('app.allowedHostnames');
-    }
-
-    public function testAllowedHostnamesEnvVarFallsBackToDotEnv(): void
-    {
-        // Only set app.allowedHostnames, not ALLOWED_HOSTNAMES
-        putenv('app.allowedHostnames=dotenv1.com,dotenv2.com');
-
-        $_SERVER['HTTP_HOST'] = 'dotenv1.com';
-        $_SERVER['SCRIPT_NAME'] = '/index.php';
-        $_SERVER['HTTPS'] = null;
-
-        $app = new App();
-
-        // Should fall back to app.allowedHostnames
-        $this->assertEquals(['dotenv1.com', 'dotenv2.com'], $app->allowedHostnames);
-        $this->assertStringContainsString('dotenv1.com', $app->baseURL);
-
-        // Clean up
-        putenv('app.allowedHostnames');
-    }
-
-    public function testAllowedHostnamesEnvVarTrimmedWhitespace(): void
-    {
-        // Set environment variable with whitespace
-        putenv('ALLOWED_HOSTNAMES= example.com , www.example.com , demo.example.com ');
-
-        $_SERVER['HTTP_HOST'] = 'example.com';
-        $_SERVER['SCRIPT_NAME'] = '/index.php';
-        $_SERVER['HTTPS'] = null;
-
-        $app = new App();
-
-        // Values should be trimmed
-        $this->assertEquals(['example.com', 'www.example.com', 'demo.example.com'], $app->allowedHostnames);
-
-        // Clean up
-        putenv('ALLOWED_HOSTNAMES');
-    }
-
-    public function testAllowedHostnamesEnvVarFiltersEmptyEntries(): void
-    {
-        // Trailing comma should not produce empty entry
-        putenv('ALLOWED_HOSTNAMES=example.com,');
-        $_SERVER['HTTP_HOST'] = 'example.com';
-        $_SERVER['SCRIPT_NAME'] = '/index.php';
-        $_SERVER['HTTPS'] = null;
-
-        $app = new App();
-        $this->assertEquals(['example.com'], $app->allowedHostnames);
-
-        // Clean up
-        putenv('ALLOWED_HOSTNAMES');
-
-        // Whitespace-only entry should be filtered
-        putenv('ALLOWED_HOSTNAMES=example.com, ,www.example.com');
-        $_SERVER['HTTP_HOST'] = 'example.com';
-
-        $app = new App();
-        $this->assertEquals(['example.com', 'www.example.com'], $app->allowedHostnames);
-
-        // Clean up
-        putenv('ALLOWED_HOSTNAMES');
-    }
 }
--- a/tests/Controllers/ItemsCsvImportTest.php
+++ b/tests/Controllers/ItemsCsvImportTest.php
@@ -237,7 +237,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $row = $this->db->table('items')
            ->where('item_number', $itemData['item_number'])
@@ -268,7 +268,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $locationId = 1;
        $quantity = 100;
@@ -298,7 +298,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $inventoryData = [
            'trans_inventory' => 50,
@@ -329,7 +329,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $taxesData = [
            ['name' => 'VAT', 'percent' => 20],
@@ -406,7 +406,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
                'deleted' => false
            ];

-            $this->assertTrue($this->item->save_value($itemData));
+            $this->assertTrue($this->item->saveValue($itemData));
        }

        $item1 = $this->item->get_info_by_id_or_number('ITEM-A');
@@ -430,7 +430,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($originalData));
+        $this->assertTrue($this->item->saveValue($originalData));

        $updatedData = [
            'item_id' => $originalData['item_id'],
@@ -443,7 +443,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($updatedData, $updatedData['item_id']));
+        $this->assertTrue($this->item->saveValue($updatedData));

        $updatedItem = $this->item->get_info($updatedData['item_id']);
        $this->assertEquals('Updated Name', $updatedItem->name);
@@ -464,7 +464,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($originalData));
+        $this->assertTrue($this->item->saveValue($originalData));

        $definitionData = [
            'definition_name' => 'Color',
@@ -510,7 +510,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $definitionData = [
            'definition_name' => 'Color',
@@ -553,7 +553,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        // Mock Attribute DROPDOWN
        $definitionData = [
@@ -604,7 +604,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $locationId = 1;

@@ -633,7 +633,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $savedItem = $this->item->get_info($itemData['item_id']);
        $this->assertEquals(-1, (int)$savedItem->reorder_level);
@@ -672,7 +672,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
                'deleted' => 0
            ];

-            $this->assertTrue($this->item->save_value($itemData));
+            $this->assertTrue($this->item->saveValue($itemData));

            $savedItem = $this->item->get_info($itemData['item_id']);

@@ -702,7 +702,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $savedItem = $this->item->get_info($itemData['item_id']);
        $this->assertEquals('8471', $savedItem->hsn_code);
@@ -719,7 +719,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $locations = [
            'Warehouse' => 100,
@@ -792,7 +792,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $this->assertIsInt($itemData['item_id']);
        $this->assertGreaterThan(0, $itemData['item_id']);
@@ -812,7 +812,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $exists = $this->item->exists($itemData['item_id']);
        $this->assertTrue($exists);
@@ -858,7 +858,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $taxesData = [];
        if (is_numeric($csvRow['Tax 1 Percent']) && $csvRow['Tax 1 Name'] !== '') {
@@ -1032,7 +1032,7 @@ class ItemsCsvImportTest extends CIUnitTestCase
            'deleted' => 0
        ];

-        $this->assertTrue($this->item->save_value($itemData));
+        $this->assertTrue($this->item->saveValue($itemData));

        $uniqueId = uniqid();
        $locations = ['Warehouse' . $uniqueId, 'Store' . $uniqueId];
Author	SHA1	Message	Date
Ollama	196d1e4d3a	fix: Remove deleted filter and primary key from insert - Remove deleted=0 filter from existence check (allow soft-deleted updates) - Remove primary key from insert payload to avoid conflicts - Cleaner approach for upsert logic Address CodeRabbit review feedback	2026-04-15 17:08:20 +00:00
Ollama	a4c0d081a2	fix: Use primary key only check and atomic insert in saveValue - Replace exists() with direct primary key check to avoid matching by other identifiers - Wrap insert + low_sell_item_id update in transaction for atomicity - Check db insert result and rollback on failure Address CodeRabbit review feedback	2026-04-15 16:01:48 +00:00
Ollama	e7daa7a9db	fix: Remove primary key from update payload - Unset item_id from data array before update - Cleaner approach to avoid including PK in update payload Address CodeRabbit review feedback	2026-04-15 15:29:43 +00:00
Ollama	baf135dd42	refactor: unify Item model save_value signature - Rename save_value() to saveValue() for PSR compliance - Remove second parameter (item_id) - now derived from data array - Check for item_id in data to determine insert vs update - Update all call sites in Items controller - Update test file references Part of #4459	2026-04-15 15:10:41 +00:00