Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Currently, UC requires that all installs on a multisite share a single templates directory (but that's a separate issue).
In order to avoid keeping all my customers' templates in the same SVN repo, it's useful to symlink from the uc_order/templates directory to the real file.
This patch makes that possible.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 387888.04-06-09-12-29.1.x.patch | 1.36 KB | cha0s |
| #3 | invoice_template_list.patch | 1.79 KB | rszrama |
| #2 | 387888.template_gathering.2.x.patch | 1.08 KB | Island Usurper |
| #1 | 387888.template_gathering.2.x.patch | 1.02 KB | cha0s |
| uc_order.linked-template.patch | 418 bytes | xurizaemon | |
Comments
Comment #1
cha0s CreditAttribution: cha0s commentedShould we maybe use Drupal's functions for this? I created a patch using file_scan_directory(). Would appreciate testing regarding symbolic links.
Comment #2
Island Usurper CreditAttribution: Island Usurper commenteduc_order_load_invoice() doesn't support template files in subdirectories, because the original code didn't look for them. file_scan_directory() shouldn't be allowed to recurse down past uc_order/templates.
Comment #3
rszrama CreditAttribution: rszrama commentedFixed a couple of things and ended up with the attached patch. Committed to the 2.x branch. I suppose it will need to be backported.
Comment #4
cha0s CreditAttribution: cha0s commentedZee backport.
Comment #5
rszrama CreditAttribution: rszrama commentedComment #6
TR CreditAttribution: TR commentedI tested the backport posted in #4 and it worked. As this fix has been in 6.x for a long time without a problem, I think we can consider it RTBC. Committed.