Rename Recording

[Pioneer4x4]

Quote[/b] (KingArgyle @ Feb. 28 2005,01:08)]Here is a version of the source code that can be run without requiring the RecordingUtils.dll file.  One of the main methods that the RenameRecording.exe used changed, and was causing problems.  So I moved the small portion that deals with Datbases into the code itself.  I've only included source here.  I'll try and get and update on the Wiki tomorrow.

Note: I've hard coded the path to the GBPVR database, because I needed a quick a dirty test.  It's been busted for the last several days, and I finally became annoyed enough to fix it.  Once I had this working, it was hard for me to not have it still working.  Also, Pioneer your fix for the Comskip should be included in this version as well.  

Again, this is only source code, and not the executable.  You'll need a C# compiler to get it to work.

I see that you have line of code referring to Star Trek with the difference being a ":" Did you figure out that that is the problem with STNG not working with rename? It would make sense, because one other thing that didn't work for me was
"Metropolis: The Power of Cities". I honestly couldn't figure out your code on that part, but the ":" did seem like the item you were catching, but it seemed like it would add it not remove it. I really only understand Basic, but code is code, logic wise. And I can look things up.

Also one of the things I changed was
if (!File.Exists(newFile)) {
archEp.updateScheduleTitle(program.getOID(), newFile);
File.Move(args[0], newFile);
}
I think the program was updating the database, even if the file name existed. I haven't tested that part, and what I was thinking of doing was, if it existed, adding a "_2" or something to the end.

[KingArgyle]

Yeah, when I was recording "Enterprise", the program guide has it has "Star Trek: Enterprise". Since : are illegal characters for filenames, GBPVR removes the : and you get "Star Trek Enterprise_xxxxxxx_xxxxx.xxx" as the filename. Short term fix for me was to add code to look for "Star Trek E" in the title name, and replace it with "Star Trek: E". Long term solution is probably to have a key word list that contains those titles that GBPV strips out the : and add them back in.

The IndexOf function returns a position within the string that the search string occurs. If it doesn't find it, it returns -1.

As I said, this was a quick a dirty fix just so I could get it working on my end.

[Pioneer4x4]

OOOHHHH! I understand. I thought it was adding the :, but I was wondering why. You were just putting it back for the search for the episode name. That also explains why last week's "Lost" didn't work. It was failing on any show that had the illegal characters stripped. (BTW, you need to strip "." also)

I couldn't find out how you were calling comskip directly from this new code, or did I misunderstand your post earlier?

How about keeping the original name, and doing a lookup on it, and using a different variable for the actual rename?

[jrockintuitive]

I was about to do fix the references in RenameRecording, but it looks like you removed the references. I'd imageing I'm off the hook?

[KingArgyle]

Yeah, you are off the hook. It appeared that getProgramsByTitle had a Date check added to it, which was causing problems when trying to use that method in the RenameRecording program since the refactoring on CDK Scheduler was done. No big deal, just drove me insane for a while.

[KingArgyle]

Pioneer,

I don't do a call to comskip directly in this code, I just added your fix for the rename of the comskip file. Again, I didn't test it as I don't use Comskip.

[KingArgyle]

Unfortunately, Tipstir, I don't have the time to develop a plugin for the CDK Scheduler. I'm sure that Jorm and Jrock will iron out the bugs.

As for RenameRecording. I'll post my new version on the Wiki today. Look for a message here once I get it posted.

[Pioneer4x4]

I was going to look at it this afternoon. The patched one that I have been using is working fine. The only thing left I really would like is it catching files it has to remove characters, and I'd like it to run before comskip.

I thank you again for the idea and source!

[KingArgyle]

Pioneer4x4:

I've uploaded the new version to the wiki with the source included. Also included some sample code that can be used to remove the hard coding of the path. Feel free to download and add the missing feature to it (i.e. removing the hard coded path to the config). Just remember to post your modified version on the Wiki. Contact Blader if you need a signon for the Wiki.

RenameRecording

[ZaDDaZ]

KingArgyle,

I hope you don't mind, but I used your code as a template and modified it to remove the hardcoded path, although it needs to be in the GBPVR folder. I also changed some of the exception handling to how I commonly do it. I'm uploading a zip file with the modified source, executable and readme. I haven't gotten into updating the WiKi yet, but I'd be happy to.

I also used your code to write another utility that allows one to specify the maximum number of recordings that are allowed per show. Its on another thread for PriorityRecordings and its called MaxRecordings.

If you don't mind, could you look over this and make sure I didn't make any mistakes that are going to mess up anything?

Cheers