I made the following improvements for a project I've worked on so here they are:

  • Fixes:
    • Code cleanup to meet Drupal coding standards (mostly indentation).
    • Converted width and height elements to varchar so other units than pixels can be specified. Defaults to pixels through HTML attributes.
    • Allowed default values to be set for all field elements even when the ID is empty.
    • Escaped all iframe URL parameters and attributes through check_plain to avoid code injection.
    • Fixes on field names and descriptions to make everything more clear.
  • Improvements:
    • Automatic i18n support (as supported by the Wikiloc site). Fallback to English for Wikiloc unsupported languages.
    • Added validation to check that ID element is numeric.
    • Moved module into Fields package.

You have to run update.php after applying the patch.

Comments

gonssal’s picture

StatusFileSize
new11.42 KB
gonssal’s picture

Issue summary: View changes
jmaties’s picture

Not received notifications!!! :(

Sorry and thanks

jmaties’s picture

Status: Needs review » Fixed

Patched

Status: Fixed » Closed (fixed)

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