Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: MythTV: Commits

Ticket #10613: MythVideo file parser no longer recognises the top level directory

 

 

MythTV commits RSS feed   Index | Next | Previous | View Threaded


noreply at mythtv

Apr 16, 2012, 10:02 AM

Post #1 of 17 (357 views)
Permalink
Ticket #10613: MythVideo file parser no longer recognises the top level directory

#10613: MythVideo file parser no longer recognises the top level directory
---------------------------+--------------------------------------
Reporter: jlbergqvist@… | Type: Bug Report - General
Status: new | Priority: minor
Milestone: 0.25.1 | Component: MythTV - General
Version: 0.25-fixes | Severity: medium
Keywords: | Ticket locked: 0
---------------------------+--------------------------------------
Before 0.25, the video browser would fill in basic metadata (show title,
subtitle, episode & season number) from the filename and directory path.
Now, it doesn't recognise the directory above the one the video is in when
filling in metadata, which is a pain if you have a TV show with seasons in
seperate folders. Previously it would recognise this heirarchy fine:

Frasier -> Season 01 -> 1x01 - The Good Son.ext and would grab the show
title (Frasier) the Episode Name and Number from the file name.

as of 0.25, the parser (using this example) ignores everything before the
"Season 01" Directory. So instead of the show title being "Frasier", the
title is "Season 01", which means when you run a metadata scan, it doesn't
pick up any info as it uses "Season 01" as the search term.

Could this behaviour be put back? I don't see why this has changed, and
its only a minor issue.

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Apr 16, 2012, 10:25 AM

Post #2 of 17 (344 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: wagnerrp
Type: Bug Report - General | Status: accepted
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------------+----------------------------
Changes (by wagnerrp):

* owner: => wagnerrp
* status: new => accepted
* component: MythTV - General => MythTV - Video Library
* milestone: 0.25.1 => unknown


Comment:

I'll look through this, but none of that code has changed since before the
0.24 release, and the behavior should be exactly the same as before.

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:1>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Apr 16, 2012, 10:33 AM

Post #3 of 17 (345 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: wagnerrp
Type: Bug Report - General | Status: infoneeded
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------------+----------------------------
Changes (by wagnerrp):

* status: accepted => infoneeded


Comment:

The only potential cause we can think of is a change in the regular
expression behavior in Qt. What version of Qt do you currently have
installed?

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:2>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Apr 16, 2012, 10:37 AM

Post #4 of 17 (352 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: wagnerrp
Type: Bug Report - General | Status: infoneeded
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------------+----------------------------

Comment (by jlbergqvist@…):

My version is: 4:4.7.4-0ubuntu8.1 and i'm using ubuntu 11.10 64-bit.

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:3>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Apr 18, 2012, 3:54 AM

Post #5 of 17 (344 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: wagnerrp
Type: Bug Report - General | Status: infoneeded
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------------+----------------------------

Comment (by jlbergqvist@…):

Any updates? :)

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:4>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Apr 18, 2012, 3:56 AM

Post #6 of 17 (349 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: wagnerrp
Type: Bug Report - General | Status: infoneeded
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 1
------------------------------------+----------------------------
Changes (by stuartm):

* mlocked: 0 => 1


--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:5>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

May 6, 2012, 5:23 PM

Post #7 of 17 (316 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: wagnerrp
Type: Bug Report - General | Status: closed
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution: Invalid
Keywords: | Ticket locked: 0
------------------------------------+----------------------------
Changes (by wagnerrp):

* status: infoneeded => closed
* resolution: => Invalid
* mlocked: 1 => 0


Comment:

The scanner uses the localized versions of the terms "Season" and
"Episode", rather than those terms themselves. That means for example, if
one is using the en-GB translation, the scanner is looking for "Series",
and the use of "Season" would be misinterpreted. The code is working
exactly as designed, if perhaps not completely as expected. I'm closing
this ticket for now, but unlocking it in the event someone wants to come
up with a patch to make the regular expression handle all forms of that
term that are available in the translation files simultaneously.

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:6>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

May 6, 2012, 5:46 PM

Post #8 of 17 (327 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: knight
Type: Bug Report - General | Status: closed
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution: Invalid
Keywords: | Ticket locked: 0
------------------------------------+----------------------------
Changes (by knight):

* owner: wagnerrp => knight


--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:7>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

May 6, 2012, 5:47 PM

Post #9 of 17 (325 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: knight
Type: Bug Report - General | Status: new
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------------+----------------------------
Changes (by knight):

* status: closed => new
* resolution: Invalid =>


--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:8>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

May 6, 2012, 5:47 PM

Post #10 of 17 (318 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: knight
Type: Bug Report - General | Status: assigned
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------------+----------------------------
Changes (by knight):

* status: new => assigned


--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:9>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

May 10, 2012, 5:11 PM

Post #11 of 17 (306 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: knight
Type: Bug Report - General | Status: assigned
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------------+----------------------------

Comment (by Nicolas Riendeau <nriendeau@…>):

In [changeset:77d0afc9b155161c402abc328edae86b0c97a797/mythtv]:
{{{
#!CommitTicketReference repository="mythtv"
revision="77d0afc9b155161c402abc328edae86b0c97a797"
Make the video file parser handle both "Series" and "Season" as
equivalent for the English GB translation.

This is the 0.25-fixes version of that fix which attempt to address
this problem while avoiding to break the "Season" matching for the
other translations. The development version will have a more complete
(but more intrusive) fix.

Refs #10613

PS: This is done with the blessing of the English GB translator.
}}}

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:10>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Jul 15, 2012, 8:59 PM

Post #12 of 17 (234 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: knight
Type: Bug Report - General | Status: assigned
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------------+----------------------------

Comment (by Nicolas Riendeau <nriendeau@…>):

In [changeset:de9ede4a8eeaa335584ab466f29f928db51238ba/mythtv]:
{{{
#!CommitTicketReference repository="mythtv"
revision="de9ede4a8eeaa335584ab466f29f928db51238ba"
Add the new strings we will use for video metadata parsing

Refs #10613
}}}

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:11>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Jul 19, 2012, 9:21 AM

Post #13 of 17 (235 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: knight
Type: Bug Report - General | Status: assigned
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------------+----------------------------

Comment (by mikey_31415@…):

Replying to [comment:11 Nicolas Riendeau <nriendeau@…>]:
> In [changeset:de9ede4a8eeaa335584ab466f29f928db51238ba/mythtv]:
> {{{
> #!CommitTicketReference repository="mythtv"
revision="de9ede4a8eeaa335584ab466f29f928db51238ba"
>I too have been bothered by the 0.25 release that does not contain the
full file path name in the DNLA published video names. Only the leaf file
name is visible. Rather than arbitrate which strings are going to be
accepted as part of the path name, I propose that the entire path name
(without translation or interpretation) be published. This would return
the usable behavior and features that were contained in the pre-0.25
releases that most people desire from their MythBackend.

The last thing that I want to do is scroll through literally thousands of
video files alphabetically listed by leaf fill names when I’m trying to
find something to watch on my DLNA client devices. Directory hierarchy
helps by creating structure to break large video collections into
something that is actually usable. Quibbling over “Season” or “Series”
isn’t going to fix this!!!

Prediction: Myth will become USELESS to anyone with a sizable collection
without directory hierarchy video file names.
> Refs #10613
> }}}

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:12>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Jul 19, 2012, 9:24 AM

Post #14 of 17 (231 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: knight
Type: Bug Report - General | Status: assigned
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------------+----------------------------

Comment (by mikey_31415@…):

I too have been bothered by the 0.25 release that does not contain the
full file path name in the DNLA published video names. Only the leaf file
name is visible. Rather than arbitrate which strings are going to be
accepted as part of the path name, I propose that the entire path name
(without translation or interpretation) be published. This would return
the usable behavior and features that were contained in the pre-0.25
releases that most people desire from their MythBackend.

The last thing that I want to do is scroll through literally thousands of
video files alphabetically listed by leaf fill names when I’m trying to
find something to watch on my DLNA client devices. Directory hierarchy
helps by creating structure to break large video collections into
something that is actually usable. Quibbling over “Season” or “Series”
isn’t going to fix this!!!

Prediction: Myth will become USELESS to anyone with a sizable collection
without directory hierarchy video file names.

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:13>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Jul 19, 2012, 9:43 AM

Post #15 of 17 (239 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: knight
Type: Bug Report - General | Status: assigned
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution:
Keywords: | Ticket locked: 1
------------------------------------+----------------------------
Changes (by beirdo):

* mlocked: 0 => 1


Comment:

mikey: your comments are off-topic, and discussion should be on the
mailing lists. Please open a new ticket for new issues.

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:14>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Aug 5, 2012, 9:58 PM

Post #16 of 17 (175 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: knight
Type: Bug Report - General | Status: closed
Priority: minor | Milestone: unknown
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution: fixed
Keywords: | Ticket locked: 1
------------------------------------+----------------------------
Changes (by Nicolas Riendeau <nriendeau@…>):

* status: assigned => closed
* resolution: => fixed


Comment:

In [changeset:0f3d088bc185605fb14954b9db9b04506e46ab90/mythtv]:
{{{
#!CommitTicketReference repository="mythtv"
revision="0f3d088bc185605fb14954b9db9b04506e46ab90"
Always recognize Season and Episode in addition to their translation
when parsing video files.

Closes #10613
}}}

--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:15>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits


noreply at mythtv

Aug 18, 2012, 5:56 PM

Post #17 of 17 (139 views)
Permalink
Re: Ticket #10613: MythVideo file parser no longer recognises the top level directory [In reply to]

#10613: MythVideo file parser no longer recognises the top level directory
------------------------------------+----------------------------
Reporter: jlbergqvist@… | Owner: knight
Type: Bug Report - General | Status: closed
Priority: minor | Milestone: 0.26
Component: MythTV - Video Library | Version: 0.25-fixes
Severity: medium | Resolution: fixed
Keywords: | Ticket locked: 1
------------------------------------+----------------------------
Changes (by wagnerrp):

* milestone: unknown => 0.26


--
Ticket URL: <http://code.mythtv.org/trac/ticket/10613#comment:16>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center
_______________________________________________
mythtv-commits mailing list
mythtv-commits [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-commits

MythTV commits RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.