Viewing 15 replies - 1 through 15 (of 30 total)
  • @manu1033,

    I like the idea but rather than enforcing it on all users I think we’d need to deploy this via anew {KEYWORD}. I’ll take a look at your patch and give it a test and see what I can do for the next version.

    Thread Starter manu1033

    (@manu1033)

    Well, one could argue that links are really part of {POST}. Perhaps an option in the admin.php?page=s2 control panel? We currently have:
    – HTML Full
    – HTML Excerpt
    – Plain Text Full
    – Plain Text Excerpt

    We could have this:
    – HTML Full
    – HTML Excerpt
    – Plain Text Ful with links
    – Plain Text Full without links
    – Plain Text Excerpt

    @manu1033,

    I’ve been looking at the patch, it works well on the full length but not good in the excerpt. Based on the email options you’ve added above it seems you feel the same.

    I’m not sure adding another email type is the way forward, it might get too confusing for users. Adding in a new keyword allows the blog author to make the decision about the layout of the emails offered and it can also be coded to not mess with the excerpt at all.

    Thread Starter manu1033

    (@manu1033)

    As you wish. Will you do the {KEYWORD} part, or should I look at it?
    I fear the excerpt case will be difficult to fix.

    @manu1033,

    I think I’ve got both parts covered. I’ll make a commit later and then you can have a look at the result.

    Thread Starter manu1033

    (@manu1033)

    Great. Where can I checkout latest code?

    @manu1033,

    Changes have just been committed.

    Thread Starter manu1033

    (@manu1033)

    I suspect you forgot to commit the bits in classes/class-s2-core.php. I see {REFERENCELINKS} in the settings help, but not the code that should handle it.

    @manu1033,

    You’re quite right, I forgot to apply my patches to that file! Pretty important they are too!!!

    Thread Starter manu1033

    (@manu1033)

    If I add an error_log() in the code I can see it works, however, all users get the result without links. It must be that they get excerpt. I tried bulk-changing, but it seems it is not taken into account.

    Where are subscribe2 user preference stored in database?

    @manu1033,

    Have you added the {REFERENCELINKS} keyword to the email notification template?

    User preferences are stored in usermeta for Registered users.

    Thread Starter manu1033

    (@manu1033)

    It seems some users have no s2_excerpt meta. Does it default to excerpt?

    @manu1033,

    s2_excerpt is a deprecated key and should be getting deleted on updates.

    Thread Starter manu1033

    (@manu1033)

    How is it decided to use excerpt then? I just have theses keys in database:
    mysql> select meta_key from wp_usermeta where meta_key like ‘s2%’ group by meta_key;
    +—————+
    | meta_key |
    +—————+
    | s2_autosub |
    | s2_cat1 |
    | s2_cat1036 |
    | s2_cat3835 |
    | s2_cat54 |
    | s2_excerpt |
    | s2_format |
    | s2_subscribed |
    +—————+

Viewing 15 replies - 1 through 15 (of 30 total)
  • The topic ‘Add links to text version’ is closed to new replies.