Ascoware / get-iplayer-automator

The goal of Get iPlayer Automator is to allow iTunes and your Mac to become the hub for your British Television experience regardless of where in the world you are. Currently, Get iPlayer Automator allows you to download and watch BBC and ITV shows on your Mac. Series-Link/PVR functionality ensures you will never miss your favourite shows. Programmes are fully tagged and added to iTunes automatically upon completion. It is simple and easy to use, and runs on any machine running Mac OS X 10.7 or later. And since the shows are in iTunes, it is extremely easy to transfer them to your iPod, iPhone, or Apple TV allowing you to enjoy your shows on the go or on your television.
GNU General Public License v3.0
144 stars 27 forks source link

v1.22 will not save changed download formats #385

Closed ghost closed 2 years ago

ghost commented 2 years ago

What steps will reproduce the problem? Be specific, and provide as much detail as possible.

What is the expected output? What do you see instead? I expect to see BBC SD (540p) as as an added download format in the list

What version of GiA are you using? On what version of OS X? GiA 1.22 MacOS 12.2.1

Are you using a web proxy? DNS proxy? VPN? VPS? no

What program are you attempting to download (full title, series and episode)? The Perfect Morecambe and Wise (ep 6)

Please provide any additional information below.

sleaux-meaux commented 2 years ago

There are a few errors in the new formats. You'll need to wait for a new release unless you build your own. These are the changes that got it working again for me:

diff --git a/AppController.m b/AppController.m
index 06758a08..4c1023da 100644
--- a/AppController.m
+++ b/AppController.m
@@ -216,9 +216,9 @@ - (void)awakeFromNib

     NSDictionary *newTvFormatMapping = @{
         @"Best" : @"HD (720p)",
-        @"Better" : @"Web (580p)",
-        @"Very Good" : @"Web (580p)",
-        @"Good" : @"Web (580p)",
+        @"Better" : @"SD (540p)",
+        @"Very Good" : @"SD (540p)",
+        @"Good" : @"Web (396p)",
         @"Worst" : @"Mobile (288p)"
     };
diff --git a/BBCDownload.m b/BBCDownload.m
index 9b40f877..a8b93c90 100644
--- a/BBCDownload.m
+++ b/BBCDownload.m
@@ -12,8 +12,8 @@
 @implementation BBCDownload
 + (void)initFormats
 {
-    NSArray *tvFormatKeys = @[@"Full HD (1080p)", @"HD (720p)", @"Web (540p)", @"Mobile (288p)"];
-    NSArray *tvFormatObjects = @[@"fhd",@"hd",@"sd", @"mobile"];
+    NSArray *tvFormatKeys = @[@"Full HD (1080p)", @"HD (720p)", @"SD (540p)", @"Web (396p)", @"Mobile (288p)"];
+    NSArray *tvFormatObjects = @[@"fhd",@"hd",@"sd",@"web",@"mobile"];
     NSArray *radioFormatKeys = @[@"High", @"Standard", @"Medium", @"Low"];
     NSArray *radioFormatObjects = @[@"high", @"std", @"med", @"low"];
raythw commented 2 years ago

I have the same problem. With 1080p and 720p selected, the 540p option actually disappears when you restart the application.

ghost commented 2 years ago

thanks @sleaux-meaux

You'll need to wait for a new release unless you build your own

I have no clue how to build myself so will wait for an update.

thanks!

skovatch commented 2 years ago

@sleaux-meaux can you make that into a pull request? I think I did something wrong merging branches.

ghost commented 2 years ago

Thanks, both!