[PATCH] Zip file names should match the latest version of their contents, not of the whole repository

[Emmanuel Blot]

The original ticket appears as ​th:ticket:558.

I don't know which solution is the best to implement such a feature (renaming feature as Alec suggested, …), but the current implemention is a real issue on multi-project host servers such as TracHacks.

[Alec Thomas]

This patch works, and I've enabled it on TracHacks:

trac/versioncontrol/web_ui/changeset.py

@@ -228 +228 @@
             req.perm.assert_permission('FILE_VIEW')
             # choosing an appropriate filename
             rpath = new_path.replace('/','_')
-            if chgset:
+            if 'filename' in req.args:
+                filename = req.args['filename']
+            elif chgset:
                 if restricted:
                     filename = 'changeset_%s_r%s' % (rpath, new)
                 else:

[john.m.camara]

zip_rev.patch sets the rev number based on history instead of the latest rev of the repository.

[Emmanuel Blot]

Not for 0.11.

[Emmanuel Blot]

Btw, please do not update the ticket description when you add a patch. Use [PATCH] when submitting a new ticket only, thanks.

[osimons]

+1 on the idea. I'll put it on my todo.

[osimons]

See also #5119.

[izzysoft@…]

If you are once at the file names:

It would also be nice to be able to specify part of the file name. Finding files named trunk-r123.zip on my disk would drive me crazy ;) I think of a new parameter like "basename", so adding "&basename=something" would result in something-trunk-r123.zip then in the given example.

Next step would be to allow a "default basename" to be defined in your trac.ini, so the pages would render it this way by default.

[Christian Boos]

Moved together with #5119.