• HeadHead [15]
  
  

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

  [16] 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.

  • [17]
  •   [18]
• dberansky [20]
  
  

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

  [21] Posted: 13.03.2003, 05:38

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

  • [22]
  •   [23]
• HeadHead [25]
  
  

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

  [26] 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!

  • [27]
  •   [28]
•  
•  

Users online

• Simon [31],
• Teb [32],
• Wendell [33],
• craigh [34],
• jmvedrine [35],
• rgasch [36],
• shyra [37],
• thezikulan [38]

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