Did that but still having the error on the when trying to load it. It won open and gives the error

" 

Argument 1 cannot be null

Class.TaskleaderNO: line 25, column 45 External entry point  "

That is the "  candDate1=System.today()-Integer.valueOf(rangedate);  "  line. "Rangedate" can;t be null. I don't get it as I ahve set is a "30"  on the public Taskleader() { rangedate = '30'; }

public class TaskleaderNO
{ 
    public PageReference Refreshtsk() {
        return page.task_leadershipNO;
    }
    
     public PageReference toexcel() {
        return page.Task_LeadershipXLSNO;
    }
     
  public String positions {get; set;}
  public String rangedate {get; set;}

  Date candDate;
  Date candDate1;

  public TaskLeaderNO()
  {
     rangedate='30';
     candDate=System.today();    
  }

Set<Id> tskNo = new Set<Id>();
{
   candDate1=System.today()-Integer.valueOf(rangedate);
   for (Task t: [Select WhoID FROM task WHERE OwnerID = :UserInfo.getUserId() and activitydate >= :candDate1])
   tskNo.add(t.WhoID);
}
  
  private List<Contact> ContactNO;
  public List<Contact> getContactNO()
   {
   
Contact[] ContactNO = [Select id, AccountId, Account.name, Email, Name, account.billingcity, account.billingstate from Contact WHERE ID IN ( SELECT Contact__c FROM Leadership_position_code__c WHERE Position__c = :positions and  (Thru_Date__c = null OR Thru_Date__c > :system.today()) and (Start_Date__C <= :system.today()) ) and ID NOT IN :tskNO];
   return ContactNO;
   }

}

Is this your entire file or did you remove some parts? The reason I ask is that there is no function called Refreshtsk() in the code you posted, and this chunk shouldn't even compile:

Set<Id> tskNo = new Set<Id>();
{
   candDate1=System.today()-Integer.valueOf(rangedate);
   for (Task t: [Select WhoID FROM task WHERE OwnerID = :UserInfo.getUserId() and activitydate >= :candDate1])
   tskNo.add(t.WhoID);
}

Have you tried putting a system.debug before line 25 to see what value is actually in rangedate?

--Sarah

Sarah, that is the entire file for the controller. I'll post the VF page again. The Refreshtsk() is on the very top and it is a pagereference . A button on the VF page calls it {!Refreshtsk} . Like you said, the whole page gets reloaded.

this is the entire VF page;
 

<apex:page controller="TaskleaderNO" showHeader="false" >
<apex:form >
<apex:pageBlock id="pageBlock" Title="NO Activities with Key Leadership">
<apex:pageMessages ></apex:pageMessages>
<b>Select Position:   </b>
<apex:selectList id="Positions" value="{!positions}" size="1">
<apex:selectoption itemLabel="President" itemValue="President"></apex:selectoption>
<apex:selectoption itemLabel="Co-President" itemValue="Co-President"></apex:selectoption>
</apex:selectList>
<b> How many days back? </b>
<apex:selectList id="rangedate" value="{!rangedate}" size="1" >
<apex:selectoption itemLabel="30" itemValue="30"></apex:selectoption>
<apex:selectoption itemLabel="60" itemValue="60"></apex:selectoption>
</apex:selectList>
<b>   </b>
<apex:commandButton action="{!Refreshtsk}" value="Run Report" rerender="tblContactNo" />
<b>   </b>
<apex:commandButton action="{!toexcel}" value="Export to Excel" />
<apex:pageBlockTable value="{!ContactNO}" var="cont" rendered="{!NOT(ISNULL(ContactNO))}" id="tblContactNo">
<apex:column headerValue="Name" ><apex:outputLink value="/{!cont.id}" target="_parent">{!cont.Name}
</apex:outputLink></apex:column>
<apex:column value="{!cont.Email}"></apex:column>
<apex:column headerValue="Congregation" value="{!cont.account.name}"></apex:column>
</apex:pageBlockTable>
</apex:pageBlock>
</apex:form>

</apex:page>

Thanks a lot for the help!

Ok I see it, sorry. I still don't understand how the following code fits in, it's not in any function:

{
   candDate1=System.today()-Integer.valueOf(rangedate);
   for (Task t: [Select WhoID FROM task WHERE OwnerID = :UserInfo.getUserId() and activitydate >= :candDate1])
   tskNo.add(t.WhoID);
}

I think that may be the problem, that code is just running at the wrong time. When do you want it to run?

That can run only when the user hits the <apex:commandButton action="{!Refreshtsk}"...

When the VF page is initally loaded, it is not necessary for the <apex:pageBlockTable value="{!ContactNO}"... to load. But when the user makes the selection on the "Positions" and on the "rangedate" SelectLists and hits the button

<apex:commandButton action="{!Refreshtsk}" value="Run Report" rerender="tblContactNo" />

 Then, the pageblocktable needs to load.