moosh icon indicating copy to clipboard operation
moosh copied to clipboard

user-create fatal php error. New hidden dependency for PhpParser\NodeVisitorAbstract?

Open willend opened this issue 6 years ago • 2 comments

  • moosh version: Aligned with GitHub master: www-data@elearn1:/srv/mcweb/moosh$ git rev-parse HEAD 35a40498dc4b34bfc9c6501bc371b487c69556ae

  • moodle version: Aligned with GitHub MOODLE_35_STABLE branch: www-data@elearn1:/srv/mcweb/moodle$ git status On branch MOODLE_35_STABLE Your branch is up-to-date with 'origin/MOODLE_35_STABLE'.

    • database: MySQL
  • php version: php --version PHP 7.0.30-0+deb9u1 (cli) (built: Jun 14 2018 13:50:25) ( NTS ) Copyright (c) 1997-2017 The PHP Group Zend Engine v3.0.0, Copyright (c) 1998-2017 Zend Technologies with Zend OPcache v7.0.30-0+deb9u1, Copyright (c) 1999-2017, by Zend Technologies

  • operating system: Debian 9.5 with all patches

Actual behaviour

PHP Fatal error: Class 'PhpParser\NodeVisitorAbstract' not found in /home/srv/mcweb/moosh/Moosh/Command/Moodle35/Dev/DevLangusage.php on line 24

Expected behaviour

User is successfully enrolled into Moodle

Steps to reproduce

In the Moodle folder, try a user-create: moosh user-create --auth=ldap --firstname="Test" --lastname="Testensen" --city=Lyngby --country=DK [email protected] --password=NONE deletemeplease PHP Fatal error: Class 'PhpParser\NodeVisitorAbstract' not found in /srv/mcweb/moosh/Moosh/Command/Moodle35/Dev/DevLangusage.php on line 24

Fatal error: Class 'PhpParser\NodeVisitorAbstract' not found in /srv/mcweb/moosh/Moosh/Command/Moodle35/Dev/DevLangusage.php on line 24

willend avatar Sep 04 '18 08:09 willend

The issue is not found if I instead use the latest .deb package from from http://ppa.launchpad.net/zabuch/ppa/ubuntu/

willend avatar Sep 04 '18 08:09 willend

OK, I see - the issue likely came in from this commit

https://github.com/tmuras/moosh/commit/35a40498dc4b34bfc9c6501bc371b487c69556ae#diff-b5d0ee8c97c7abd7e3fa29b9a27d1780

I guess I should not blindly run off of your master branch @tmuras . :-)

What is your recommendation - I run of the "debian" branch or use the .deb package instead?

willend avatar Sep 04 '18 08:09 willend