zyronix / auto-sub

Automatically exported from code.google.com/p/auto-sub
0 stars 0 forks source link

Syntax problem with skip show #110

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.Start Auto-Sub and go to the "Home" screen
2.Click a show in the "Wanted" list starting with " [ "
3.Choose "Skip show"

OR

1.Start Auto-Sub and go to the "Config" screen
2. Go to the "Skipshow" tab
3. Enter a show starting with " [ " and ending with " ] "e.g. " [horriblesubs] 
One Piece "
4. Press Save button

What is the expected output? What do you see instead?
- Expected:" [horriblesubs] Bleach " entry in "Skipshow" list
- Result: No entry

What version of the product are you using? On what operating system?
- Beta 0.5.5 on Windows 7 X64 Ultimate
- Problem persisted also in 0.5.3

Please provide any additional information below.

It seems that the combination of the " [ " symbol and " ] " cannot be entered 
in the "Skipshow" list manually nor automatically.
Probably because many files end with [720p] or [1080p] and those extensions are 
purposely omitted when deteting Shownames, so it's probably by design.

It only applies to shows starting, from character 1, with a " [ " and ending 
somewhere with " ] " e.g. [test]or [t]est.
Shows only starting with " [ " are entered correctly.
Shows only endig with " ] " are entered correctly.
Show containing " [] " in the middle or end are entred correctly.
Other symbols"like " { " and " } " or " \ " or " | " don't pose any problems. 

The files are correctly detected by Auto-Sub by the way. Filenames start indeed 
with [.....] .....
Renaming the files is the work-around.

I don't know if it's possible to solve this, but thanks in advance for reading 
this and possible come up with a solution.

Greetings Patrick.  

Original issue reported on code.google.com by pvdg...@gmail.com on 3 Jul 2012 at 8:48

GoogleCodeExporter commented 9 years ago
Thanks for reporting, ' [' have to get parsed than.

Original comment by romke.va...@gmail.com on 11 Jul 2012 at 5:23

GoogleCodeExporter commented 9 years ago
Issue 118 has been merged into this issue.

Original comment by romke.va...@gmail.com on 29 Jul 2012 at 8:35

GoogleCodeExporter commented 9 years ago
don't forget to parse ':' when closing this issue.

Original comment by Donny.va...@gmail.com on 30 Jul 2012 at 6:58