StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

PO
text
Body
copied!<p>One thing about that code that would make this hard to figure out is that you are ignoring the success of the query execution, and assuming that it's successful. The <code>$stmt4->execute()</code> returns a boolean indicating whether or not it was in fact successful. You should be checking that and, if it fails, logging/printing the error.</p> <pre><code>//convert video id array to single lines $pieces = explode(",", $video_ids); //iterate through video IDS in our DB foreach ($pieces as $key) { $sql ="SELECT id, video_name, link, phase FROM videos WHERE id=?"; if ($stmt = $mysqli->prepare($sql)) { $stmt->bind_param("i", $key); if ($stmt->execute()) { $stmt->bind_result($id, $vid_name, $vid_link, $phase); while ($stmt->fetch()) { echo "<input type='checkbox' name='checkbox[]' id='$id' value='$id' />"; echo "<a href='http://www.rotaryswing.com/golf-instruction/video /rst-index.php?cat=$phase&subcat=Rotary%20Swing%20Tour&video=$id&id=$vid_link&name=$vid_name' target=\"blank\">$vid_name</a><br>"; } } else { trigger_error("SQL query failed: " . $stmt->error, E_USER_ERROR); } } } </code></pre> <p>Now, if the query fails for some reason, this will either log or print the error, depending on your server settings.</p> <p>Also note two things. First, I'm using a parametrized query there, which prevents SQL injection. Second, I've intended the code in a sane manner, making this far easier to read than your code.</p>

Querying!

Guidance

An individual column

Larger individual text columns get their own page to allow for proper reading.

SQuiL has stopped working due to an internal error.

If you are curious you may find further information in the browser console, which is accessible through the devtools (F12).

Reload