Skip to content
This repository has been archived by the owner on Nov 8, 2021. It is now read-only.

php Fatal error 004_HostedPaymentPage.php #17

Open
stxxo opened this issue Sep 5, 2019 · 13 comments
Open

php Fatal error 004_HostedPaymentPage.php #17

stxxo opened this issue Sep 5, 2019 · 13 comments

Comments

@stxxo
Copy link

stxxo commented Sep 5, 2019

Fatal error: Method BTCPayServer\Client\Request::__toString() must not throw an exception, caught Error: Call to undefined method BTCPayServer\Client\Request::getSchema() in /var/www/clients/client0/web3/web/myhost/btcpay/004_HostedPaymentPage.php on line 0

error in file BTCPayServer/Client/Request.php
Workaround: insert this code in file Request.php

    public function getSchema()
    {
        return $this->scheme;
    }

@woutersamaey
Copy link
Collaborator

woutersamaey commented Sep 5, 2019

This method already exists. Are you using the latest code from the master branch?

@stxxo
Copy link
Author

stxxo commented Sep 5, 2019

This method already exists. Are you using the latest code from the master branch?

I've installed the btcpayserver-php-client a few hours ago with composer.
Just have checked my files again "public function getSchema" is missing.

@woutersamaey
Copy link
Collaborator

The method does exit. Look at

@stxxo
Copy link
Author

stxxo commented Sep 5, 2019

The method does exit. Look at

ah, yes! Please search for "getSchema" and "getScheme", a typo in the last character
Didnt have seen it :)

@psyb0t
Copy link
Contributor

psyb0t commented Sep 5, 2019

@woutersamaey do you have a packagist account? I wanna add you as a maintainer so you can update.

@woutersamaey
Copy link
Collaborator

@psyb0t yes, my username is woutersamaey

@woutersamaey
Copy link
Collaborator

@psyb0t can you approve this access request? It's needed so we can connect Packagist to Github.
image

@psyb0t
Copy link
Contributor

psyb0t commented Sep 5, 2019

@woutersamaey nope.
@NicolasDorier ?

@woutersamaey
Copy link
Collaborator

Packagist says the repo needs an update to use the new GitHub hook system. For this to work, the organization needs to approve access.
image

@NicolasDorier
Copy link
Member

I added the authorization.

@woutersamaey
Copy link
Collaborator

Thanks @NicolasDorier
Sync is working now.

@RedLicorice
Copy link

This has not yet been fixed.
Using v 1.0 on a Laravel project.

@MarianMatys
Copy link

I've just installed the package using composer to localhost and live server and in both cases, the typo is still there ("getSchemA"). I know it's fixed on Github. How is this even possible? Could someone please explain?

I've fixed it manually, but I'm worried it will come back during an update in the future - obviously, I will not remember it:)

Thank you!

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

No branches or pull requests

6 participants