Plugin Directory


Opened 10 years ago

Last modified 10 years ago

#727 assigned enhancement

combine link_summary() and get_link_summary()

Reported by: Ammaletu Owned by: GrafZahl
Priority: low Severity: minor
Plugin: link-summarizer Keywords:
Cc: johannes@…


I wanted to change some details of the plugin (more to come if you like) and, to make these changes easier, started with combining the two functions link_summary() and get_link_summary() which were virtually equal. I've attached the changed plugin file, based on the current development version. Both functions now use lnsum_generate_link_summary() to get their output. If you want to incorporate this into your plugin, you might want to check my use of tab characters (I'm using spaces usually).

Attachments (2)

link-summarizer-combined-functions.php (14.3 KB) - added by Ammaletu 10 years ago.
plugin with new function lnsum_generate_link_summary
link-summarizer-further-changes.php (15.3 KB) - added by Ammaletu 10 years ago.
plugin with further changes, see comment

Download all attachments as: .zip

Change History (6)

@Ammaletu10 years ago

plugin with new function lnsum_generate_link_summary

comment:1 @Ammaletu10 years ago

  • Cc johannes@… added; jr@… removed

@Ammaletu10 years ago

plugin with further changes, see comment

comment:2 @Ammaletu10 years ago

I did some further changes (based on my previous change):

  • accepts both '...' and "..." as attribute delimiters (I didn't filter for wrong cases like '...", they are accepted as well)
  • doesn't copy the ID attribute of links (because it's supposed to be unique)
  • if the link has no title attribute and the URL is used as new link text, the original link text is used as title attribute

comment:3 @GrafZahl10 years ago

  • Status changed from new to assigned

comment:4 @GrafZahl10 years ago

The suggestions on combining the two functions into one and call it via wrapper functions has been incorporated and will be contained in the next release (1.5).
For your suggestions regarding link attributes I will have to take a deeper look at them. Maybe they'll be also included into 1.5

Note: See TracTickets for help on using tickets.