• **unknown user**
  
  

  • Rank: Registered User
  • Registered: Mar 16, 2002
  • Last visit: Mar 15, 2003
  • Posts: 3

  [15] Posted: Mar 12, 2003 - 09:06 AM

  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.

  • [16]
  •   [17]
• **unknown user**
  
  

  • Rank: Helper
  • Registered: Mar 16, 2002
  • Last visit: Oct 21, 2009
  • Posts: 718

  [19] Posted: Mar 12, 2003 - 05:38 PM

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

  • [20]
  •   [21]
• **unknown user**
  
  

  • Rank: Registered User
  • Registered: Mar 16, 2002
  • Last visit: Mar 15, 2003
  • Posts: 3

  [23] Posted: Mar 15, 2003 - 05:40 AM

  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!

  • [24]
  •   [25]
•  
•  

Users on-line

• Guite [27],
• mdee [28],
• mercromina [29]

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