Jump to content

mac_gyver

Staff Alumni
  • Posts

    5,382
  • Joined

  • Days Won

    173

Community Answers

  1. mac_gyver's post in sess_start() makes session variables vanish. was marked as the answer   
    did verifymail.php ever have a sess_start(); call in it?
     
    all your files must be doing the same thing for the session to match up. it's likely that this was initially working because you already had an existing session, using session_start(), that matched up between the files. when you added sess_start() to just the index.php page, that created a new session with a second name, alongside the existing session, and so your verifymail.php had session data. once you finally closed your browser and started over, index,php was using the sess_start() settings, verifymail.php was using the session_start() settings, and there was no matching session data for verifymail.php to use.
     
    most of the code you have shown at the top of index.php is common logic that ALL your pages should use. why don't you have them in a file and require it into each page (Don't Repeat Yourself - DRY) or better yet, if you are at the point of wanting to set up custom session settings, why aren't you using a single file, index.php, to implement your entire site? having a single index.php file site would eliminate the need to even pass data in session variables and would eliminate all the repetitive code and html you are trying to write, test, and maintain.
  2. mac_gyver's post in Process the value of a searched result was marked as the answer   
    do you have php's error_reporting set to E_ALL and display_errors set to ON in the php.ini (putting these settings into your code won't help show syntax errors in the same file where the settings are being set) on your development system so that php would help you by reporting and displaying all the errors it detects?
     
    you would be getting a php syntax error because you are mixing php and html markup inside of php code tags. you would also be getting a blank php page, which you didn't tell us you are getting as a symptom when you run your code. to output in-line html markup, you need to have a closing php tag to switch out of php 'mode'. with 300+ posts, you should be past having these basic problems.
  3. mac_gyver's post in What did I do wrong? was marked as the answer   
    an auto-increment column needs to be an integer data type.
  4. mac_gyver's post in How can i insert data from biomatric device to mysql + PHP was marked as the answer   
    it would be helpful if you define what the work-flow (steps) is (are) before you try to write code. it will also help us if you tell us what the work-flow is supposed to be. something like - 1) connect to device, 2) retrieve user and attendance data, 3) display data in forms, 4) submit form, 5) store submitted data into database table.
     
    are you displaying the data as part of the learning/debugging process and you want to 'automatically' insert the data into the database tables OR are you trying to display the data in a form and you will manually submit the form to cause the data to be inserted into the database tables?
     
    next, you need to separate the different concerns in your code.
     
    any form processing code should be near the top of your file and come before you output any html to the page. the form processing code would come before the <!DOCTYPE tag. all the form processing code would be grouped together.
     
    to display the page, your code is retrieving user and attendance data from the time-clock. the code to retrieve this data and store it in php variables should be grouped together and come before you start outputting the html document. you would then simply loop over this data when you output the html document.
     
    if you are outputting the data in a form, perhaps to allow it to be verified and edited by a manager, you would need to output it using form fields, in a valid form. there are currently no <form...> tag(s) and no data-input form fields in your code. your attendance data output doesn't even have a <table> tag, no <tr></tr> <td></td> tags and no echo statements for the data.
  5. mac_gyver's post in extract image data (BLOB) from imagejpeg() was marked as the answer   
    you would need to use php's output buffering statements.
  6. mac_gyver's post in incremental number for each table row was marked as the answer   
    actually, naming form fields (variables, db column names, or anything else) with an incremental number is a bad design that takes more code to produce the markup and process the data.
     
    you should use an array name for the form fields.  - name="School[]", name="Student[]", and name="Class[]".
     
    this will submit the data as a php array for those field names, that you can simply loop over and access the values from each set of form fields within the group.
  7. mac_gyver's post in insert a query after getting a query was marked as the answer   
    i recommend that you tell or show us what symptom or error you are getting that leads you to believe that something doesn't work. we are not sitting there with you and don't know what it is you are seeing in front of you.
     
    why are you using both the msyql and mysqli extensions in one script? do you even have a connection using the mysql extension? there would be php error's if you don't. do you have php's error_reporting set to E_ALL and display_errors set to ON so that php would help you by reporting and displaying all the errors it detects?
     
    your code should ALWAYS detect and handle errors with database statements, so that it doesn't try to run logic that's dependent on database statements working when they haven't and test to make sure that a query matched a row of data before using the data.
     
    why are you using a multi_query() statement for a single query? this opens the door for sql injection to run ANY type of query, especially since you are not doing anything to protect against sql injection. even if you currently trust the data source, someone at some point can get nefarious data onto the site you are getting your data from. you must ALWAYS protect against sql injection.
     
    btw - the php mysql extension has been removed from php for almost a year. you should not be using it at all at this point in time.
  8. mac_gyver's post in PHP Error - mysqli_result, boolean given was marked as the answer   
    return, the point where the sql syntax error is occurring at is a reserved mysql (database server) keyword. are you sure the original query wasn't as follows, with back-ticks around the table name - 
    "SELECT * FROM `return` WHERE nMemberID='$nMemberID' AND nInfoID='$nInfoID' ORDER BY nReturnID DESC"; next, just converting the mysql_ statements to mysqli equivalents is not enough. you must also safely supply data values to the sql query statement. if $nMemberID or $nInfoID are coming from external data or they could contain any sql special characters, they must either be properly escaped (see the mysqli real escape string function/method) or you need to use a prepared query with place-holders in the sql statement for the data values and then bind the actual data to the place-holders. a prepared query is actually the best choice since it is impossible for injected sql in the data to be treated as sql syntax, whereas using the escape string function must have the proper character encoding set for the database connection to match what the database tables are set up for.
  9. mac_gyver's post in Skipping first line of db was marked as the answer   
    look at your code, between where you are executing the sql query and where you are looping over the data from that query.
  10. mac_gyver's post in jQuery Sortable Portlet Save position to database was marked as the answer   
    so, i went through your code to figure out what it is doing, in order (pun intended) to figure out how to make this work.
     
    starting with your html markup -
     
    1) the id='...' attribute you have for the column div's, should start with a letter, to make them valid. i choose to use id='COL_n', where n = 1, 2, ..., 6
     
    2) the portlet div's needs an id attribute so that the serialize/toArray functions have something to use as data. i choose to use  id='ID_n', where n is the job id auto-increment column value from your database table (if you don't have a job id, you need one.)
     
    3) to allow the php code to dynamically produce the html, for the different status values 1-6, you need a way of mapping the status value to the display label - 'New' through 'To Be Invoiced'. i choose to use an array, and since you (probably) want to output each status section, regardless of if there is any data for it, you would loop over this array to produce the sections on the page, then loop over any data for each section.
     
    the html, from the <div style="clear:both;"></div> to the end of the page should look more like the following (note: data values that you output to the browser should be passed through htmlentities(). this is not in the example code and is left up to you as a programming exercise)  -  
    <?php // define status value to label mapping $status_map = array(); $status_map[1] = 'New'; $status_map[2] = 'Artwork Rec'; $status_map[3] = 'Approved & Ordered'; $status_map[4] = 'In Production'; $status_map[5] = 'Delivered'; $status_map[6] = 'To Be Invoiced'; // query for all the data you want, in the order that you want it $query = "SELECT * FROM jobs ORDER BY status ASC, job_title DESC"; $result = mysqli_query($con,$query); $data = array(); while($row = mysqli_fetch_assoc($result)) {     $data[$row['status']][] = $row; // index/pivot the data using the status value - 1..6 } // note: i used some made-up data in the $data array at this point. the above query code should work, but is untested. // loop over the data and produce the output foreach($status_map as $key=>$label) {     echo "<div class='column' id='COL_$key'>\n";     echo "<h3>$label</h3>\n";     if(isset($data[$key])) // is there any data from the database for this key/status     {         foreach($data[$key] as $row)         {             echo "<div class='portlet' id='ID_{$row['id']}'>\n";             echo    "<div class='portlet-header'>{$row['job_title']}</div>\n";             echo    "<div class='portlet-content'><a href='pdfs/{$row['pdf_link']}' target='_blank'>View PDF</a></div>\n";             echo "</div>\n";         }     }     echo "</div>\n"; } ?> </body> </html> next, the jquery you have that is using an id selector - "#portlet" should be removed since this exercise is operating on a class basis, not an id. also, you would not use the update : method, since this triggers for every column that gets updated. if you move something from one column to another, it triggers two times. you need to use the stop : method. see the following javascript/jquery that i came up with - 
      <script>   $(document).ready(function(){     $( ".column" ).sortable({       connectWith: ".column",       handle: ".portlet-header",       cancel: ".portlet-toggle",       placeholder: "portlet-placeholder ui-corner-all",         stop: function() {             var dat = [];             var i = 0;             $(".column").each(function() {                 dat[i++] = [this.id,$(this).sortable("toArray")]; // this.id is the column id, the 2nd element are the job id's in that column             });                          $.ajax({                 method: "POST",                 url: "save_order.php",                 data: { data: dat }             });         }     });     $( ".portlet" )       .addClass( "ui-widget ui-widget-content ui-helper-clearfix ui-corner-all" )       .find( ".portlet-header" )         .addClass( "ui-widget-header ui-corner-all" )         .prepend( "<span class='ui-icon ui-icon-minusthick portlet-toggle'></span>");     $( ".portlet-toggle" ).on( "click", function() {       var icon = $( this );       icon.toggleClass( "ui-icon-minusthick ui-icon-plusthick" );       icon.closest( ".portlet" ).find( ".portlet-content" ).toggle();     }); });   </script> this will submit an array of data to the .php file, in $_POST['data']. see the following example code that extracts the column number and job id (if any), and logs the information to log.txt -
    <?php // for some reason, the portlet toArray 'adds' an empty element to the start of each array if(isset($_POST['data'])) {     foreach($_POST['data'] as $arr)     {         //$arr[0] is the column id - COL_1, COL_2 (these are the status number 1-6 = New - To Be Invoiced)         //$arr[1] is an array of the ids that are in the column - ID_1, ID_5                  // get the status (column) number         list($not_used,$status) = explode('_',$arr[0]);                  // get the id's in each status/column         $arr[1] = array_filter($arr[1]); // remove empty elements         if(empty($arr[1]))         {             // an empty status/column             $str = "Status: $status, empty";             file_put_contents('log.txt',print_r($str,true)."\n",FILE_APPEND);         }         else         {             // non-empty status/column             foreach($arr[1] as $element)             {                 // get the id number                 list($not_used,$id) = explode('_',$element);                 $str = "Status: $status, Id: $id";                 file_put_contents('log.txt',print_r($str,true)."\n",FILE_APPEND);             }         }     } }
  11. mac_gyver's post in Hi, how to have jquery check if one of several select menus have quantity select allow submit... was marked as the answer   
    assuming that you want to look at all the select/option menus and they all have class='qty', you would loop over each of the elements with that class - 
    $(document).ready( function() {     $('input[type="image"]').click( function()     { var total = 0; $( ".qty" ).each(function() { total += Number($( this ).val()); });         if(total == 0)         {             alert("You have not selected a product quantity!");             return false;         }     }); });
  12. mac_gyver's post in mysql query not working with drop down boxes was marked as the answer   
    promised example code - 
    // define, or get from an sql query, the choices for the select/option menus $author_data = array(); $author_data[] ="ken davies"; $author_data[] = "arthur smith"; $author_data[] ="gill rafferty"; $author_data[] ="molly brown"; $author_data[] ="gilbert riley"; // define and populate the data for the other select/option menus. left up to you as a programming exercise. $genre_data = array(); $year_data = range(2002,2008); $publisher_data = array(); // define a list/array of expected inputs for the dynamic WHERE clause $choices = array(); $choices[] = 'author'; $choices[] = 'genre'; $choices[] = 'year'; $choices[] = 'publisher'; // produce the dynamic WHERE clause $and_terms = array(); // holds each term for the WHERE clause // loop over the defining array of choices foreach($choices as $choice) {     if(isset($_GET[$choice]))     {         // the choice is set, add it to the terms to use         $and_terms[] = "`$choice` = '$_GET[$choice]'"; // you need to either - validate that the input value is exactly one of the possible choices, properly escape the value using the database escape string function, or use a prepared query to supply the value to the sql statement. left up to you as a programming exercise.     } } // db  connection - convert to PDO and use exceptions to handle errors. left up to you as a programming exercise. $con = mysql_connect("localhost","root",""); If (!$con){     die("Can not Connect with database" .  mysql_error()); } mysql_select_db("authors",$con); $where_clause = !empty($and_terms) ? 'WHERE ' . implode(' AND ', $and_terms) : ''; // build the sql query statement $sql = "SELECT * FROM books $where_clause";      // execute the query and retrieve the data into an array $result_data = array(); $myData = mysql_query($sql,$con); while($row = mysql_fetch_assoc($myData)) {     $result_data[] = $row; } // start the html document/template  ?> <html> <head> <title>My Page</title> </head> <body> <br> <form> <select name="author" size="2"> <?php foreach($author_data as $value) {     $sel = isset($_GET['author']) && $_GET['author'] == $value ? ' selected': '';     echo "<option value='$value'$sel>$value</option>\n"; // you need to apply htmlentities() to any data being output. left up to you as a programming exercise. } ?> </select> <input type = "submit" value = "go"> <select name="genre" size="4"> <?php // use code similar to the above to produce each set of select/option choices. left up to you as a programming exercise. ?> <option value="adventure">adventure</option> <option value="biography">biography</option> <option value="crime">crime</option><br /> <option value="romance">romance</option> <option value="thriller">thriller</option> </select> <input type = "submit" value = "go"> <select name="year" size="4"> <option value="2002">2002</option> <option value="2003">2003</option> <option value="2004">2004</option> <option value="2005">2005</option> <option value="2006">2006</option> <option value="2007">2007</option> <option value="2008">2008</option>                                       </select> <input type = "submit" value = "go"> <select name="publisher" size="4"> <option value="blue parrot">blue parrot</option> <option value="yonkers">yonkers</option> <option value="zoot">zoot</option> </select> <input type = "submit" value = "go"> </form> <?php // you need to use css to style your table border. left up to you as a programming exercise. echo"<table border='3'> <tr> <th>id</th> <th>author</th> <th>title</th> <th>publisher</th> <th>year</th> <th>genre</th> <th>sold</th> </tr>"; foreach($result_data as $row) {     echo "<tr>";     echo "<td>" . $row['id'] . "</td>"; // you need to apply htmlentities() to any data being output. left up to you as a programming exercise.     echo "<td>" . $row['author'] . "</td>";     echo "<td>" . $row['title'] . "</td>";     echo "<td>" . $row['publisher'] . "</td>";     echo "<td>" . $row['year'] . "</td>";     echo "<td>" . $row['genre'] . "</td>";     echo "<td>" . $row['sold'] . "</td>";     echo "<tr />"; } echo "</table>"; ?> </body> </html>
  13. mac_gyver's post in Why Warning "Cannot modify header information" Is NOT Fired ? was marked as the answer   
    For the case you have just posted, php's output buffering is turned on in the php.ini or possibly .htaccess file (if php is running as a server module) on your server. You can determine this by looking at the output from a phpinfo() statement.
  14. mac_gyver's post in My code is broken *somewhere* was marked as the answer   
    textarea's don't have value='...' attributes. you output the existing content in between the <textarea>content goes here</textarea> tags.
  15. mac_gyver's post in Foreach loop doesn't bring all the records from the database was marked as the answer   
    setting php's error_reporting to E_ALL and display_errors to ON (which should already both be set this way in the php.ini on your development system), should help with finding the cause of the problem.
     
    does the 'view source' of the page show all the repeated sections of content? maybe there's a broken html comment or tag that's preventing the output from being rendered by the browser.
     
    also, does the content, that's after the end of the loop, get displayed?
  16. mac_gyver's post in looping results from query and limit amount per row was marked as the answer   
    you will also want to JOIN the image table, whatever its name is, with the products table, so that you can run just ONE query. and did you really name the database column catagory, rather than category?
     
    your code should look like this - 
    $items_per_row = 5; // number of items per output row $query = "SELECT p.id, p.catagory as category, p.name, p.make, i.name as imgName  FROM products p  JOIN $table3 i ON p.id = i.insert_id  ORDER BY p.catagory, p.make, p.name"; // execute query using PDO, $pdo contains an instance/connection to the msyql database server $stmt = $pdo->query($query); // pre-process the data and index it by category, creating an array of arrays with the main index being the category $data = array(); foreach($stmt as $row) {     $data[$row['category']][] = $row; } // produce the output from the data foreach($data as $category => $arr) {     // Create a table with an id name of the category     echo "<table width='100%' id='$category'>\n";     // Write the first row of the table - Category Title     echo "<tr><td><span class='catTitle'>$category</span></td></tr>\n";          // output the rows of data     $chunks = array_chunk($arr, $items_per_row);     foreach($chunks as $chunk)     {         echo "<tr>"; // start a new row         foreach($chunk as $row)         {             // Write new <td> in table with the data of the title             echo '<td width="20%" class="cellPadd"><a href="product.php?id=' . $row['id'] . '"><div class="latest"><div class="latestTop">';             echo "<img class='latestImg' src='images/listings/" . $row['imgName'] . "'  border='0' width='100%' />";             echo '</div><div class="latestBottom">';             echo $row["make"];             echo $row["name"];             echo "</div></div></a></td>\n";         } // complete a partial row         $count = count($chunk);         if($count < $items_per_row)         {             echo str_repeat("<td> </td>\n", $items_per_row - $count);         }         echo "</tr>\n"; // end the row     }     echo "</table>"; // end the table }  
  17. mac_gyver's post in how to make this pagination work with search results? was marked as the answer   
    i don't really think i can help you 'get' what it is you need to do to be able to write code that does something useful. you are randomly changing code, because you haven't taken the time to learn what the statements in the code actually mean and do, then dumping the code on a forum for someone to tell you what to do to fix it. you won't learn anything by getting some to fix your code for you.
     
    some problems in the posted code - you are no longer preparing the query, yet you are trying to bind parameters to it. you are using a wrong PDO defined constant in the bindParm() statement, you are using global $per_page; that does absolutely nothing in the context of this code.
     
    the only thing you are currently doing different, from at the start of this thread, is changing the sql query statement, specifically just the WHERE ... term. the only code you need to change are the lines that are building the WHERE term. the rest of the code needs to remain the same. and even you were doing this for a completely different sql query statement, the only things you would change in the code are the two queries, the SELECT COUNT(*) ... query , that gets the total number of matching rows and the SELECT list of columns ... query, that retrieves the matching data.
     
    this is the code, from reply #10, that is building the WHERE term -  
    // define the possible search fields - this is used to produce a data driven/dynamic design, where you don't write out block after block of code that only differs in the value it operates on $search_fields = array('title','name','description');  $and_terms = array(); // WHERE terms to be AND'ed $params = array(); // bound input parameters for a prepared query foreach($search_fields as $field) {     if(isset($_GET[$field]) && $_GET[$field] != 'ALL') // only if the field is set and it's not 'ALL'     {         // add the search field to the WHERE terms         $and_terms[] = "$field = :$field";         $params[] = array(":$field",$_GET[$field],PDO::PARAM_STR);     } } $where_term = ''; if(!empty($and_terms)) {     $where_term = "WHERE " . implode(' AND ', $and_terms); } all you have to do is change this section of code so that it produces WHERE memberID = :memberID as the where term and then add the correct entry to the $params array for the $_SESSION['memberID'] value. in its simplest, bespoke form, this is what you would have left - 
    $params = array(); // bound input parameters for a prepared query $where_term = "WHERE memberID = :memberID"; $params[] = array(":memberID",$_SESSION['memberID'],PDO::PARAM_INT); the rest of the code, starting with - $query = "SELECT COUNT(*) FROM table $where_term"; doesn't change, unless you want it to do something differently, such as output the data differently or you want the pagination links to be different.
     
  18. mac_gyver's post in Updateing Software Apache/PHP/Mysql was marked as the answer   
    the php.net documentation has migration sections in the appendix that lists the major changes that have been made to the language. if your code is dependent on any of the Backward Incompatible Changes, from the starting php version you were using to the final php version you have now, you will need to rewrite that portion of your code.
     
    we cannot specifically help you since we don't know what exact method, among all the possibilities available in programming, your code is using. you would need to post enough of your code that reproduces each problem to get specific help.
  19. mac_gyver's post in Array Question (Issue)... was marked as the answer   
    you are probably storing some white-space in with the data, which you should be storing not as a comma separated list, but as a separate row in the table for each check box that's checked.
     
    what exactly does the $row['hw_lev'] look like using var_dump($row['hw_lev']);
  20. mac_gyver's post in .= causing errors was marked as the answer   
    that will fix the immediate issue of the simple html dom class not running, when it's not where the error is at. any php error, no matter how benign, prior to using the load() method, will cause this problem.
     
    you should be defining/assigning the first line to the variable before using a concatenate operator on it anyway. every error in your code, even if the reporting/display/logging is turned off, is still being detected and handled by php. the reporting/display/logging is just the last step in the php error handling.
  21. mac_gyver's post in getting error : Catchable fatal error was marked as the answer   
    when you build the query string part of links, you should use a function like http_build_query(). this will let you take any existing $_GET parameters, add/remove/modify any of them, then produce the part of the link after the ? - 
    $get = $_GET; // get a copy of any existing get parameters. you only need to do this once // in your pagination link code, for each link you produce $get['currentpage'] =1; // set the current page to whatever value you want $qs = http_build_query($get, '', '&'); // produce the query string part of the link echo "<span class='rest'> <a href='?$qs><<</a> </span>"; // output the link. note: i removed the use of $_SERVER['HTTP_SELF'] since it is open to cross site scripting and it's not necessary in modern browsers
  22. mac_gyver's post in PHPMailer error on line 3042 class.phpmailer.php was marked as the answer   
    phpmailer should (according to the documentation) work for any php 5+ version (5.4+ if you are using oauth.)
     
    however, assuming you are using the latest version of php mailer, that particular section of code IS using php syntax that was introduced in php 5.3 and will throw a parse/syntax error for lower versions.  the php version check logic they put in that particular code doesn't have any affect because the code never runs when there is a syntax error.
     
    you can edit the section of code for the clearQueuedAddresses method so that it only contains the first set of conditional logic.
     
    you could also put the section of code that's in the else{} conditional statement, that's using the php5.3+ syntax, into an external file and require it in place of the in-line code. the require statement will only be executed if the else{} condition is true and this won't throw a parse/syntax error since the code in the required file is only evaluated when the require statement is executed. 
     
    you can also (at your own risk) download and use older versions of php mailer - https://github.com/PHPMailer/PHPMailer/releases
  23. mac_gyver's post in How do I automate this script to create individual CSV files? was marked as the answer   
    the following implements your logic, saving the output from each input .htm file to a corresponding .csv file - 
    <?php ini_set('display_errors',1); error_reporting(E_ALL); $stats_wanted = array('playedPositionsShort', 'name', 'playerId', 'age', 'rating', 'shotOnTarget', 'shotsTotal', 'penaltyScored', 'penaltyMissed', 'passLongBallAccurate', 'passLongBallTotal', 'passTotal', 'passCrossAccurate', 'passCrossTotal', 'passThroughBallTotal', 'passThroughBallAccurate', 'keyPassTotal', 'dribbleWon', 'dribbleTotal', 'tackleWon', 'tackleLost', 'tackleWonTotal', 'tackleTotalAttempted', 'challengeLost', 'interceptionLost', 'penaltyTaken', 'interceptionAll', 'clearanceTotal', 'offsideGiven', 'offsideProvoked', 'foulGiven', 'foulCommitted', 'dispossessed', 'duelAerialWon', 'duelAerialLost', 'duelAerialTotal', 'touches', 'totalPasses', 'offsideWonPerGame', 'offsideGivenPerGame', 'passSuccessInMatch' ); $keys = array_flip($stats_wanted); // change stats wanted into keys $source_path = 'directory/'; $dest_path = ''; // enter the path, with trailing /, where you want the destination files to be written foreach (glob("$source_path*.*") as $filename) {          echo "Processing: $filename<br>";     $info = pathinfo($filename); // get the ['filename']     $content = json_decode(file_get_contents($filename),true); // read file contents as an array          $output = fopen("$dest_path{$info['filename']}.csv",'w'); // create output file     fputcsv($output, $stats_wanted); // write header line to file          // loop over players in the data     foreach($content['playerTableStats'] as $arr){         $arr = array_intersect_key($arr,$keys); // keep only the stats wanted elements         fputcsv($output, $arr); // write the data to the file     }     fclose($output); // close the current file } this makes the same assumption that your code does, that the order of the $stats_wanted elements matches the order that the data elements will exist as. if not, you can throw in a sort operation before writing the data to the file.
  24. mac_gyver's post in login redirect blank page? was marked as the answer   
    what debugging have you done to narrow down the problem? have you dumped (see: var_dump()) the role value so that you know what if anything it is?
     
    are you sure that login success code is even running for the users with those roles?
     
    if all your code is doing is mapping a set of values to other values, you shouldn't write out conditional logic (if/elseif/switch/case) for each possible choice. this will require that you edit the program logic just to add, remove, or change any of the possible choices.  it also makes for cluttered code, which i just noticed is the cause of your problem. you have a missing {, which has made all your conditional logic off a bit. properly indenting your code, based on where matching { and } are at would help you make sure all the { and } are where you intend. 
     
    you should instead write general purpose, data driven code, that defines data (arrays) that tells simple code what to do. see the following example -    
    // define categories of roles (in a configuration file) $management = array('Admin', 'Manager', 'Front_Desk'); $staff = array('Writer'); // at the point of decided what to do for the category a role belongs to if(in_array($role,$management)){     header("location: reservation.php");     exit; } elseif (in_array($role,$staff)){     header("location: articles.php");     exit; } else {     // none of the defined roles, handle that condition here... } next, logging someone in, involves identifying who they are, not what permissions/roles they have, which is a different concern/different process. all your login code should do, when the username/password has been confirmed, is to store the user_id in a session variable. after your post method form processing code successfully runs (with no errors), it should do a header() redirect to the exact same url that the form submitted to. this will cause a get request for the page, which stops the browser from trying to resubmit the form data.
     
    when each page gets requested, it should take the user_id from the session variable and query for the current user permissions/role. this will insure that any change made to a user's permissions/roles take affect on the very next page request. any page should take the user permissions/role and use them to determine what will be processed on the page and what will be displayed.
  25. mac_gyver's post in Special text sort of array? was marked as the answer   
    the examples in the documentation, SELECTing the value returned by the FIELD() statement are only to demonstrate what value is returned for each example. imagine calculating that value for each row of data in the result set and ordering the data by that value - 
    ORDER BY FIELD(cstype,'main character in', 'secondary cast in', 'appears in', 'cameo appearance in')
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.