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

Fix #48: associate textitems to words on 'i know all' click. #49

Merged
merged 1 commit into from Nov 1, 2022

Conversation

jzohrab
Copy link

@jzohrab jzohrab commented Oct 31, 2022

Attempted fix for Issue #48 .

With this fix, the textitems are associated with the new words:

mysql> select woid, wowordcount, wolgid, wotext, wotextlc, wostatus from words where wotext in ('Insecto', 'gatita');
+------+-------------+--------+---------+----------+----------+
| woid | wowordcount | wolgid | wotext  | wotextlc | wostatus |
+------+-------------+--------+---------+----------+----------+
|  576 |           0 |      1 | gatita  | gatita   |       99 |
|  575 |           0 |      1 | Insecto | insecto  |       99 |
+------+-------------+--------+---------+----------+----------+
2 rows in set (0.00 sec)

mysql> select * from textitems2 where ti2txid = 4;
+---------+---------+---------+---------+----------+--------------+---------+----------------+
| Ti2WoID | Ti2LgID | Ti2TxID | Ti2SeID | Ti2Order | Ti2WordCount | Ti2Text | Ti2Translation |
+---------+---------+---------+---------+----------+--------------+---------+----------------+
|     575 |       1 |       4 |     169 |        1 |            1 | Insecto |                |
|       0 |       1 |       4 |     169 |        2 |            0 |         |                |
|     576 |       1 |       4 |     169 |        3 |            1 | gatita  |                |
|       0 |       1 |       4 |     169 |        4 |            0 | .       |                |
+---------+---------+---------+---------+----------+--------------+---------+----------------+
4 rows in set (0.00 sec)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"set all to known" doesn't seem to stick, and then creates problems when trying to create terms
2 participants