• Previous message: [greg-dev] http://svn.gregarius.net/trac/ticket/320
  • Next message: [greg-dev] http://svn.gregarius.net/trac/ticket/320
  • Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

    [greg-dev] http://svn.gregarius.net/trac/ticket/320

    For anybody following this on the mailing list, please refer to trac for 
    my latest patch and comments.
    
    keith d. zimmerman wrote:
    > A proposed patch is up at http://gregarius.dropcode.net/trac/ticket/320
    >
    > Thanks in advance to all who look through it and provide advice.  This 
    > patch is only for the configuration part; I'm still working on the 
    > Activated/Deactivated parts.
    >
    > -kz
    >
    > keith d. zimmerman wrote:
    >> Ok, I think I followed this thread well enough.  I'm not very 
    >> experienced with php, so I'll need a bit of validation for the patch 
    >> I am making.  I started this patch today, and I intend to use this 
    >> code to generate the plugin function prefix from the ///Name: metadata.
    >>  preg_replace( "/^[-0-9]+|[^a-z0-9-_]/i", "", $plugin_info['name'] )
    >> Now Chris says that str_replace is better, and I'll use it if i 
    >> can... I can't figure out to invert the sense like I can with 
    >> preg_replace, though.  (I want to replace everything other than 0-9, 
    >> a-z, _ and -)  The ^[-0-9]+| part is because I believe a function 
    >> name can't start with a number (not sure of this either).
    >>
    >> I plan to prefix this value with a double underscore, and append 
    >> _Configuration, _Activated and _Deactivated to the end.  So a plugin 
    >> with
    >>  /// Name: Printer Friendly URLs
    >> could define functions called __PrinterFriendlyURLs_Configuration, 
    >> __PrinterFriendlyURLs_Activated, and 
    >> __PrinterFriendlyURLs_Deactivated that would "magically" be called.
    >>
    >> Does this all make sense?   I have already started work on this 
    >> patch, so I'll likely post a preliminary version to trac before too 
    >> long.
    >>
    >> thanks all,
    >> -kz
    
    
    Posted by keith d. zimmerman [reply] at Mon Jan 9 22:09:02 CET 2006