Zikula Community

• **unknown user**
  
  

  • Rank: Softmore
  • Registered: Mar 16, 2002
  • Last visit: Oct 21, 2009
  • Posts: 104

  Posted: Jul 18, 2006 - 05:31 AM

  In a module i am working on, I have a form on the admin page to update records in the DB. A part of this form is generated dynamically based on what is in the database for that record.
  
  
  Kinda like the phpbb poll form. Where it generates x number of fields depending on how many questions you have in your poll.
  
  
  My question is, how do I pass those variables from the form to the update function? I'm working from the example module and the way that is set up, I need to list all the varibles before I can use them. But I obviously cannot do that if I don't know how many variables there will be for a given record.
  
  Here is the piece of code I'm talking about:
  
  

  Code

  function sked_admin_updatestuff($args)
  {
  
      // Get parameters
      list($sid,
           $s_preview,
           $s_review,
           $link_id1,
           $link_type1,
       $link_lbl1,
       $link_lnk1,
       $link_lang1) = pnVarCleanFromInput('sid',
                                              's_preview',
                                              's_review',
                                              'link_id1',
                                              'link_type1',
                                              'link_lbl2',
                                              'link_lnk1',
                                              'link_lang1');
     
      extract($args);

  
  
  Is there a better way I should be doing this? Any advice would be great. thanks.
  
  
  
  
  
  
  
  
  edited by: rowjoe, Jul 18, 2006 - 05:33 PM

  • 
  •  
• **unknown user**
  
  

  • Rank: Senior
  • Registered: Mar 16, 2002
  • Last visit: Oct 21, 2009
  • Posts: 2330

  Posted: Jul 19, 2006 - 02:30 AM

  In your template form, use inputs with names such as:
  
  

  Code

  <input type="text" name="form_data[this]" value="" />
  <input type="text" name="form_data[that]" value="" />
  <input type="text" name="form_data[theother]" value="" />

  
  
  That way, in the PHP (update function) you can simply grab a single variable as:
  
  

  Code

  sked_admin_updatestuff($args)
  {
      $woo_hoo = pnVarCleanFromInput('form_data');
  
      // And the rest of the code...
  
  }

  
  
  ...this will grab all the form data in a nice, neat single variable, no matter how many (or few) fields there are.

  • 
  •  
• cyber_wolf
  
  

  • Rank: Developer
  • Registered: Jun 03, 2002
  • Last visit: Dec 21, 2009
  • Posts: 284

  Posted: Jul 19, 2006 - 02:37 AM

  If you don't know how many rows there will be, you can use arrays. In your form, in the input, textarea, select rows; use as your parameter....
  
  

  Code

  name="link_id[]"
  name="link_type[]"
  name="link_lbl[]"
  name="link_lnk[]"
  name="link_lang[]"

  
  
  Then in your PHP code.....
  
  

  Code

  list($link_id,
           $link_type,
           $link_lbl,
           $link_lnk,
           $link_lang) = pnVarCleanFromInput('link_id',
                                             'link_type',
                                             'link_lbl',
                                             'link_lnk',
                                             'link_lang')

  
  
  Then use a foreach statement to loop through them.
  
  --
  cyber_wolf
  www.bkbsolutions.com - My Zikula module development site.

  • 
  •  
• **unknown user**
  
  

  • Rank: Senior
  • Registered: Mar 16, 2002
  • Last visit: Oct 21, 2009
  • Posts: 2330

  Posted: Jul 19, 2006 - 04:23 AM

  cyber_wolf

  If you don't know how many rows there will be, you can use arrays.

  
  
  The method I posted is /somewhat/ the same, but uses less code.
  
  :)
  

  • 
  •  
• cyber_wolf
  
  

  • Rank: Developer
  • Registered: Jun 03, 2002
  • Last visit: Dec 21, 2009
  • Posts: 284

  Posted: Jul 19, 2006 - 02:13 PM

  We both were writing our methods at the same time; but yes, your way would be more efficient. I would have suggested that method, but I think I read too much into rowjow's question. I thought he was asking how to do 'x' rows with 5 fields per row and my method has been the only way I have come up for that type of a complex form. I've used your way many times myself for simple 1 field per row forms.
  
  --
  cyber_wolf
  www.bkbsolutions.com - My Zikula module development site.

  • 
  •  
•  
•  

Users on-line

• internetking

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