Zikula: A Flexible Open Source Content Management System
home | forum | international support | contact us

Support Forum

Start ::  Developers Corner ::  Module Development ::  An 'old' module, please adivise if you have some minutes...

Moderated by: Support Team

Bottom
An 'old' module, please adivise if you have some minutes...

  • Link to this postingPosted: 28.04.2006, 06:25
    Profile Homepage
    edfel
    rank:
    Professional Professional
    registered:
     
    Status:
    offline
    last visit:
    02.09.07
    Posts:
    831
  • Link to this postingPosted: 28.04.2006, 11:31
    Profile Homepage
    Simon
    rank:
    Steering Committee Steering Committee
    registered:
     December 2002
    Status:
    offline
    last visit:
    03.10.08
    Posts:
    13410
    If it was me, I'd rewrite it from scratch to be API compliant.

    -----
    Regards,
    Simon

    itbegins.co.uk - Zikula Consulting

    Please read the Support Guide
  • Link to this postingPosted: 28.04.2006, 12:26
    Profile Homepage
    Landseer
    rank:
    Steering Committee Steering Committee
    registered:
     January 2003
    Status:
    offline
    last visit:
    17.09.08
    Posts:
    847
    I must agree - everything else would be a total waste of time.

    Frank

    -----
    "He is not dangerous, he just wants to play...."
  • Link to this postingPosted: 28.04.2006, 13:43
    Profile Homepage
    markwest
    rank:
    Moderator Moderator
    registered:
     March 2002
    Status:
    offline
    last visit:
    26.08.08
    Posts:
    7720
    edfelHi:

    Somehow old code. Whats is easy to detect in order to make it run with PN .762 and PHP 5.1x??

    I suspect from:
    1- Module call format
    2- $_SERVER and SELF calls not compatible with PHP 5.1.x
    3- PN DB connections

    So, any tip will be appreciated. It could be a similar 'old' module that was fixed so I can use it as example.


    To this list i'd add

    1) insecure obtaining of variables from input - potential SQL injectctions.
    2) Invalid HTML - certainly not XHTML or sematically correct.

    I'd, like others so far, suggest a from the ground up re-write. Start with the DB structure - if this looks ok - and work from there.

    -Mark
  • Link to this postingPosted: 28.04.2006, 18:07
    Profile Homepage
    edfel
    rank:
    Professional Professional
    registered:
     
    Status:
    offline
    last visit:
    02.09.07
    Posts:
    831
    Hi All:

    Thanks for the feedback!. Will look into the example module to see what can be done.

    Regards!

    icon_wink

Start ::  Developers Corner ::  Module Development ::  An 'old' module, please adivise if you have some minutes...

Main Menu

Extensions Database

Documentation

Development

Login





 


 Log in Problems?
 New User? Sign Up!

Donate to Zikula