Dict Engine: improve error handling.
ClosedPublic

Authored by dfaure on Oct 22 2017, 11:10 AM.

Details

Summary

Instead of returning "<dl>\n<dt><b></b></dt>\n<dd></dl>"
when the word doesn't exist, return a proper readable string so users
know that the lookup is done and nothing was found.

Test Plan

I'm porting the dict applet to QtQuick, review request coming soon.

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
dfaure created this revision.Oct 22 2017, 11:10 AM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 22 2017, 11:10 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.Oct 22 2017, 4:55 PM
This revision is now accepted and ready to land.Oct 22 2017, 4:55 PM

Don't you also need a Messages.sh and TRANSLATION_DOMAIN define?

Don't you also need a Messages.sh and TRANSLATION_DOMAIN define?

Well spotted. The Messages.sh file was there, but not the define. Fixed and pushed.

This revision was automatically updated to reflect the committed changes.