[fixed] [JIRA MP1-2469] - DetectDrive method obsolete ? (3 Viewers)

[azzuro]

Now, this evening, i need test with USB DVD drive !!! if inserted , and removed ! because in configuration.exe, the option add "optical drive" is checked !
but not related to this mantis ! (all users already have DVD drive ! but not Bluray drive !)

 

[Deda]

Well then - the icons shown for the removable drives are definitely not the defaultRemovable.png/defaultRemovableBig.png - so something not correct here @Deda

Don't have a clue , like it is now it was from the time when USB flash drives added to MP (never touched that in my life )

Just, checked, defaultRemovable... images does not exist in skins.

 

[azzuro]

Ah !!!!!!!!!! one thing !!!!!!!!!!!!!!
In configuration/ shared folder : set letter "J:" to USB device with name "Dummy"

in MP GUI : plug an other USB Dongle to System : Windows set as "J: REAL" because letter is not used by a drive
MP show "J: Dummy" instead of "J: REAL" because was set in configuration.

 

[catavolt]

Just, checked, defaultRemovable... images does not exist in skins.

In Default skins for sure

 

[azzuro]

In Default skins for sure

You can add it to this Mantis/Branch ? (@elliottmc : no need to open new mantis for this, right ? only add comments in mantis, should be clear)

 

[azzuro]

Result of my test with USB DVD Drive
- MP ask for read DVD ? : Yes, MP ask when is inserted
- Drive letter appear in file list : NO, Optical drive is not added in Folder list.

[2013-04-25 19:38:24,970] [Log	] [MPMain  ] [DEBUG] - Detected new optical media: I
[2013-04-25 19:38:25,019] [Log	] [MPMain  ] [INFO ] - Check if we want to autoplay media type: UNKNOWN  //DVD drive plugged to PC
[2013-04-25 19:38:42,120] [Log	] [MPMain  ] [INFO ] - Main: OnResizeEnd
[2013-04-25 19:39:00,274] [Log	] [MPMain  ] [DEBUG] - Detected new optical media: I  // insert Audio CD in DVD drive
[2013-04-25 19:39:00,317] [Log	] [MPMain  ] [INFO ] - Check if we want to autoplay media type: AUDIO

[2013-04-25 19:39:43,190] [Log	] [MPMain  ] [DEBUG] - Detected optical media removal: I  //
Remove CD audio
[2013-04-25 19:39:43,190] [Log	] [MPMain  ] [DEBUG] -  time after ExamineCD 29,2766745 s
[2013-04-25 19:39:51,411] [Log	] [MPMain  ] [DEBUG] - Detected device remove: I  //unplug DVD drive

 

[elliottmc]

Result of my test with USB DVD Drive
- MP ask for read DVD ? : Yes, MP ask when is inserted
- Drive letter appear in file list : NO, Optical drive is not added in Folder list.

[2013-04-25 19:38:24,970] [Log	] [MPMain  ] [DEBUG] - Detected new optical media: I
[2013-04-25 19:38:25,019] [Log	] [MPMain  ] [INFO ] - Check if we want to autoplay media type: UNKNOWN  //DVD drive plugged to PC
[2013-04-25 19:38:42,120] [Log	] [MPMain  ] [INFO ] - Main: OnResizeEnd
[2013-04-25 19:39:00,274] [Log	] [MPMain  ] [DEBUG] - Detected new optical media: I  // insert Audio CD in DVD drive
[2013-04-25 19:39:00,317] [Log	] [MPMain  ] [INFO ] - Check if we want to autoplay media type: AUDIO

[2013-04-25 19:39:43,190] [Log	] [MPMain  ] [DEBUG] - Detected optical media removal: I  //
Remove CD audio
[2013-04-25 19:39:43,190] [Log	] [MPMain  ] [DEBUG] -  time after ExamineCD 29,2766745 s
[2013-04-25 19:39:51,411] [Log	] [MPMain  ] [DEBUG] - Detected device remove: I  //unplug DVD drive

So, is this all good or bad?

Really needs to state expected and actual behaviour.

 

[azzuro]

For me is BAD because
* HDD identified as Local HDD
* External DVD drive, not display in MP Filelist

@Deda we can have better ? (no sarcasm please )

 

[elliottmc]

I think we should delay this from 1.4.0 and take time to get it right in 1.5.0. If we rush now to fix this, without considering all usage scenarios, we will break something. Thoughts @Deda ?

 

[Deda]

No objection, this is far from finished (even in my case I don't need more as it is now ).

USB HDD is fixed HDD as it is reported like that by the system (blame MS ). This is unrepairable unless some messing within virtual directory and shares.....

USB Optical disk, don't have a clue. Don't have any so can't do anything with it.