Skip to content
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

Can't import texts #33

Closed
rc-ops opened this issue May 30, 2022 · 1 comment
Closed

Can't import texts #33

rc-ops opened this issue May 30, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@rc-ops
Copy link

rc-ops commented May 30, 2022

Extracted from #31

  1. If I try to import a long text (doesn't matter if I paste or select a txt file), when I select NEXT STEP: Check the texts I get an error message and as you can see, it doesn't seem to process the text, it returns Length: 0 Bytes, and two of the buttons here don't work (Cancel and Go Back), only the Create 1 text, which returns us to the UI problem.

Notice: Undefined index: TextTags in C:\Program Files (x86)\EasyPHP-Devserver-17\eds-www\lwt\long_text_import.php on line 30
Warning: Invalid argument supplied for foreach() in C:\Program Files (x86)\EasyPHP-Devserver-17\eds-www\lwt\long_text_import.php on line 67

2

1.1 After selecting Create 1 text, since it's null, I get

Fatal Error in SQL Query: insert into texts (TxLgID, TxTitle, TxText, TxAnnotatedText, TxAudioURI, TxSourceURI) values( 1, 'Mondaiji 2', NULL, '', NULL, NULL)
Error Code & Message: [1048] Column 'TxText' cannot be null

Backtrace:

#0  do_mysqli_query(insert into texts (TxLgID, TxTitle, TxText, TxAnnotatedText, TxAudioURI, TxSourceURI) values( 1, 'Mondaiji 2', NULL, '', NULL, NULL)) called at [C:\Program Files (x86)\EasyPHP-Devserver-17\eds-www\lwt\inc\database_connect.php:60]
#1  runsql(insert into texts (TxLgID, TxTitle, TxText, TxAnnotatedText, TxAudioURI, TxSourceURI) values( 1, 'Mondaiji 2', NULL, '', NULL, NULL), ) called at [C:\Program Files (x86)\EasyPHP-Devserver-17\eds-www\lwt\long_text_import.php:160]
  • The only way I can import a text is if paste it in New Text, it can't be through Long Text. Note that here the Check button more or less... works? However it graphically represents the line breaks and the line numbers. Is that intended?

3

  • I can also see that it returns a word list and non-word list which is cool and would that mean MeCab is working... ?
  • But since it doesn't parse, nor can I select it to parse manually, it doesn't save any new terms.

Desktop (please complete the following information):

  • Same setup as last time, except this time Chromium has another version
  • OS: Windows 11 21H2 and Windows 10 20H2
  • Browser: Chromium 101.0.4951.64
  • EasyPHPDevserver-17 PHP 7.1.3
@HugoFara HugoFara added the bug Something isn't working label May 31, 2022
@rc-ops rc-ops mentioned this issue May 31, 2022
@HugoFara
Copy link
Owner

HugoFara commented Jun 4, 2022

Hello!

I have been trying to solve the long_text_import this issue today, without much success for Japanese. For other languages, I managed to fix problems introduced in 2.4.0, but for Japanese I'm still facing a formatting issue.

That being said, the main problem is that the code is a mess, so it's always difficult to sort out who does what...

For the normal text import, the data displayed when clicking Check are what was needed, so it it means MeCab is working fine. I'm not sure why you are still unable to save the text.

I hope to release the bug fix version on Monday, I will give you more information there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants