hide delete button of gridview for some records

Dipa Ahuja

16y20.6k.NET

hi.. i want to hide/show delete button of gridview for some records based on particular value of field...

its like scrapbook in orkut however the user who has sent msg can delete only his/her scrap so there he/she find delete button but for other scraps not..

i am using gridview with sqldatasource control

Thank you!

8 Replies

Know the answer? Post it — somebody with the same question will find it here.

Amit Choudhary16y ago
Accepted answer
hi diya,

you have to change your function little bit.

Like this:
public bool isOwner(int scrapId)
    {
     // assign a default value to flag
bool flag=false;
        for (int i = 0; i < GridView1.Rows.Count; i++)
        {
            Label lbl = (Label)GridView1.Rows[i].FindControl("lblfromId");

            if (lbl.Text == pid.ToString())
            {
               flag=true;
            }
            else
            {
                flag=false;
            }

        }
      return flag;
}

Explaination: The error you are doing is, you are returning the valued inside the for loop that itself having a condition so if loop does't get start cause condition not me then nothing will return to the function. so giving error.

Please Don't forget to mark as answer.
0
Sandeep Sharma12y ago
bool flag = false;
for (int i = 0; i < GvDetails.Rows.Count; i++)
{

string LinkButtonText = ((LinkButton)GvDetails.Rows[i].FindControl("lnkBtnActivate")).Text;
LinkButton btn = (LinkButton)GvDetails.Rows[i].FindControl("lnkRemove");
//Button btn = (Button)GvDetails.Rows[i].FindControl("btndelete");
if (LinkButtonText == "Activated")
// if (LinkButtonText.Equals("Activated"))
{
btn.Visible = true;
flag = true;
}
else
{
btn.Visible = false;
flag = false;

}
}

this is working for me try something like this.

0
Dipa Ahuja16y ago
hi amit its done.. but with another way... i do like this..

void showdlt()
    {
        bool flag = false;
        for (int i = 0; i < GridView1.Rows.Count; i++)
        {
            Label lbl = (Label)GridView1.Rows[i].FindControl("lblfromId");
            Button btn = (Button)GridView1.Rows[i].FindControl("btndelete");

            if (lbl.Text == p.profile_Id.ToString())
            {
                btn.Visible = true;
                flag = true;
            }
            else
            {
                btn.Visible = false;
                flag = false;

            }
        }
    }

and call this method in page_load() event...

bt thanx for showing your code too...
0
Dipa Ahuja16y ago
hi amit.. i marked your previours ans bt still its not working ... i dont know its giving wierd result..

in some record its showing delete button where i wont bt in some its not..

i thing i have note is its not giving any effect on 1st record of gridview..
0
Sam Hobbs16y ago
In scrapboook.isOwner you don't need a for loop, but the compiler does not know that. The compiler does not know what to do when the for loop ends; it does not know what to return from the function. You can put a return statement at the end of scrapboook.isOwner (after the for loop), but I think the best solution is to not use the for loop. If the for loop is needed, then there is aproblem with the code that needs to be fixed in some other way.
0
Dipa Ahuja16y ago
hi amit.. i write this way as u have said like:

aspx:



                                          Visible='<%# isOwner(Convert.ToInt32(Eval("sid")))%>'
                     runat="server" Text="mydelete" />



.cs
pid = Convert.ToInt32(Request.QueryString["profileId"].ToString());

public bool isOwner(int scrapId)
    {
        for (int i = 0; i < GridView1.Rows.Count; i++)
        {
            Label lbl = (Label)GridView1.Rows[i].FindControl("lblfromId");

            if (lbl.Text == pid.ToString())
            {
                return true;
            }
            else
            {
                return false;
            }

        }
}

but its giving error:

'scrapboook.isOwner(int)': not all code paths return a value
0
Lalit M16y ago
you can also achieve this using checkbox

show here sample code

checkbox
delete button

CS Code sample

bool OneChecked = false;
    protected void CheckBox1_CheckedChanged(object sender, EventArgs e)
    {
        Button Button1 = (Button)GridView1.HeaderRow.FindControl("Button1");
        foreach (GridViewRow row in GridView1.Rows)
        {
            CheckBox CheckBox1 = (CheckBox)row.FindControl("CheckBox1");
            if (CheckBox1.Checked == true)
            {
                OneChecked = true;
            }
        }
        if (OneChecked == false)
        {
            Button1.Enabled = false;
        }
        else if (OneChecked == true)
        {
            Button1.Enabled = true;
        }
    }
0
Amit Choudhary16y ago
hi diya..

Create you button inside the grid view like this:

now create a method in code behind
```
public bool isOwner(int Scrapid)
```
```
{
```
```
 // Check if the owner of Scrap using Scrapid is equal to the Logged in User
   
   if( )
        return true;
   else
      return false;
```
```
}
```
I hope your problem get solved.
Please Don't forget to mark as answer.
0