A couple bugs.

1) The mysql options handling is wrong. You don’t put a space between -u and -p and the user/pass. The block at the end needs to look like this.

MYSQLOPTS=”$MYSQLDB -u$MYSQLUSER”
if [[ -n "$MYSQLPASS" ]]; then
MYSQLOPTS=”$MYSQLOPTS -p$MYSQLPASS”
fi;

The username was also hardcoded to root. I have corrected that in the code above.

2) The directories for movies and TV dirs require a trailing slash for “find’ to find the files. The comments say you don’t need one, and refer to it as a backslash, which is true I suppose, but UNIX systems don’t use a backslash as a directory separator. Examples that work:

# Directory where your TV files are stored. Do not use a trailing
# backslash.
TVDIR=”/data/mythtv/videos/TV/”

# Directory where your movie files are stored. Do not use a
# trailing backslash.
MOVIEDIR=”/data/mythtv/videos/Movies/”

3) SQL queries need to be sanitized. Any filenames with a single quote in them screw up the mysql calls. You should also be checking for SQL injection, just to be safe.

Tested on Mythbuntu 9.10. Can movies be parsed using the filename as well? It would be nice.

I’m running Myth .22 on Ubuntu 9.10 what’s the recommended way of installing convert, mkvinfo & mkvextract – As I’ve just tried running your script and get the missing dependency for those items.

Thanks

I’ve installed both via
apt-get install mkvtoolnix imagemagick

Cheers

Nice idea, but I’m having a big problem.

1. It matches my movie titles, but not my TV titles. I end up with _NO_TITLE and no metadata.

Doesn’t match on the following TV show patterns, for example:
Angel.1×01.city_of.avi
Angel.s05e15.avi
deadwood.s01.e02.avi

It’s all episodes. Not a single one gets through. I’ve got about a dozen shows with 1-7 seasons each. I think those samples are representative of how I named them.

@Benjamin: I was having the same problem as Donald – to fix this I needed the remove the double quotes that were around the regular expressions. so instead of =~”blahblah” ]] it’s just =~ blahblah ]]

From tldp.org :
As of version 3.2 of Bash, expression to match no longer quoted.

In addition to the quote thing (I’m running into it as well, since I’m on ubuntu 9.10) I’ve found that, when importing TV shows, if you have filenames that don’t make it into any of the filters, but those filenames come *after* files that have, that you’ll get lots of weirdness.

For instance, the first file it processes is:

Blah.S01E01.mkv

This is fine; TITLE is Blah, SEASON is 01, and EPISODE is 01.

If the next file it processes is:

Llama.2009.03.28.mkv

The script does not have a handler for that format, and will skip down to inserting. At that point, it uses the prior file’s values and put that show into the database as Blah, 1, 1 (and pulling the appropriate values from TVRage for that).

A simple thing to get around. Just add the following at the top of the insertShow function to initialize things:

TITLE=”";
SEASON=”";
EPISODE=”";

Another problem I’ve had is that I was still getting some shows crossed up. I tracked it down to an import of different shows that had the same season and episode number.

If you look in the insertShow function, and look for:

if [ "$SEARCHRESULT" = "" ]; then
SHOWDATAFILE=”${DATADIR}/showData_${SEARCHTITLE}_${SEASON}x${EPISODE}.tmp”

the next line establishes a name for a temporary storage file. However, it uses a previously undefined variable named ${SEARCHTITLE}. Since it hadn’t been assigned anything it was always blank. So, when the second show with the same season/episode number came along, it reused the existing tmp file.

I’m a newb at bash scripting, so I took a brute force approach, and changed my file to read:

if [ "$SEARCHRESULT" = "" ]; then
RANDOM_VAR=`echo “${FILENAME}” | sed ‘s/ //g’`
SHOWDATAFILE=”${DATADIR}/showData_${RANDOM_VAR}.tmp”

Basically, I yanked all white spaces out of the filename using sed and assigned that to a throw away variable named RANDOM_VAR. Then, I tacked on some other stuff and away it goes.. That should get me unique enough.

Hi there,

Was hoping this script was the end of my issues with MythVideo, however not quite yet Could you support Sickbeard’s naming format? Its not overly configurable im afraid – examples below:

Sleeping 2…
Inserting: Angry Boys – 01×04.avi…
Sleeping 2…
Inserting: Angry Boys – 01×03.avi…

Using spaces is the default, with an option to replace with periods instead of spaces.

Thanks,
iMx

Can you please change all filepath variables to be double quoted as to support folder names that include spaces

ie. ${TVDIR} becomes “${TVDIR}”

Opps, for subtitle recording rules to take effect you also need to set duplicate = 1 into your SQL

ie. SQL=”insert into recorded (commflagged,filesize,title,subtitle,description,basename,storagegroup,starttime,endtime,originalairdate,progstart,progend,chanid,duplicate) values”
SQL=”$SQL (3,$FILESIZE,’`echo $TITLE |sed “s/’/\\\\\’/g”`’,'`echo $SUBTITLE |sed “s/’/\\\\\’/g”`’,'`echo $DESC |sed “s/’/\\\\\’/g”`’,'`echo mmm.tv.$FILENAME |sed “s/’/\\\\\’/g”`’,'$RECORDINGGROUP’,NOW(),NOW(),’$AIRDATE’,NOW(),NOW(),$CHANID,1);”

You may also wish to insert into oldrecorded to keep track of episodes after they are deleted….

Would it be possible to support TV file paths like “Show Title\SeasonXX\SXXEXX – Episode Title.xxx” ?

The script works great, thanks!

I have a request: is it possible to have the frontend update the “Watch Recordings” screen after the file is inserted into the database?
Currently, I do not see the newly added file until I re-enter the “Watch Recordings” screen and it reloads the recorded list. I know it is possible because the list gets updated when a new recording starts.

Or if you have any ideas how to do it, I can hack at it.

Thanks.

I love this little shell script… My only complaint is that I can’t delete the shows off of mythtv once I watch them without them showing back up. Can you add an option to not re-add shows. I was thinking you could just add a new table and add a check there before checking the recordings table

Is there a way to get the script to pull the TV show’s description as well?

I seem to have aproblem that the are not falling nicely into the programs classification. All the shows are in the main Programs listing.

Otherwise great work. (I’m on the latest mythtv)

@Brad also make sure your mythtv user has writes to delete the file. I have to do a chmod on the files otherwise it does not delete them.

FYI, I started using Mythtv-0.26 where they migrated to storing all date/times in UTC (see http://www.mythtv.org/wiki/UTC ). Thus, in the script all of the times are off when shown in the frontend. My ugly hack was to replace all occurrences of:
NOW()
with
CONVERT_TZ(NOW(), ‘America/Los_Angeles’, ‘UTC’)