Currently the title of the Tos cart & checkout panes as well as the modal window are hard-coded. We can make this configurable without adding a configuration by using the title of the ToS node. The attached patch implements this feature. Thanks!

CommentFileSizeAuthor
#1 tos_title-2.patch2.88 KBezra-g
tos_title.patch2.78 KBezra-g

Comments

ezra-g’s picture

ezra-g commented
StatusFileSize
new tos_title-2.patch2.88 KB

If you're using uc_tos to display a license node called "License" then it doesn't make sense to agree with the Terms of service.

Here's a revised patch that changes the agree checkbox text to "I agree with the terms above."

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented

Ok, makes sense.

The label for the checkbox is passed through translation so anybody who wants can change it, I am changing it because is more generic.

In the first line of the patch should be $node = uc_termsofservice_get_node('cart'); instead of $node = uc_termsofservice_get_node('checkout');

I'm testing this now

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs review » Fixed

Ok, I've also added a title callback for hook_menu so modal frame title is the node title as well instead of the "hardcoded" (:P) string.

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.