IMDb spelling and grammar correction

[IanSav]

In the IMDb plugin accessed via RED from the EPG displays the MENU > Setup > Words / phrases to ignore" help text could use some correction.

The existing text is "This option allows you to add words/phrases for IMDb to ignore when searching. please seperaate with a comma"

The second sentence in the help text needs fixing. Capital "P" for "please", "separate" for "seperaate", add "the words/phrases" after separate and add a full stop at the end of the sentence. In other words "Please separate the words/phrases with a comma."

I would make the change but I didn't see the plugin source in the Beyonwiz repository.

[prl]

The IMDb plugin source is in the Beyonwiz Github repository.

You can find this out by doing

opkg list-installed | grep imdb

in the command-line interface on the T series, and then, using the package name that that reveals, do

opkg info enigma2-plugin-extensions-imdb

which tells you:

Source: git://github.com/beyonwiz/enigma2-plugins.git;protocol=git;branch=2.3 file://pluginnotwanted.patch

Using http://github.com/beyonwiz/enigma2-plugins will take you to the repository in your browser.

[IanSav]

I thought the GIT repository was old and unused. Anyway, when I get some time I will clone that repository and submit my update proposals.

Thanks for the pointer.

[prl]

You're probably better off forking the oe-alliance/enigma2-plugins repository (there's a link to it from beyonwiz/enigma2-plugins) and submitting to it. That's what I'm doing with the fix I'm currently working on.

[prl]

I've gone through the plugin text, and I'm proposing to change:

"Save current Details as .html for offline using" -> "Save current Details as .html for offline use"
'IMDb can not get Movie Information, to\n write .txt-file!' -> 'IMDb can not get Movie Information to\nwrite to .txt file!'
'IMDb can not get Movie Information, to\n write .jpg and .txt-file!' -> 'IMDb can not get Movie Information to\nwrite to .jpg and .txt files!'
"IMDb Save-Download completed" -> "IMDb Save - Download completed"
"This option allows you add words/phrases for IMDb to ignore when searching. please seperaate with a comma"
->
"This option allows you add words/phrases for IMDb to ignore when searching. Please separate the words/phrases with commas."

[Peter Urbanec]

I would suggest dropping the unnecessary capitalisation of some words in the middle of sentences.

Please see if you can get these changes upstream, I'll then pull those changes into the Beyonwiz repo.

[prl]

The capitalisation seems to fairly uniformly refer to specific screen functions or display items, so I left it as it was. There's more of it than is in those suggested corrections, but it doesn't seem to be the German-ish "capitalise all the Nouns" sort of error that is sometimes seen in other parts of the code.

[prl]

My text corrections for imdbsearch have been merged into the OE-Alliance repository.

[IanSav]

This issue has been resolved.