Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update ophandlers.php #283

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

Conversation

frank42hh
Copy link

updateAddress requires exactly 4 options, the call within importPTRData only submits 3, which results in this error:

AH01071: Got error 'PHP message: PHP Fatal error: Uncaught ArgumentCountError: Too few arguments to function updateAddress(), 3 passed in /var/www/htdocs/RackTables/wwwroot/inc/ophandlers.php on line 2178 and exactly 4 expected in /var/www/htdocs/RackTables/wwwroot/inc/database.php:2600\nStack trace:\n#0 /var/www/htdocs/RackTables/wwwroot/inc/ophandlers.php(2178): updateAddress('\xD4\f6\x02', 'reserve-router-...', 'no')\n#1 /var/www/htdocs/RackTables/wwwroot/index.php(232): importPTRData()\n#2 /var/www/htdocs/rackspace/index.php(11): require('/var/www/htdocs...')\n#3 {main}\n thrown in /var/www/htdocs/RackTables/wwwroot/inc/database.php on line 2600'

updateAddress requires exactly 4 options, the call within importPTRData only submits 3, which results in this error:

AH01071: Got error 'PHP message: PHP Fatal error:  Uncaught ArgumentCountError: Too few arguments to function updateAddress(), 3 passed in /var/www/htdocs/RackTables/wwwroot/inc/ophandlers.php on line 2178 and exactly 4 expected in /var/www/htdocs/RackTables/wwwroot/inc/database.php:2600\nStack trace:\n#0 /var/www/htdocs/RackTables/wwwroot/inc/ophandlers.php(2178): updateAddress('\\xD4\\f6\\x02', 'reserve-router-...', 'no')\n#1 /var/www/htdocs/RackTables/wwwroot/index.php(232): importPTRData()\n#2 /var/www/htdocs/rackspace/index.php(11): require('/var/www/htdocs...')\n#3 {main}\n  thrown in /var/www/htdocs/RackTables/wwwroot/inc/database.php on line 2600'
@infrastation
Copy link
Member

Thank you for preparing these changes Frank. Is this on PHP 8? Which version?

@frank42hh
Copy link
Author

I'm running on php 7.3 currently. Didn't have the time to update to 8.x yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants