Zikula Community

• HeadHead
  
  

  • Rank: Freshman
  • Registered: 05.03.03
  • last visit: 16.03.03
  • Posts: 3

  Posted: 12.03.2003, 21:06

  I'm beginning work on a module which adds four new tables to the database. I've gotten my initialization routine to work nicely, adding the tables and session vars that I need. But when I test "Remove", it reports that the module was removed, but the tables (and Session vars) remain in the database.
  
  It appears that the module_delete() function in my pninit.php file doesn't seem to be called at all. If I put my delete code up into my init block, it works without error -- adding and then deleting the same table when I initialise. That rules out a database rights issue, which I also checked.
  
  When I test the Template module, it works perfectly as well. When I copy the Template delete function and just change the module from "template" to my module name, and change the name of the pnTable name, I get the previous result: PN reports a sucessful removal (no error message -- and I test for DB errors), but no change to the database.
  
  Thanks in advance for any tips for validating my init and delete functions to get this working.

  • 
  •  
• dberansky
  
  

  • Rank: Professional
  • Registered: 27.06.02
  • last visit: 02.04.03
  • Posts: 746

  Posted: 13.03.2003, 05:38

  it has to work. check and double check for stupid errors, like typos, etc., especially the function name.

  • 
  •  
• HeadHead
  
  

  • Rank: Freshman
  • Registered: 05.03.03
  • last visit: 16.03.03
  • Posts: 3

  Posted: 15.03.2003, 17:40

  I ended up removing all of the "function module_upgrade($oldversion)" code that I had modified only to comment out the template example code. It didn't appear that this commented out my module_delete() function, but once I got rid of that code, the delete code worked perfectly.
  
  Thanks for the pointer!

  • 
  •  
•  
•  

Users online

• Simon,
• Teb,
• Wendell,
• craigh,
• dks,
• jmvedrine,
• shyra,
• thezikulan,
• videokid

This list is based on the users active over the last 60 minutes.