[PATCH] Make Transmission look for the web ui folder in the Application Support folder first

[Waldorf]

I think that being able to easily replace the web interface would be a great addition to Transmission, and doesn't add bloat.

Currently, the only way to change the web ui on the mac is to mod the web folder inside the app bundle, which isn't recommended. Let alone the (potential) permission issues. Or, by setting an environment variable. Which isn't a regular user will easily be able to do.

So, I recommend that T looks for a web ui in ~/Application Support/transmission/web first, and if that's empty, look inside the app bundle.

[Waldorf]

I ran into an issue that the appString was freed before it could be read by tr_buildPath(). Both CFRelease() functions should be moved to the bottom of the mac specific part, wether or not the patch is used. (I think)

I added a second version, using an alternative method of getting the c-string from a CFString. I'm not sure wether it's better or different...

[Waldorf]

Small note: The folder T should be looking for in the Application Support folder should be Web. (Tittle Case)

[livings124]

Thanks! r10417

[Waldorf]

As I mentioned in the second comment, to be conform to the naming in of the support folders (eg. Torrents, Incomplete, Resume...), the web ui folder should be named Web. With capital "W". (Though blocklists seems to be an exception)

Line 550​:

s = tr_buildPath( tr_sessionGetConfigDir( session ), "Web", NULL );

[livings124]

There is no consistency in that directory, and since internally the folder is lowercase, I think that is the best choice. Now if you can make it case-insensitive I would be interested.