Callback Data

[idvlab]

Hi.

The documentation says:

Callback query data must be defined with the following structure:

action:action_name;key1:foo;key2:bar

and will be handled by a public action_name method inside a custom webhook handler.

But, for example, I have a method for callback data:

public function test()
{
    $this->reply('Text', true);
}

If I type the /test command in the chat, it will cause a fatal error. How can this be avoided?

Is there another way to handle the callback function? For example:

match ($action) {
    'test' => MyClass::myMethod(),
    default => $this->reply('Invalid action');
}

[fabio-ivona]

Hi, could you paste here the error stacktrace?

[idvlab]

Of course:

Error stacktrace

[2023-03-07 21:15:22] local.DEBUG: Telegraph webhook message {"text":"/test"} 
[2023-03-07 21:15:22] local.ERROR: Typed property DefStudio\Telegraph\Handlers\WebhookHandler::$callbackQueryId must not be accessed before initialization {"exception":"[object] (Error(code: 0): Typed property DefStudio\\Telegraph\\Handlers\\WebhookHandler::$callbackQueryId must not be accessed before initialization at /vendor/defstudio/telegraph/src/Handlers/WebhookHandler.php:205)
[stacktrace]
#0 /app/Http/Webhooks/TelegramMain.php(91): DefStudio\\Telegraph\\Handlers\\WebhookHandler->reply()
#1 /vendor/defstudio/telegraph/src/Handlers/WebhookHandler.php(81): App\\Http\\Webhooks\\TelegramMain->test()
#2 /vendor/defstudio/telegraph/src/Handlers/WebhookHandler.php(108): DefStudio\\Telegraph\\Handlers\\WebhookHandler->handleCommand()
#3 /vendor/defstudio/telegraph/src/Handlers/WebhookHandler.php(222): DefStudio\\Telegraph\\Handlers\\WebhookHandler->handleMessage()
#4 /vendor/defstudio/telegraph/src/Controllers/WebhookController.php(28): DefStudio\\Telegraph\\Handlers\\WebhookHandler->handle()
#5 /vendor/laravel/framework/src/Illuminate/Routing/ControllerDispatcher.php(46): DefStudio\\Telegraph\\Controllers\\WebhookController->handle()
#6 /vendor/laravel/framework/src/Illuminate/Routing/Route.php(260): Illuminate\\Routing\\ControllerDispatcher->dispatch()
#7 /vendor/laravel/framework/src/Illuminate/Routing/Route.php(205): Illuminate\\Routing\\Route->runController()
#8 /vendor/laravel/framework/src/Illuminate/Routing/Router.php(798): Illuminate\\Routing\\Route->run()
#9 /vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(141): Illuminate\\Routing\\Router->Illuminate\\Routing\\{closure}()
#10 /vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(116): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}()
#11 /vendor/laravel/framework/src/Illuminate/Routing/Router.php(799): Illuminate\\Pipeline\\Pipeline->then()
#12 /vendor/laravel/framework/src/Illuminate/Routing/Router.php(776): Illuminate\\Routing\\Router->runRouteWithinStack()
#13 /vendor/laravel/framework/src/Illuminate/Routing/Router.php(740): Illuminate\\Routing\\Router->runRoute()
#14 /vendor/laravel/framework/src/Illuminate/Routing/Router.php(729): Illuminate\\Routing\\Router->dispatchToRoute()
#15 /vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(190): Illuminate\\Routing\\Router->dispatch()
#16 /vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(141): Illuminate\\Foundation\\Http\\Kernel->Illuminate\\Foundation\\Http\\{closure}()
#17 /vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/TransformsRequest.php(21): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}()
#18 /vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/ConvertEmptyStringsToNull.php(31): Illuminate\\Foundation\\Http\\Middleware\\TransformsRequest->handle()
#19 /vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(180): Illuminate\\Foundation\\Http\\Middleware\\ConvertEmptyStringsToNull->handle()
#20 /vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/TransformsRequest.php(21): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}()
#21 /vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/TrimStrings.php(40): Illuminate\\Foundation\\Http\\Middleware\\TransformsRequest->handle()
#22 /vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(180): Illuminate\\Foundation\\Http\\Middleware\\TrimStrings->handle()
#23 /vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/ValidatePostSize.php(27): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}()
#24 /vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(180): Illuminate\\Foundation\\Http\\Middleware\\ValidatePostSize->handle()
#25 /vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/PreventRequestsDuringMaintenance.php(86): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}()
#26 /vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(180): Illuminate\\Foundation\\Http\\Middleware\\PreventRequestsDuringMaintenance->handle()
#27 /vendor/laravel/framework/src/Illuminate/Http/Middleware/HandleCors.php(49): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}()
#28 /vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(180): Illuminate\\Http\\Middleware\\HandleCors->handle()
#29 /vendor/laravel/framework/src/Illuminate/Http/Middleware/TrustProxies.php(39): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}()
#30 /vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(180): Illuminate\\Http\\Middleware\\TrustProxies->handle()
#31 /vendor/laravel/framework/src/Illuminate/Pipeline/Pipeline.php(116): Illuminate\\Pipeline\\Pipeline->Illuminate\\Pipeline\\{closure}()
#32 /vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(165): Illuminate\\Pipeline\\Pipeline->then()
#33 /vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(134): Illuminate\\Foundation\\Http\\Kernel->sendRequestThroughRouter()
#34 /public/index.php(52): Illuminate\\Foundation\\Http\\Kernel->handle()
#35 /vendor/laravel/framework/src/Illuminate/Foundation/resources/server.php(16): require_once('...')
#36 {main}
"} 

In general, this is not important, the situation itself is wrong.

[fabio-ivona]

In general, this is not important, the situation itself is wrong

On the contrary, the stack trace allows me to identify the issue

I've released a fix for this (please upgrade to ^v1.33.1)

If I type the /test

you are referring to the documentation for callback queries, but your message says you want to handle a command instead. Your code works anyway, but you should not refer to that part of the documentation

Is there another way to handle the callback function? For example:

match ($action) {
'test' => MyClass::myMethod(),
default => $this->reply('Invalid action');
}

if you want to handle an invalid action you can use this:

class MyWebhookHandler extends WebhookHandler
{
    public function test(): void
    {
        MyClass::myMethod();
    }

     protected function handleUnknownCommand(Stringable $text): void
    {
        $this->reply('Invalid action');
    }
}

[idvlab]

I mean that the test() method will be called in two cases:

1. Сallback queries
2. Handle a command

It seems to me that this is wrong.

Sorry for my English, I'm using a translator.

[fabio-ivona]

in this case you can check if isset($this->callbackQueryId) to differ from command to callback query