Skip to content

Revised DashboardConfig.php fix for PHP 8.3 (and some housekeeping)#951

Merged
timschofield merged 6 commits into
timschofield:masterfrom
dalers:main
Jul 8, 2026
Merged

Revised DashboardConfig.php fix for PHP 8.3 (and some housekeeping)#951
timschofield merged 6 commits into
timschofield:masterfrom
dalers:main

Conversation

@dalers

@dalers dalers commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator
  1. fixes DashboardConfig.php failing on PHP 8.3.6 due to uninitialized array variable "ScriptArray" (fwiw ok with 8.2.32)
  2. prevent incorrect 2nd insert being performed resulting in an SQL duplicate key error (hidden using production debug settings)
Function Call StackFrame File Line Function Class Arguments
0 /home/dale/weberp/includes/ConnectDB_mariadb.php 82 ShowDebugBackTrace N/A
Array
(
    [0] => The SQL that failed was
    [1] => INSERT INTO scripts (script,
                                pagesecurity,
                                description
                            ) VALUES (
                                'mrp_dashboard.php',
                                '1',
                                'MRP Status'
                            )
)

1/home/dale/weberp/DashboardConfig.php 70 DB_query N/A
Array
(
    [0] => INSERT INTO scripts (script,
                                pagesecurity,
                                description
                            ) VALUES (
                                'mrp_dashboard.php',
                                '1',
                                'MRP Status'
                            )
)
  1. git ignore Windows/WSL files and fonts created JIT by DOMPDF
  2. added Copilot direction file for developing using VS Code on WSL Ubuntu (free plan for GitHub Copilot provides "2,000 completions per month" whatever that means).

@timschofield timschofield merged commit aeb0757 into timschofield:master Jul 8, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants