Why is the “view” command link working only the first time I click it? How do I make it work for all links?

Question

I'm using a Developer Force example to recreate that Visualforce page. It works except for the fact that clicking on the "view" link only works once. How do I make it work for each and every link / case?

Visualforce:

<apex:page controller="MyConsoleController">
    <apex:form >

    <apex:sectionHeader title="Cases Cockpit"/>

    <div id="caseInfo" style="float:left;width:25%">
        <apex:pageBlock title="My Cases">
            <apex:pageBlockTable value="{!myCases}" var="case">
                <apex:column >
                    <apex:commandLink value="View" action="{!viewCase}">
                        <apex:param name="caseId" value="{!case.Id}"/>
                    </apex:commandLink>
                </apex:column>
                <apex:column headerValue="Subject" value="{!case.Subject}"/>
                <apex:column headerValue="Status" value="{!case.Status}"/>
            </apex:pageBlockTable>
        </apex:pageBlock>
    </div>

    <div style="float:right;width:73%">
        <apex:detail id="caseDetail" subject="{!selectedCase.Id}" relatedList="false" title="false"/>
    </div>

    <div style="width:100%">
        <apex:pageBlock title="Solutions">
            <apex:pageBlockTable value="{!solutions}" var="sol">
                <apex:column >
                    <apex:commandLink value="View" action="{!viewSolution}">
                        <apex:param name="caseId" value="{!sol.Id}"/>
                    </apex:commandLink>
                </apex:column>
                <apex:column headerValue="Solution #" value="{!sol.SolutionNumber}"/>
                <apex:column headerValue="Name" value="{!sol.SolutionName}"/>
                <apex:column headerValue="Status" value="{!sol.Status}"/>
            </apex:pageBlockTable>
        </apex:pageBlock>
    </div>

    </apex:form>
</apex:page>

Controller:

public class MyConsoleController {

    public Case selectedCase { get; set; }

    public Solution[] getSolutions() {
        return [SELECT Id, SolutionNumber, SolutionName, Status FROM Solution];
    }

    public Case[] getMyCases() {
        return [SELECT Id, Subject, Status FROM Case WHERE OwnerId = :UserInfo.getUserId()
            AND Status != 'Closed'];
    }

    public PageReference viewCase() {
        String caseId = ApexPages.currentPage().getParameters().get('caseId');
        selectedCase = [SELECT Id FROM Case WHERE Id = :caseId];
        return null;
    }

    public PageReference viewSolution() {
        String solId = ApexPages.currentPage().getParameters().get('solId');
        return new PageReference('/' + solId);
    }
}

Answer 1

The issue that I see is on lines 27 - 31 (copied below) and causes it so that you cannot view solutions. I was able to reproduce that error; however there's no error for the case links. I can click those without any problems just like @grigriforce.

  <apex:column >
    <apex:commandLink value="View" action="{!viewSolution}"> 
        <apex:param name="caseId" value="{!sol.Id}"/>
    </apex:commandLink>
  </apex:column>

Note that it is assigning solution Id to a parameter of caseId. Then in the controller viewSolution method it has:

 String solId = ApexPages.currentPage().getParameters().get('solId');

So, solId is null and it gives you the not found / no longer exists URL. Change the Visualforce to:

<apex:param name="solId" value="{!sol.Id}"/>

After you make that change the controller method will have a value of the clicked Solution's ID for solId instead of null and the PageReference that is constructed will go to the Solution's detail page.

Update There is another error if the detail page (from the page layout) has the Submit for Approval button on it because it has an html name attribute of submit which conflicts with the form's submit function.

A solution for that is to break the page up in to two forms, leaving the detail page out of the forms.

The Visualforce page would be changed to:

<apex:page controller="MyConsoleController">

    <apex:sectionHeader title="Cases Cockpit"/>

<!-- One form begins here -->
<apex:form >
    <div id="caseInfo" style="float:left;width:25%">
        <apex:pageBlock title="My Cases">
            <apex:pageBlockTable value="{!myCases}" var="case">
                <apex:column >
                    <apex:commandLink value="View" action="{!viewCase}">
                        <apex:param name="caseId" value="{!case.Id}"/>
                    </apex:commandLink>
                </apex:column>
                <apex:column headerValue="Subject" value="{!case.Subject}"/>
                <apex:column headerValue="Status" value="{!case.Status}"/>
            </apex:pageBlockTable>
        </apex:pageBlock>
    </div>
</apex:form>
<!-- One form ends here, leaving the detail page out -->
    <div style="float:right;width:73%">
        <apex:detail id="caseDetail" subject="{!selectedCase.Id}" relatedList="false" title="false"/>
    </div>
<!-- Second form begins here, leaving the detail page out -->
<apex:form >
    <div style="width:100%">
        <apex:pageBlock title="Solutions">
            <apex:pageBlockTable value="{!solutions}" var="sol">
                <apex:column >
                    <apex:commandLink value="View" action="{!viewSolution}">
                        <apex:param name="caseId" value="{!sol.Id}"/>
                    </apex:commandLink>
                </apex:column>
                <apex:column headerValue="Solution #" value="{!sol.SolutionNumber}"/>
                <apex:column headerValue="Name" value="{!sol.SolutionName}"/>
                <apex:column headerValue="Status" value="{!sol.Status}"/>
            </apex:pageBlockTable>
        </apex:pageBlock>
    </div>
</apex:form>
<!-- Second form ends here -->
</apex:page>