Right now, the qr_codes/0 block doesn't do much of anything unless token is available. I propose making this module depend on token being installed.

Currently, if you go to the block configure page, the description on "Enter the text to embed in your barcode:" reads:

Available token replacement patterns are listed below and will be applied on node or user pages.

Which doesn't make sense unless token is available, since there are no tokens.

Comments

AlexisWilke’s picture

Also there should be a default of including the current page URL or the site email when the block $data is empty because otherwise the barcode represents an empty string which is fairly useless.

However, the token module is not required if all you want to show on your pages is an email address that doesn't require a token to get generated. Yet I agree that the message in the block should change depending on whether the token module is available or not.

Thank you.
Alexis