Retrieving checked boxes from array comparison

[crawdidly]

Trying to echo out an array but it seems to be looping. The intent is to match the contents of the second query to the first and then output results in the form of checked boxes. Three boxes(publichealth, primarycare, collaborative) should all show and be checked, but final array set is looping it 3 times. I attached an image of the result.

[ATTACH=CONFIG]4965[/ATTACH]

$query = "SELECT * FROM esu_videotypes";
$result = mysql_query($query);
while ($row = mysql_fetch_array($result)) {
    $id = $row['id'];
    $cards[$id] = $row['name'];
}

$query = "SELECT esu_checks.id, esu_checks.vidID, esu_checks.groupID, esu_videotypes.id, esu_videotypes.name FROM esu_videotypes, esu_checks WHERE esu_checks.id = esu_videotypes.id AND esu_checks.vidID = ".$_GET['editarchive']." AND esu_checks.groupID = '1'";
//query echoes 3 results from my test, each matching a id in table videotypes

$result = mysql_query($query) or die(mysql_error());

while ($row = mysql_fetch_array($result, MYSQL_ASSOC)) {
    $rname = $row['name'];
    $cid = $row['id'];
    $accepted[$rname][] = $cid;
}
foreach ($accepted as $key => $value) {

foreach ($cards as $cid => $card) {
    echo "<input type=\"checkbox\" name=\"vidtype[]\" value=\"$cid\"";
    if (in_array($cid, $value)){
        echo " checked";
    }
    echo "> $card ";
}
}

code-ex.jpg

[dalecosp]

What if you

print_r($accepted);

before the first foreach()?

I think you have some unintended consequences from the way you're building the array....

[crawdidly]

dalecosop, I performed an array readout and it is as follows:

Array ( [Public Health] => Array ( [0] => 1 ) [Primary Care] => Array ( [0] => 2 ) [Collaborative] => Array ( [0] => 3 ) ) Public Health Primary Care Collaborative Array ( [Public Health] => Array ( [0] => 1 ) [Primary Care] => Array ( [0] => 2 ) [Collaborative] => Array ( [0] => 3 ) ) Public Health Primary Care Collaborative Array ( [Public Health] => Array ( [0] => 1 ) [Primary Care] => Array ( [0] => 2 ) [Collaborative] => Array ( [0] => 3 ) ) Public Health Primary Care Collaborative

[dalecosp]

Exactly. Nine items.

Array ( 
   [Public Health] => Array ( [0] => 1 ) 
   [Primary Care] => Array ( [0] => 2 ) 
   [Collaborative] => Array ( [0] => 3 )
   ) 
Public Health Primary Care Collaborative 

Array (
   [Public Health] => Array ( [0] => 1 ) 
   [Primary Care] => Array ( [0] => 2 ) 
   [Collaborative] => Array ( [0] => 3 ) 
) 
Public Health Primary Care Collaborative 

Array ( 
   [Public Health] => Array ( [0] => 1 ) 
   [Primary Care] => Array ( [0] => 2 ) 
   [Collaborative] => Array ( [0] => 3 ) 
)
Public Health Primary Care Collaborative

Have you tried it without the extra "[]" ?

//$accepted[$rname][] = $cid;  /* remove the empty brackets */
$accepted[$rname] = $cid;

[crawdidly]

It echoes the same result without any of the 9 boxes selected (which should be 3)..

[dalecosp]

Well, at this point, it starts to look like a design issue. I suppose we'd need to see the table structure and the desired result. I would imagine we need to create one array and loop through it instead of looping through two arrays each using elements of the other.

[crawdidly]

I printed out my two tables. My desired is to display all of the checkbox entries(3) from videotypes (first query). Then compare those to the result of the second query, and then ultimately use an array to check the appropriate results based on the matches defined in the checklist table.

checks table

---

vidID id groupID
42 7 3
42 8 3
46 59 2
46 1 1
46 2 1
46 3 1

videotypes

id name
1 Public Health
2 Primary Care
3 Collaborative

[crawdidly]

So basically my ultimate and complete goal from this example is to display all 3 videtypes and have them all checked (execution performed by this portion in second query: esu_videotypes.id = esu_checks.id)

[crawdidly]

I've been trying to think of a simpler approach, and have been playing around with this but it won't outputed the 'checked' boxes.

[

$query="SELECT id, name FROM esu_videotypes"; 
$result=mysql_query($query);

$query = "SELECT esu_checks.id, esu_checks.vidID, esu_checks.groupID, esu_videotypes.id, esu_videotypes.name FROM esu_videotypes, esu_checks 
WHERE esu_checks.id = esu_videotypes.id AND esu_checks.vidID = ".$_GET['editarchive']." AND esu_checks.groupID = '1'";
$q1=mysql_query($query); 


$id_for_this_restid = array(); 

while($id_for_this_restid[] = mysql_fetch_array($q1))
{ 
}

while($array= mysql_fetch_array($result)) 
{ 
    if (in_array($array[id], $id_for_this_restid)){
    echo "<input type='checkbox' name='vidtype[]' value='$array[id]' checked>$array[name]<br>\n"; 
    } else {
    echo "<input type='checkbox' name='vidtype[]' value='$array[id]'>$array[name]<br>\n"; 
    }
} 

/PHP]

[dalecosp]

crawdidly;11033759 wrote:

results based on the matches defined in the checklist table.

checks table
videotypes

Sorry to be pedantic, but this is programming. "Checklist" table? You show a "checks" table and a "videotypes" table. Where's the "checklist" table?

crawdidly;11033761 wrote:

So basically my ultimate and complete goal from this example is to display all 3 videtypes and have them all checked (execution performed by this portion in second query: esu_videotypes.id = esu_checks.id)

Now, if I read what you have written here, I can do this:

<input type="checkbox" name="Public Health" checked>  <!-- checked="checked" in XHTML-->
<input type="checkbox" name="Primary Care" checked>
<input type="checkbox" name="Collaborative" checked> 

Obviously I'm misunderstanding something.

I'll play with your code and data, because my employer doesn't pay me enough 😉 😉 . This isn't a homework assignment, is it? 😃

[dalecosp]

In your data example, it looks as if you have the column names in the wrong order. Aren't 1,2 and 3 the vidtypes?

[crawdidly]

dalecosp;11033771 wrote:

In your data example, it looks as if you have the column names in the wrong order. Aren't 1,2 and 3 the vidtypes?

Dalescop, yes those are the vidtypes, which data example are you referring too? The table names are esu_checks and esu_videotypes (there is no checklist table was a typo). Any help would be appreciated as I've been steaming over this for awhile. :bemused:

As far as homework assignment, so let it be written ....so let it be done. Where are the virtual beer emoticons?

[crawdidly]

Sorry, I see what you're suggesting, but no the second column 'id' is just what I decided to call the column. VidID is the first column and is not set as incremental I have an array in another script outputting all inserted values to that table.

[crawdidly]

Now, if I read what you have written here, I can do this:

<input type="checkbox" name="Public Health" checked>  <!-- checked="checked" in XHTML-->
<input type="checkbox" name="Primary Care" checked>
<input type="checkbox" name="Collaborative" checked> 

Yes, this is the outcome that I'd like.

<input type="checkbox" name="Public Health" checked>  <!-- checked="checked" in XHTML-->
<input type="checkbox" name="Primary Care" checked>
<input type="checkbox" name="Collaborative" checked> 

The following echoes out the 3 videotypes but it's not checking the boxes:

$query = "SELECT * FROM esu_videotypes";
$result = mysql_query($query);
while ($row = mysql_fetch_array($result)) {
    $id = $row['id'];
    $cards[$id] = $row['name'];
}

$query = "SELECT esu_checks.id, esu_checks.vidID, esu_checks.groupID, esu_videotypes.id, esu_videotypes.name FROM esu_videotypes, esu_checks WHERE esu_checks.id = esu_videotypes.id AND esu_checks.vidID = ".$_GET['editarchive']." AND esu_checks.groupID = '1'";
//query echoes 3 results from my test, each matching a id in table videotypes

$result = mysql_query($query) or die(mysql_error());

while ($row = mysql_fetch_array($result, MYSQL_ASSOC)) {
    $rname = $row['name'];
    $cid = $row['id'];
    $accepted[$rname][] = $cid;
}
foreach ($accepted as $key => $value) {
 // foreach ($cards as $cid => $card) {

    $cid = $value->id;
	$cards = $value->card;
    echo "<input type=\"checkbox\" name=\"vidtype[]\" value=\"$cid\"";
    if (in_array($cid, $value)){
        echo " checked";
    }
    echo "> ";

}

[dalecosp]

$value->id?

You don't have any objects in this code. Did you mean $value["$id"]?

That isn't set either, as far as I can see.

More to follow. Stupid script here is still executing 😃

[dalecosp]

Something like this?

$result = mysql_query($query) or die(mysql_error());

$x=0;

if (mysql_num_rows($result)) {
   while ($row = mysql_fetch_array($result, MYSQL_ASSOC)) {
       $accepted[$x]['rname'] = $row['name'];
       $accepted[$x]['cid']=$row['id'];
       $accepted[$x][$row['name']] = $cid;
       $x++;
   }
} else {
   die("No rows returned.  Query was $query");
}

foreach ($accepted as $key => $value) {

    $cid = $value['cid'];;
    $cards = $value['rname'];
    echo "<input type=\"checkbox\" name=\"vidtype[]\" value=\"$cid\"";
    if (in_array($cid, $value)){
        echo " checked";
    }
    echo "> ";

}

[dalecosp]

Incidentally: Mysql_foo is deprecated (see my .sig) ... we should probably convert this to mysqli_foo ;-)

[crawdidly]

dalecosp;11033783 wrote:

$value->id?

You don't have any objects in this code. Did you mean $value["$id"]?

That isn't set either, as far as I can see.

More to follow. Stupid script here is still executing 😃

Yeah I did, tried that but it didn't work either. I'm not totally following the logic of the arrays thus I need another thinking mind :eek:

[crawdidly]

Sorry just saw your example, let me try a few things with this and will get back to you.

[crawdidly]

It echoes the following with 1 line but no category name and the other two selection boxes need to be in place:

<input type="checkbox" name="vidtype[]" value="3" checked>

and should be:

<input type="checkbox" name="vidtype[]" value="1"> Name of videotype
<input type="checkbox" name="vidtype[]" value="2"> Name of videotype
<input type="checkbox" name="vidtype[]" value="3" checked> Name of videotype

So it's checking the right box, but I need the other two checkboxes to appear with it unchecked.

$query = "SELECT * FROM esu_videotypes";
$result = mysql_query($query);
while ($row = mysql_fetch_array($result)) {
    $id = $row['id'];
    $cards[$id] = $row['name'];
}

$query = "SELECT esu_checks.id, esu_checks.vidID, esu_checks.groupID, esu_videotypes.id, esu_videotypes.name FROM esu_videotypes, esu_checks WHERE esu_checks.id = esu_videotypes.id AND esu_checks.vidID = ".$_GET['editarchive']." AND esu_checks.groupID = '1'";
//query echoes 3 results from my test, each matching a id in table videotypes

$result = mysql_query($query) or die(mysql_error());

$x=0;

if (mysql_num_rows($result)) {
   while ($row = mysql_fetch_array($result, MYSQL_ASSOC)) {
       $accepted[$x]['rname'] = $row['name'];
       $accepted[$x]['cid']=$row['id'];
       $accepted[$x][$row['name']] = $cid;
       $x++;
   }
} else {
   die("No rows returned.  Query was $query");
}

foreach ($accepted as $key => $value) {

    $cid = $value['cid'];;
    $cards = $value['rname'];
    echo "<input type=\"checkbox\" name=\"vidtype[]\" value=\"$cid\"";
    if (in_array($cid, $value)){
        echo " checked";
    }
    echo "> ";

}