Trying to use custom PHP code in WordPress Page

What you are getting by doing print_r ($result); ?

Thanks for the reply.

For ‘print_r ($result);’ I am getting the following

Array ( [0] => stdClass Object ( [value] => 2000 [tot_year] => 3 ) [1] => stdClass Object ( [value] => 2002 [tot_year] => 8 ) [2] => stdClass Object ( [value] => 2004 [tot_year] => 5 ) [3] => stdClass Object ( [value] => 2006 [tot_year] => 4 ) [4] => stdClass Object ( [value] => 2008 [tot_year] => 1 ) [5] => stdClass Object ( [value] => 2010 [tot_year] => 3 ) [6] => stdClass Object ( [value] => 2012 [tot_year] => 3 ) [7] => stdClass Object ( [value] => 2014 [tot_year] => 1 ) )

I have noticed that the theme sidebar and footer disappear unless i comment out

   
        while ($row = mysql_fetch_array($result))
                {
                $row_id = $idx % $num_rows;
                $row[row_id] = "<td>$row[value]</td>";
                $idx++;
                }
        // Print the whole thing now.
        echo ("<table>");
                foreach ($rows as $cur_row)
                {
                echo ("<tr>" . $cur_row . "</tr>\n");
                }

could it be something to do with this line
while ($row = mysql_fetch_array($result))

Thanks

• This reply was modified 7 years, 10 months ago by mdrabble.
• This reply was modified 7 years, 10 months ago by mdrabble.

For one thing, I don’t see where $rows (plural) is assigned any value, so the foreach does nothing.

I think you’re missing one level of looping. You’ve collected the cell contents into the $row array, but no where are those cells compiled into a $rows element. You may need a foreach loop on each row to build each $rows element.

I think you are overwriting each cell element in $row[n] instead of concatenating to it, thus you end up with a single column instead of a grid. If this was your intent, my suggestion of a foreach on each row would not apply, but you still need to assign values to $rows elements.

Do this:

foreach( $result as $row ) {
    // do your stuff here
    
}

Thanks for the reply – after you pointed me in the right direct I went back and started from scratch.

Here is the working code – would this open up my site up to SQL Injection attacks? I read somewhere that using $wpdb is the correct method as helps prevent such attacks.

        <?php
        //Code to display Alumni Years
        global $wpdb;
        $alumniyears = $wpdb->get_results("SELECT <code>value</code>, COUNT(*) as tot_year FROM <code>wp_bp_xprofile_data</code> WHERE <code>field_id</code> =45 GROUP BY value", ARRAY_A);

        $rows = $wpdb->num_rows;    // Find total rows returned by database
        if($rows > 0) {
                $cols = 3;    // Define number of columns
                $counter = 1;     // Counter used to identify if we need to start or end a row
                $nbsp = $cols - ($rows % $cols);    // Calculate the number of blank columns

                echo '<table width="100%" align="center" cellpadding="4" cellspacing="1">';
                foreach ($alumniyears as $row) 
                //while ($row = $result->fetch_array()) {
                {
                        if(($counter % $cols) == 1) {    // Check if it's new row
                                echo '<tr>';
                        }
                        echo '<td>'.$row[value].'</td>';

                        if(($counter % $cols) == 0) { // If it's last column in each row then counter remainder will be zero
                                echo '</tr>';
                        }
                        $counter++;    // Increase the counter
                }
                $result->free();
                if($nbsp > 0) { // Add unused column in last row
                        for ($i = 0; $i < $nbsp; $i++)  {
                                echo '<td>&nbsp;</td>';
                        }
                        echo '</tr>';
                }
                echo '</table>';
        }
?>

@mdrabble
Yes you are right. Anything you want to do in WP, use only WP functions and standards as much as possible.

• This reply was modified 7 years, 10 months ago by Bunty. Reason: forgot to mention name
• This reply was modified 7 years, 10 months ago by Bunty.

Ok next question.

Using a similar sql query – I want to display anyone who had a leavers year of 1987.

So I thought the following query

SELECT * wp_bp_xprofile_data, wp_users.displayname WHERE wp_bp_xprofile_data.field_id=45 AND wp_bp_xprofile_data.value=1987

would return a single user as only 1 user have a leaving year of 1987 – however, the above query returns the same filed_id and value for all users in wp_users

|————–|———-|——-|
|Display Name | Field_ID | value |
|————–|———-|——-|
| User1 | 45 | 1987 |
| User2 | 45 | 1987 |
| User3 | 45 | 1987 |
| User4 | 45 | 1987 |
|————–|———-|——-|

User 2 & 3 should have a value of 1988 and user 4 should have a value of 1999

What is the best way to pull in details from different tables and display on a single page?

Thanks ??

• This reply was modified 7 years, 10 months ago by mdrabble.
• This reply was modified 7 years, 10 months ago by mdrabble.

You can achieve this by using posts_where filter. It’s a simple filter and you can easily use in your code.

Reference link:
https://codex.www.ads-software.com/Plugin_API/Filter_Reference/posts_where