When editing content, there's a odd space due to a missing short filter tip callback:

Tips Callback

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

FluxSauce created an issue. See original summary.

FluxSauce’s picture

Status: Active » Needs review
FileSize
1.59 KB
nicrodgers’s picture

Hi FluxSauce, thanks for the patch.

I'm trying to recreate the issue you describe. In a clean Drupal install, I installed and enabled shortcode-7.x-2.x.

I then enabled the Shortcode filter on an input format, then went to the node add basic page, selected the input format and observed the results. I saw an empty blank line as per your screenshot.

I then applied your patch, which applied cleanly, and refreshed the node add page. I see 'Render Shortcodes' instead of the empty line.

However, even with the patch applied, when $long == TRUE, an empty line is still produced.

This is caused when there are no shortcodes enabled, or no shortcodes enabled for the selected input filter, as there is no check for whether $shortcodes is empty or not and returns a list regardless.

Thinking it through some more, I can see why the original code doesn't distinguish between $long == TRUE and $long == FALSE, as in both cases it's useful to display the shortcodes that are set up. Therefore, I'm attaching a new patch, which is more closely based on the original code but updated to check if one or more shortcode is actually enabled.

  • Denes.Szabo committed 5cd57f3 on 7.x-2.x
    Issue #2594175 Denes.Szabo: Add DCS fixes
    
  • Denes.Szabo committed c8937cb on 7.x-2.x authored by nicrodgers
    Issue #2594175 by nicrodgers: Shortcode filter tips missing short tip
    
  • Denes.Szabo committed ca0e5be on 7.x-2.x authored by FluxSauce
    Issue #2594175 by FluxSauce: Shortcode filter tips missing short tip
    
  • Denes.Szabo committed dad248f on 7.x-2.x
    Issue #2594175 Denes.Szabo: Shortcode filter tips missing short tip -...
Denes.Szabo’s picture

Status: Needs review » Fixed
Denes.Szabo’s picture

Status: Fixed » Closed (fixed)