-
Notifications
You must be signed in to change notification settings - Fork 585
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
Unable to run tagUI on MacOS & Windows - csv bug after macOS Catalina upgrade - fixed #597
Comments
I'm having the same issue on Catalina. Hopefully there's a fix. |
same here. encountering error with the file tagui_parse.php (line 31, 32, 34). Warning: count(): Parameter must be an array or an object that implements Countable in /Users/jacky/tagui/src/tagui_parse.php on line 31 Warning: count(): Parameter must be an array or an object that implements Countable in /Users/jacky/tagui/src/tagui_parse.php on line 32 Warning: count(): Parameter must be an array or an object that implements Countable in /Users/jacky/tagui/src/tagui_parse.php on line 34 I was just trying out with my own test script of just one line to go to "http://www.google.com". Running on: |
ok guys, after doing some research, I realised that the above error is due to PHP7.3 (or rather from v7.2 onwards). So my work around is to install another older version (7.1 in my case) and it is now working fine. $ ./tagui samples/1_yahoo START - automation started - Fri Oct 25 2019 10:40:38 GMT+0800 (+08) NOTE - this sample may not work for EU users due to website changes for GDPR type search-box as github https://duckduckgo.com/ - DuckDuckGo — Privacy, simplified.
|
Thanks for raising this! I can replicate, and confirm that this issue happens after macOS applies the update to a new release to Catalina. Thanks @JackyChuang74! Let me see if I can change something to fix this, so users do not need to do their own PHP installation to use TagUI.
|
In the interim, remove tagui_local.csv from the samples folder when you are running the samples and the error should disappear. The issue is with a backward-incompatible change in the PHP count() function, which comes with the upgraded PHP v7.3 in macOS Catalina. UPDATE - fix implemented is suppressing the warning messages from using count() function. As the usage context is straightforward, there isn't a need to capture potential error messages from calling count() function, which happens during the parsing phase of the automation (converting from human language syntax to JavaScript code). |
+1 |
for details, see aisingapore#597
update TagUI cutting edge version for PHP warning fix for details, see aisingapore#597
Fix implemented in TagUI cutting edge release (v5.9). To use it, follow the steps in setup section for overwriting existing packaged installation with the cutting edge release codebase - |
This issue impacts Windows users as well, updated issue title so that it's easier to search by users. The solution is using the fix implemented in TagUI cutting edge version. To use it, follow the steps in setup section for overwriting existing packaged installation with the cutting edge release codebase - https://github.com/kelaberetiv/TagUI#set-up This fix will be an upcoming packaged release later this month. And in a future planned v7, PHP issues would be removed permanently as the execution engine switches purely to NodeJS. Copying @siowyisheng fyi too! |
Implemented today in packaged release with @siowyisheng - fresh downloads of the packaged installations should not have this problem anymore! https://github.com/kelaberetiv/TagUI#set-up |
OS: macOS Catalina
Nodejs: v10.13.0
Python: 2.7.16
Java: 1.8.0_172
I tried running the samples by, unzip and run and npm install, both cases i get the following error.
The text was updated successfully, but these errors were encountered: