Unused filter afs-import_details ln 382 in class.add-from-server

  • Resolved Andrew

    (@simbolo)


    12 years, 5 months ago

    Hi,

    Within the plugin I was going through the code and noticed on line 382 within the file class.add-from-server.php the following line:

    $attachment = apply_filters('afs-import_details', $attachment, $file, $post_id, $import_date);

    As far as I can tell afs-import_details doesn’t exist. I could certainly be missing something but just thought I’d point it out for code cleanup. I’m thinking it is old because the prefix is using “afs” instead of the more prominent “frmsvr” found elsewhere in the plugin.

    🙂

    Best,

    Andrew

    http://wordpress.org/extend/plugins/add-from-server/

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author Dion Hulse

    (@dd32)

    Meta Developer

    12 years, 5 months ago

    Filters are just names which are called, If there’s nothing hooked to it (The plugin itself doesn’t use it, it’s simply there for other plugins and users to filter the attachment details) then it doesn’t do anything (other than pass $attachment straight through).

    The prefix is different from the rest, but unfortunately, since it’s already written/released like that, it’ll be staying that way, just so as not to break anyone who’s using it!

    Thread Starter Andrew

    (@simbolo)

    12 years, 5 months ago

    Cool, sounds like a god reason to me. Thanks for responding.

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘Unused filter afs-import_details ln 382 in class.add-from-server’ is closed to new replies.