[Symfony][SF2] Command Console Opcje

[Michal_Sz]

Witam,

Chciałbym wywołać w konsoli akcję, a później wrzucić ją do cronów.

Znalazłem prosty przykład jak wyświetlić coś w konsoli czyli:

[PHP] pobierz, plaintext 
namespace Cron\CronBundle\Command;
 
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
 
class GreetCommand extends Command
{
    protected function configure()
    {
         $this
            ->setName('phax:action')
            ->setDescription('Execute Phax action using command line')
            ->addArgument('controller', InputArgument::OPTIONAL, 'Controller name', 'help')
            ->addArgument('action', InputArgument::OPTIONAL, 'Action name', 'default')
            ->addOption(
                'parameters',
                'p',
                InputOption::VALUE_OPTIONAL | InputOption::VALUE_IS_ARRAY,
                'Parameters to send to the controller'
            )
        ;
    }
 
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $name = $input->getArgument('name');
        if ($name) {
            $text = 'Hellosa '.$name;
        } else {
            $text = 'Helloaaa';
        }
 
        if ($input->getOption('yell')) {
            $text = strtoupper($text);
        }
 
        $output->writeln($text);
    }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
#!/usr/bin/env php
<?php
// application.php
 
require __DIR__.'/vendor/autoload.php';
 
use Cron\CronBundle\Command\GreetCommand;
use Symfony\Component\Console\Application;
 
 
$application = new Application();
$application->add(new GreetCommand());
$application->run();
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$ php application.php demo:greet Fabien
[PHP] pobierz, plaintext 

Ale teraz nie wiem jak zrobić aby wywołać np. akcję z Cron\CronBundle\Controller\DefaultController->indexAction()

Prosiłbym o jakąś pomoc i wskazówki

[Damonsson]

Hmm skoro chcesz robić coś takiego, to na kontroler mi Twoja klasa (Controller\DefaultController) nie wygląda, prędzej jakiś serwis. A jak utworzyć serwis i się do niego odwołać gdziekolwiek w Symfony2 masz ładne opisy w dokumentacji.

Ten post edytował Damonsson 1.07.2015, 15:31:56

[Michal_Sz]

Dzięki za radę. Stworzyłem service ale kiedy próbuje się odwołać do niego w EXECUTE w taki sposób:

[PHP] pobierz, plaintext 
 $this->container = $this->getApplication()->getKernel()->getContainer();
       $c1m = $this->container->get('cron_1m');
[PHP] pobierz, plaintext 

lub w taki sposób:

[PHP] pobierz, plaintext 
 $this->getContainer()->get('cron_1m');
[PHP] pobierz, plaintext 

dostaję komunikat:

[PHP] pobierz, plaintext 
 Call to undefined
method Symfony\Component\Console\Application::getKernel() in /home/httpd/virtual/nowepaliwa/src/Cron/Cron1mBundle/Command/TestCommand.php on line 36
[PHP] pobierz, plaintext 

lub

[PHP] pobierz, plaintext 
PHP Fatal error:  Call to undefined method Cron\Cron1mBundle\Command\TestCommand::getContainer() in /home/httpd/virtual/nowepaliwa/src/Cron/Cron1mBundle/Command/TestCommand.php on line 34
 
[PHP] pobierz, plaintext 

czy coś pominąłem lub co zrobiłem nie tak?

[Damonsson]

A rozszerzasz klasę ContainerAwareCommand, zamiast Command?

[Michal_Sz]

Tak,próbowałem obu rozszerzeń.
Teraz wygląda totak:

[PHP] pobierz, plaintext 
<?php
 
namespace Cron\Cron1mBundle\Command;
use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
 
class TestCommand extends ContainerAwareCommand
{
    protected function configure()
    {
         $this
            ->setName('phax:action')
            ->setDescription('Execute Phax action using command line')
 
        ;
    }
 
    protected function execute(InputInterface $input, OutputInterface $output)
    {
 
       $c1m = $this->container->get('cron_1m');
        $txt = $c1m->indexAction();
        $output->writeln($txt );
    }
 
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
require __DIR__.'/vendor/autoload.php';
 
use Cron\Cron1mBundle\Command\TestCommand;
//use Cron\CronBundle\Command\GreetCommand;
use Symfony\Component\Console\Application;
 
 
$application = new Application();
$application->add(new TestCommand());
$application->run();
[PHP] pobierz, plaintext 

i wywołanie:

[PHP] pobierz, plaintext 
~$ php application_1m.php phax:action
[PHP] pobierz, plaintext 

[Damonsson]

Zainwestuj w jakieś porządne IDE np. PHPStorm, to od razu dostaniesz info, że container jest private i musisz się dobrać przez getter.

czyli u Ciebie powinno to wyglądać tak:

[PHP] pobierz, plaintext 
$c1m = $this->getContainer()->get('cron_1m');
[PHP] pobierz, plaintext 

[Michal_Sz]

Przepraszam nie to skopiowałem, oczywiście jest tak:

[PHP] pobierz, plaintext 
        $this->getContainer()->get('cron_1m');
[PHP] pobierz, plaintext 

Błąd ciągle pozostał.

[Damonsson]

ee niemożliwe, a treść błędu?

[Michal_Sz]

[PHP] pobierz, plaintext 
PHP Fatal error:  Call to undefined method Symfony\Component\Console\Application::getKernel() in /home/httpd/virtual/nowepaliwa/vendor/symfony/symfony/src/Symfony/Bundle/FrameworkBundle/Command/ContainerAwareCommand.php on line 43
[PHP] pobierz, plaintext 

[Damonsson]

i jeszcze jakbyś był łaskaw pokazać tę 143 linię

jeżeli masz tam coś takiego:

[PHP] pobierz, plaintext 
$this->container = $this->getApplication()->getKernel()->getContainer(); 
[PHP] pobierz, plaintext 

to wywal to.

[Michal_Sz]

ale to wskazuje na plik którym rozszerzam czyli ContainerAwareCommand z tej ścieżki: /vendor/symfony/symfony/src/Symfony/Bundle/FrameworkBundle/Command/ContainerAwareCommand.php

[PHP] pobierz, plaintext 
namespace Symfony\Bundle\FrameworkBundle\Command;
 
use Symfony\Component\Console\Command\Command;
use Symfony\Component\DependencyInjection\ContainerInterface;
use Symfony\Component\DependencyInjection\ContainerAwareInterface;
 
/**
 * Command.
 *
 * @author Fabien Potencier <fabien@symfony.com>
 */
abstract class ContainerAwareCommand extends Command implements ContainerAwareInterface
{
    /**
     * @var ContainerInterface|null
     */
    private $container;
 
    /**
     * @return ContainerInterface
     *
     * @throws \LogicException
     */
    protected function getContainer()
    {
        if (null === $this->container) {
            $application = $this->getApplication();
            if (null === $application) {
                throw new \LogicException('The container cannot be retrieved as the application instance is not yet set.');
            }
 
            $this->container = $application->getKernel()->getContainer();
        }
 
        return $this->container;
    }
 
    /**
     * {@inheritdoc}
     */
    public function setContainer(ContainerInterface $container = null)
    {
        $this->container = $container;
    }
}
[PHP] pobierz, plaintext 

jak to wywalam to jest fatal error

Ten post edytował Michal_Sz 1.07.2015, 21:57:05

[Damonsson]

Aaa dobra moje niedopatrzenie nic nie usuwaj, zasugerowałem się poprzednim postem, w którym miałeś taką konstrukcję.

W app/console Zamień

[PHP] pobierz, plaintext 
use Symfony\Component\Console\Application;
[PHP] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
use Symfony\Bundle\FrameworkBundle\Console\Application;
[PHP] pobierz, plaintext 

[Michal_Sz]

Zmieniłem i pojawiło sie jeszcze coś gorszego:

[PHP] pobierz, plaintext 
PHP Catchable fatal error:  Argument 1 passed to Symfony\Bundle\FrameworkBundle\Console\Application::__construct() must be an instance of Symfony\Component\HttpKernel\KernelInterface, none given, called in /home/httpd/virtual/nowepaliwa/application_1m.php on line 13 and defined in /home/httpd/virtual/nowepaliwa/vendor/symfony/symfony/src/Symfony/Bundle/FrameworkBundle/Console/Application.php on line 38
[PHP] pobierz, plaintext 

[Damonsson]

Spokojnie, jest lepiej (IMG:style_emoticons/default/smile.gif)

Hmm skąd wziąłeś ten plik app/console? To jakiś wymóg jakiegoś bundle?

Teoretycznie tak powinien wyglądać i wtedy wszystko powinno działać: https://github.com/symfony/symfony-demo/blo...ter/app/console

Ten post edytował Damonsson 1.07.2015, 22:18:17

[Michal_Sz]

u mnie to wygląda tak:

[PHP] pobierz, plaintext 
<?php
 
// application.php
 
require __DIR__.'/vendor/autoload.php';
 
use Cron\Cron1mBundle\Command\TestCommand;
 
use Symfony\Bundle\FrameworkBundle\Console\Application;
 
 
$application = new Application();
$application->add(new TestCommand());
$application->run();
[PHP] pobierz, plaintext 

gdzieś w Internecie znalazłem taki przykłąd

Ten post edytował Michal_Sz 1.07.2015, 22:20:51

[Damonsson]

Ok, ale pytanie za sto punktów czemu tak wygląda? Sam go napisałeś? Zamień go na ten który Ci podałem na github, jak nic nie wybuchnie i będzie wszystko działać to ok. A jak coś przestanie działać to będzie większy problem niestety.

[Michal_Sz]

Namieszałem, plik o którym mówisz wygląda dokłądnie tak jak podałeś, to co CI wysłąłem to plik który wywołuje w konsoli.
Plik: appliaction.php:

[PHP] pobierz, plaintext 
<?php
 
// application+1m.php
 
require __DIR__.'/vendor/autoload.php';
 
use Cron\Cron1mBundle\Command\TestCommand;
 
use Symfony\Component\Console\Application;
 
 
$application = new Application();
$application->add(new TestCommand());
$application->run();
[PHP] pobierz, plaintext 

iw konsoli wywołuje:

[PHP] pobierz, plaintext 
php application_1m.php phax:action
[PHP] pobierz, plaintext 

[Damonsson]

Wiesz co, prościej będzie jak wywalisz wszystko, zainstalujesz po bożemu symfony i po bożemu wg dokumentacji dodasz komendę i serwis (IMG:style_emoticons/default/biggrin.gif) To jest naprawdę banalne i trwa 2 minuty, a te wymyślane pliki typu application_1m.php tylko wszystko pieprzą.

Ale spróbujmy to odratować.

A co się stanie jak wywołasz:

[PHP] pobierz, plaintext 
php app/console phax:action
[PHP] pobierz, plaintext 

lub

[PHP] pobierz, plaintext 
php app/console
[PHP] pobierz, plaintext 

[Michal_Sz]

Powiem Ci, ze przepisywanie wszystkiego na nowo to byłaby masakra, ale Twoje rozwiązanie pomogło i mogę to wywołać, a przynajmniej się nie wysypuje. Muszę dokońćzyć funkcję, żeby coś wyciagać z indexAction i wtedy będę pewny czy jest ok bo na tą chwilę zwraca mi:

[PHP] pobierz, plaintext 
PHP Fatal error:  Call to a member function get() on a non-object in /home/httpd/virtual/nowepaliwa/vendor/symfony/symfony/src/Symfony/Bundle/FrameworkBundle/Controller/Controller.php on line 108
[PHP] pobierz, plaintext 

NIe bardzo jednak rozumiem dlaczego akurat phax:action wywołuje a np. jeśli w innym pliku jest phax:action1 to już nie

Ten post edytował Michal_Sz 1.07.2015, 22:51:53

[Damonsson]

Ok czyli

[BASH] pobierz, plaintext 
php app/console phax:action
[BASH] pobierz, plaintext 

działa poprawnie?

Teoretycznie dla swojego pliku application_1m.php mógłbyś stworzyć taką samą strukturę jak jest w app/console i ją wywoływać, ale że musisz przekazać AppKernel, skoro chcesz się odwołać do serwisu to nie miałoby żadnego sensu bo i tak miałbyś wszystkie komendy z symfony.

NIe bardzo jednak rozumiem dlaczego akurat phax:action wywołuje a np. jeśli w innym pliku jest phax:action1 to już nie

To znaczy? phax:action tak nazwałeś swoją komendę, jak chcesz móc wywołać phax:action1, to musisz taką stworzyć.

Ten post edytował Damonsson 1.07.2015, 22:56:43

[Michal_Sz]

[PHP] pobierz, plaintext 
php app/console phax:action
[PHP] pobierz, plaintext 

działa poprawnie, tylko nadal nie rozumiem jednego.
Sworzyłem plik w takiej ścieżce:

[PHP] pobierz, plaintext 
src\Cron\Cron1mBundle\Command\TestCommand.php
[PHP] pobierz, plaintext 

gdzie faktycznie jest komenda

[PHP] pobierz, plaintext 
phax:action
[PHP] pobierz, plaintext 

- i ją wywołuje tak jak wyżej

Stworzyłem plik w takiej ścieżce:

[PHP] pobierz, plaintext 
src\Cron\Cron1mBundle\Command\TestCommand1.php
[PHP] pobierz, plaintext 

gdzie jest komenda

[PHP] pobierz, plaintext 
phax:action1
[PHP] pobierz, plaintext 

- i już tego nie mogę wywołać w ten sposób

[PHP] pobierz, plaintext 
php app/console phax:action1
[PHP] pobierz, plaintext 

- bo jest błąd że komenda nie istnieje.

Szczerze mówiąc nie kojarzę żebym coś jeszcze robił aby wywołać:

[PHP] pobierz, plaintext 
phax:action
[PHP] pobierz, plaintext 

[prz3kus]

Bez błędów i kodu nie ma odpowiedzi, szklana kula gdzieś zagineła (IMG:style_emoticons/default/smile.gif)

[Michal_Sz]

Błąd pojawia się taki:

[PHP] pobierz, plaintext 
[InvalidArgumentException]
  Command "phax:action1" is not defined.
  Did you mean this?
      phax:action
[PHP] pobierz, plaintext 

Gdzie należy zdefiniować komendę?

[prz3kus]

wklej cale źródło pliku src\Cron\Cron1mBundle\Command\TestCommand1.php

Ten post edytował prz3kus 2.07.2015, 07:19:27

[Michal_Sz]

[PHP] pobierz, plaintext 
namespace Cron\Cron1mBundle\Command;
 
 
use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
 
class TestCommand1 extends ContainerAwareCommand
{
    protected function configure()
    {
         $this
            ->setName('phax:action1')
            ->setDescription('Execute Phax action using command line')
 
        ;
    }
 
    protected function execute(InputInterface $input, OutputInterface $output)
    {
 
        $c1m = $this->getContainer()->get('cron_1m');
        $output->writeln('$texst');
    }
 
}
[PHP] pobierz, plaintext 

Ten post edytował Michal_Sz 2.07.2015, 07:25:18

[prz3kus]

no to jeszcze wklej metode action1 z kontrolera phax

Pozatym pytanie czy czsami nie prościej sobie wywołac jakieś zapytanie czy cuś (IMG:style_emoticons/default/smile.gif) odrazu w tej klasie

Tutaj jedno z moich zadań robiące update do bazy

[PHP] pobierz, plaintext 
class StatusCommand extends ContainerAwareCommand
{
    protected function configure()
    {
        $this
            ->setName('statusupdate')
            ->setDescription('Aktualizacja statusów')
        ;
    }
 
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $em = $this->getContainer()->get('doctrine.orm.entity_manager');
 
        $today = new \DateTime("now");
        $repository = $em->getRepository('OFPortalBundle:Offer');
 
        $query = $repository->createQueryBuilder('o')
            ->update()
            ->set('o.status',3)
            ->where('o.expirationDateTime < :today and o.status < 3')
            ->setParameter('today', $today)
            ->getQuery()
            ->getResult();
 
        $output->writeln('ok');
    }
}
[PHP] pobierz, plaintext 

[Michal_Sz]

Ta metoda action1 okazuje się, że nie ma znaczenia, bo czy ona jest czy nie ma błąd jest ten sam.
Właściwie to był bo dorzuciłem do app/config/config.yml:

[PHP] pobierz, plaintext 
services:
    app.command.my_command:
        class: Cron\Cron1mBundle\Command\TestCommand1
        arguments: ["%command.default_name%"]
        tags:
            -  { name: console.command }
[PHP] pobierz, plaintext 

i teraz widzi mi komendę:

[PHP] pobierz, plaintext 
phax:action1
[PHP] pobierz, plaintext 

z konsoli już to mogę uruchomić, ale nie rozumiem jednej rzeczy, bo wcześniej nie rejestrowałem w configu komendy

[PHP] pobierz, plaintext 
phax:action
[PHP] pobierz, plaintext 

a mimo to działa

[Damonsson]

Bo dodałeś 1 na końcu, zrób komendę Test1Command to nic nie będziesz musiał dodawać. Zamiast marnować czas na forum przez ten czas przeczytałbyś tę dokumentację Symfony w końcu, tam jest WSZYSTKO o co pytasz.

Pozatym pytanie czy czsami nie prościej sobie wywołac jakieś zapytanie czy cuś (IMG:style_emoticons/default/smile.gif) odrazu w tej klasie

Tutaj jedno z moich zadań robiące update do bazy

[PHP] pobierz, plaintext 
class StatusCommand extends ContainerAwareCommand
{
    protected function configure()
    {
        $this
            ->setName('statusupdate')
            ->setDescription('Aktualizacja statusów')
        ;
    }
 
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $em = $this->getContainer()->get('doctrine.orm.entity_manager');
 
        $today = new \DateTime("now");
        $repository = $em->getRepository('OFPortalBundle:Offer');
 
        $query = $repository->createQueryBuilder('o')
            ->update()
            ->set('o.status',3)
            ->where('o.expirationDateTime < :today and o.status < 3')
            ->setParameter('today', $today)
            ->getQuery()
            ->getResult();
 
        $output->writeln('ok');
    }
}
[PHP] pobierz, plaintext 

A tak broń Boże się nie ucz pisać, bo nie ma to nic wspólnego z OOP, nie mówiąc o napisaniu do tego testów.

Ten post edytował Damonsson 2.07.2015, 09:38:54

[Michal_Sz]

Dokumentację czytam, ale bywa tak że najprostsze rzeczy bardzo się komplikują.
Dzięki za pomoc.

[aras785]

Zamiast marnować czas na forum przez ten czas przeczytałbyś tę dokumentację Symfony w końcu, tam jest WSZYSTKO o co pytasz.

Idąc Twoim tokiem myślenia to nie powinno być for dyskusyjnych bo przecież do każdego języka/frameworka jest dokumentacja, a jak nie to przecież można kod źródłowy przeanalizować... (IMG:style_emoticons/default/facepalmxd.gif)

[Damonsson]

@aras785 Weź wyjdź przeczytaj jakąś netykietę, poradnik jak zadawać pytania, poczytaj o akronimie RTFM i skąd się wziął, wróć i przeproś.

Chętnie pomagam każdej osobie na forum, a sugestia zajrzenia do dokumentacji to też pomoc, bo po co ma czekać, aż ja za niego przeczytam dokumentację, przetłumaczę i opiszę co jest w niej napisane?

[Michal_Sz]

witam ponownie,

Pojawił sie u mnie jeszcze jeden problem dotyczący wywoływania akcji z poziomu konsoli.
Udaje mi się wywoływać proste akcje z poziomu konsoli.
Opisuje po kolei bo może coś pominąłem.
Posiadam kontroler:

[PHP] pobierz, plaintext 
class DefaultController extends Controller
{
    public function indexAction($em_mysql)
    {
        $cronlogs = new Cronlogs();
        $cronlogs->setCreateDate(new \DateTime("now"));
        $cronlogs->setCronId('1M');
        $cronlogs->setName('minutowy ogolny');
 
        $em_mysql->persist($cronlogs);
        $em_mysql->flush();
 
        $cron = 'CRON 1 M ';
        return new Response($cron);
    }
}
[PHP] pobierz, plaintext 

Stworzyłem serwis, aby odwływać się do tej akcji i wygląda to tak:

[PHP] pobierz, plaintext 
 
class Cron1mCommand extends ContainerAwareCommand
{
    protected function configure()
    {
         $this
            ->setName('phax:cron1m')
            ->setDescription('Cron minutowy');
    }
 
    protected function execute(InputInterface $input, OutputInterface $output)
    {
 
        $c1m = $this->getContainer()->get('cron_1m');
        $entityManager = $this->getContainer()->get('doctrine')->getManager('default');
 
        $txt = $c1m->indexAction($entityManager);
        $output->writeln($txt);
    }   
}
[PHP] pobierz, plaintext 

Kiedy w konsoli odwołuje się do tego poprzez:

[PHP] pobierz, plaintext 
 php app/console phax:cron1m
[PHP] pobierz, plaintext 

to jest OK, ale dlatego, że do indexAction przekazuje:

[PHP] pobierz, plaintext 
 $entityManager = $this->getContainer()->get('doctrine')->getManager('default');
[PHP] pobierz, plaintext 

Jeśli natomiast nie przekazywałbym tego do akcji, ale wywoływał to w samej akcji czyli:

[PHP] pobierz, plaintext 
class DefaultController extends Controller
{
    public function indexAction()
    {
        $cronlogs = new Cronlogs();
        $cronlogs->setCreateDate(new \DateTime("now"));
        $cronlogs->setCronId('1M');
        $cronlogs->setName('minutowy ogolny');
        [b]$em_mysql= $this->getContainer()->get('doctrine')->getManager('default');[/b]
        $em_mysql->persist($cronlogs);
        $em_mysql->flush();
 
        $cron = 'CRON 1 M ';
        return new Response($cron);
    }
}
[PHP] pobierz, plaintext 

to wtedy dostaję błąd:

[PHP] pobierz, plaintext 
Call to a member function get() on a non-object in /home/httpd/virtual/nowepaliwa/vendor/symfony/symfony/src/Symfony/Bundle/FrameworkBundle/Controller/Controller.php on line 275
[PHP] pobierz, plaintext 

Z czego to może wynikać?
Myślę, że takie rozwiązanie jest błędne, poza tym cokolwiek bym nie chciał wywołać w tej akcji, choćby serwis to musiałbym wszystko przekazywać do akcji.