Drupal , developing custom module, it is correct way am doing -

-

below drupal custom modules,

can u please confirm it,

is correct way of developing custom module,

else please advise, 

<?php /** *  implementation of hook_form_alter(). */ function register_form_alter(&$form, $form_state, $form_id) {   switch($form_id) {       case 'user_register': // value stole rendered form         // customizations go here          // drupal_set_message('hey, we\'ve tapped form!');          $form['account']['bharani'] = array(             '#title' => 'bharani',             '#type' => 'textfield',             '#description' => t(' bharanikumar custom field '),           );          $form['#submit'][] = 'register_submit_handler'; // add        break;   } }  function register_submit_handler($form, &$form_state) {   $value = $form_state['values']['bharani'];   $mail = $_post['mail'];   $query  = "update users set language='$value' mail='$mail'";   db_query($query); }   ?>

i not answer "correct way of developing custom module" part of question, here note way you're doing sql query :

you using :

$value = $form_state['values']['bharani']; $mail = $_post['mail']; $query  = "update users set language='$value' mail='$mail'"; db_query($query);

with this, code subject sql-injections : no matter users send $_post['mail'], it'll endup in query, un-escaped !


drupal , db_query(), should, instead, use :

$value = $form_state['values']['bharani']; $mail = $form_state['values']['mail'];; $query  = "update users set language='%s' mail='%s'"; db_query($query, $value, $mail);

this way, drupal take care of escaping, protecting sql-injections.


Comments

Post a Comment

Popular posts from this blog

c# - How to set Z index when using WPF DrawingContext? -

-
how set z-index drawing object when using drawingcontext.drawxxx() methods? the object drawn last have higher z index. can't change index of drawn objects. way draw in order. if using wpf (as placed tag), can use, example, canvas control. create shapes need like polyline obj = new polyline(); //... // ... set properties of obj and add them canvas uielementcollection: yourcanvasname.children.add(obj); //or yourcanvasname.children.insert(i, obj); first items of collection have higher z index. advantages in way: no need redraw on window changes, can anytime move objects , change order.
Read more

razor - Is this a bug in WebMatrix PageData? -

-
i think may have found bug in webmatrix's pagedata, not sure. concerns how pass data partial page calling page. in webmatrix documentation (tutorials, e.g. " 3 - creating consistent look ", , example code), pagedata recommended mechanism pass data between pages (e.g. content page layout page, or partial page). however have found not work other way, pass data partial page calling page. modifying or adding entries in pagedata in partial page, not seem calling page. cutting right down simplest possible example, in test page may have this: @{ pagedata["test"] = "initial entry"; } <p>before calling partial page, test value @pagedata["test"]</p> @renderpage("_testpartial.cshtml") <p>after returning calling page, test value @pagedata["test"]</p> and in _testpartial.cshtml page might have this: @{ pagedata["test"] = "modified entry"; } <p>in partial page...
Read more

visual c++ - Using relative values in array sorting ( asm ) -

-
i need sort through array , sort each individual row in array in ascending order. doesn't seem going (surprise!) keep getting hit 2 errors: a2101: cannot add 2 relocatable labels , a2026: constant expected here's sort, makes sense me, think i'm still trying implement high-lvl language techniqes assembly. there way around not being able use relative values? (the array 7 rows 9 columns, btw). mov cx, 7; cx = number of rows outer: ; outer loop walk through rows push cx mov cx, 9 mov row, cx ;rows middle: ; middle-loop walk through columns push cx sub cx, 1 ;cx = cx-1 mov column, cx ;columns inner: ;inner loop - compare , exchange column values cmp marray[row*9 + column], marray[row*9 + column+1] xchg marray[row*9 + column+1], marray[row*9 + column] ; compare , exchange values marray table inc column loop inner pop cx loop mid...
Read more