search

ngoctuan / my-films

Automatically exported from code.google.com/p/my-films
0 stars 0 forks source link

Add Option to Select NFO grabber to setup Wizard #165

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What area does your enhancement or feature relate to? (i.e. Config/Setup,
GUI, options, Skin, AMC Updater, Grabber Interface, etc.)
Config/Setup
What is the purpose or benefit of the new feature or enhancement?
To clarify support for nfo files, and allow preset of nfo grabber script and 
other options (such as picture/cover options) that may relate to using NFO 
files.
How would you expect it to work? Can you illustrate it with examples such
as images (screenshots), scenarios or examples from other Plugins?
Basically the same as other catalog types - but you can select NFO in catalog 
type drop down box, or in Setup Wizard.
Presets and setup options may need to be examined carefully to determine which 
fields should display in drop down boxes,or if some features/options are not 
supported for NFO based catalog types.
If this issue has been discussed in the MyFilms forum, add a link to the
related thread or post.
As discussed in NFO conversation!

Please sign with your MP forum username as often google accounts are quite
different.

NOTE: feature requests will be evaluated based on the number of
users who 'star' the issue as well as the details provided.

Original issue reported on code.google.com by Dade...@gmail.com on 15 Aug 2012 at 5:15

GoogleCodeExporter commented 9 years ago 
Based on discussion, I have changed this issue. NFO is not really a catalog 
type - it is a grabber, and can be used with AMC catalog type.

Thus I think a better solution would be to add a checkbox to the Setup Wizard 
(first screen where you select Country etc.) 'Use NFO grabber'. Ofc users can 
always change a catalog to use a different grabber, but it makes the feature 
more obvious imo.

Original comment by Dade...@gmail.com on 27 Oct 2012 at 12:16

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r2641.

Original comment by Mikeplan...@web.de on 28 Oct 2012 at 3:34

GoogleCodeExporter commented 9 years ago

Original comment by Guzz...@googlemail.com on 30 Oct 2012 at 3:33

GoogleCodeExporter commented 9 years ago

Original comment by Guzz...@googlemail.com on 13 Nov 2012 at 4:35

GoogleCodeExporter commented 9 years ago

Original comment by Dade...@gmail.com on 14 Nov 2012 at 5:24

GoogleCodeExporter commented 9 years ago

Original comment by Guzz...@googlemail.com on 14 Nov 2012 at 2:10

GoogleCodeExporter commented 9 years ago 
I can verify the option displays, but when I select it in Setup wizard it does 
NOT preset the grabber to NFO

Original comment by Dade...@gmail.com on 14 Nov 2012 at 11:54

GoogleCodeExporter commented 9 years ago 
Very very strange - I can confirm, it is not working - and even the code doing 
it is missing completely !?? So "not bug, missing feature" ... probably 
something went wrong with some reverts I did some time ago.

Original comment by Guzz...@googlemail.com on 15 Nov 2012 at 8:51

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r2717.

Original comment by Mikeplan...@web.de on 15 Nov 2012 at 10:25

GoogleCodeExporter commented 9 years ago 
Verified - in  build 2719 NFO grabber is correctly set in both MF Setup (and 
thus AMCU)

Original comment by Dade...@gmail.com on 16 Nov 2012 at 4:01

GoogleCodeExporter commented 9 years ago 
Updated Setup Wizard page in Wiki

Original comment by Dade...@gmail.com on 16 Nov 2012 at 5:32

GoogleCodeExporter commented 9 years ago 
nfo grabber option should not display for non AMC ctalog typed in setup wizard.

Original comment by Guzz...@googlemail.com on 16 Nov 2012 at 10:36

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r2720.

Original comment by Mikeplan...@web.de on 16 Nov 2012 at 10:54

GoogleCodeExporter commented 9 years ago 
verified in build 2720 - nfo option only displays for AMC cat type in Setup 
Wizard

Original comment by Dade...@gmail.com on 17 Nov 2012 at 4:55