search

troggy / anki-leo

Chrome extension to export dictionary from LinguaLeo to CSV/Anki
http://troggy.github.io/anki-leo/
ISC License
42 stars 8 forks source link

Update leo-export.js #10

Closed lagunovsky closed 7 years ago

lagunovsky commented 7 years ago

Sometimes found the import error when the first character of the item quote.

troggy commented 7 years ago

@lagun4ik спасибо за правку. Есть ли пример данных, на которых происходит ошибка?

lagunovsky commented 7 years ago

Есть вероятность что вот этим коммитом было исправлено, так как сейчас воспроизвести не смог. Импорт падал на транскрипции, первым символом которой был '

troggy commented 7 years ago

Точно не им, этот код выполняется только для контекста слова. Одинарные кавычки везде тоже должны быть окей (я специально их использую вместо двойных).

Однако транскрипция вообще никак не обрабатывается и, если в ней будут спецсимволы вроде точки с запятой и двойной кавычки, это сломает импорт. Возможно, это и было причиной. Если так, то не думаю, что данный PR исправит это. Попробую воспроизвести

lagunovsky commented 7 years ago

смог воспроизвести, появилось когда в первом слове (не уверен важно ли что именно оно), в примере использования, взял последнее слово в ", которые на выходе cконвертировались в ' (скрин на дифф)

Максимально отфильтровал и выделил часть с которой падает

camping;лагерь для автотуристов, кемпинг;http://contentcdn.lingualeo.com/uploads/picture/559014.png;ˈkæmpɪŋ;He's '<b>camping</b>';http://audiocdn.lingualeo.com/v2/3/6848-631152008.mp3
tent;палатка;http://contentcdn.lingualeo.com/uploads/picture/1248.png;tɛnt;'He's got a rucksack and a <b>tent</b> on his back in the photo,' says the policeman;http://audiocdn.lingualeo.com/v2/3/40735-631152008.mp3
sunglasses;солнцезащитные очки;http://contentcdn.lingualeo.com/uploads/picture/305951.png;ˈsʌnˌglæsɪz;He has got <b>sunglasses</b> and a blue hat;http://audiocdn.lingualeo.com/v2/3/123120-631152008.mp3
party;вечеринка;http://contentcdn.lingualeo.com/uploads/picture/3249621.png;'pɑːtɪ;Going down, <b>party</b> time;http://audiocdn.lingualeo.com/v2/3/29540-631152008.mp3
rhyme;рифма;http://contentcdn.lingualeo.com/uploads/picture/3304131.png;raɪm;Don't need reason, don't need <b>rhyme</b>;http://audiocdn.lingualeo.com/v2/3/34592-631152008.mp3
troggy commented 7 years ago

Иван, я смерджил твой код (немного причесал): https://github.com/troggy/anki-leo/commit/d6ea7206fc886376bdb611f2bb2941624ecf649f

Спасибо за помощь!