Ch 17: Hyperlink from Favorite Reviews fails

paulf · January 16th, 2014, 11:49 AM

Now on Ch 17 - had a break after last issue! I have made the changes to default.aspx in the Reviews folder and everything builds OK with no errors. I view the page in IE, after logging in, and it displays everything as it should. I hover over one of the hyperlinks and the url looks correct but when I click it I get an 404 error - page not found.

I then went back to the All Reviews page and the hyperlinks look the same and in this page they work fine. I've compared the HTML in both pages and the <asp:hyperlink> code looks identical. So, I'm confused.

HMTL from default.aspx (Reviews folder):

HTML Code:

    <asp:Repeater ID="GenreRepeater" runat="server">
        <HeaderTemplate>
            <p>Below you will find a list with reviews for your favourite music genres.</p>
        </HeaderTemplate>
        <ItemTemplate>
            <h3><asp:Literal ID="Literal1" runat="server" Text= '<%# Eval("Name")%>'></asp:Literal></h3>
            <asp:Repeater ID="ReviewRepeater" runat="server" DataSource='<%# Eval("Reviews")%>' ItemType="PlanetWroxModel.Review">
                <ItemTemplate>
                    <asp:HyperLink ID="HyperLink1" runat="server" Text='<%# Item.Title%>' NavigateUrl='<%# "ViewDetails.apsx?ReviewId=" + Item.Id.ToString()%>'>
                    </asp:HyperLink>
                </ItemTemplate>
                <SeparatorTemplate>
                    <br />
                </SeparatorTemplate>
            </asp:Repeater>
        </ItemTemplate>
    </asp:Repeater>

Hyperlink HTML from ALL.aspx:

HTML Code:

     <asp:HyperLink ID="HyperLink1" runat="server" NavigateUrl='<%# "ViewDetails.aspx?ReviewId=" + Item.Id.ToString() %>' Text='<%# Item.Title %>'></asp:HyperLink>

Any thoughts? I can't spot any difference between these two. Perhaps something is wrong elsewhere but I haven't found it.

Many thanks.

Paul

Imar · January 16th, 2014, 12:20 PM

Can you show the HTML from the browser? Also, are you sure ViewDetails.aspx exists and is placed in the correct folder?

Imar

paulf · January 16th, 2014, 01:13 PM

Quote:

Can you show the HTML from the browser?

I have just copied the relevant bit - did you want more?

HTML Code:

            <p>Below you will find a list with reviews for your favourite music genres.</p>
        
            <h3>Hard Rock</h3>
            
                    <a id="cpMainContent_GenreRepeater_ReviewRepeater_0_HyperLink1_0" href="ViewDetails.apsx?ReviewId=6">Death Magnetic by Metallica</a>
                
                    <br />
                
                    <a id="cpMainContent_GenreRepeater_ReviewRepeater_0_HyperLink1_1" href="ViewDetails.apsx?ReviewId=14">Travelling the Face of the Globe by Oi Va Voi - A genuinely fresh and enthralling sound</a>
                
        
            <h3>Pop</h3>
            
                    <a id="cpMainContent_GenreRepeater_ReviewRepeater_1_HyperLink1_0" href="ViewDetails.apsx?ReviewId=8">Wait for Evolution by De Staat</a>
                
                    <br />
                
                    <a id="cpMainContent_GenreRepeater_ReviewRepeater_1_HyperLink1_1" href="ViewDetails.apsx?ReviewId=18">Sounds Of The Universe by Depeche Mode - Overhyped or a master piece?</a>
                
                    <br />
                
                    <a id="cpMainContent_GenreRepeater_ReviewRepeater_1_HyperLink1_2" href="ViewDetails.apsx?ReviewId=25">This is the title</a>

Quote:

Also, are you sure ViewDetails.aspx exists and is placed in the correct folder?

ViewDetails is in the Reviews folder, so I think that's a "Yes". It does exist and the hyperlinks from the All.aspx page work correctly and do bring up the ViewDetails page with the correct review.

Paul

Imar · January 17th, 2014, 03:45 AM

Ah, now I see what the problem is:

Code:

<asp:HyperLink ID="HyperLink1" runat="server" Text='<%# Item.Title%>' NavigateUrl='<%# "ViewDetails.apsx?ReviewId=" + Item.Id.ToString()%>'>

Note the extension. You have apsx instead of aspx ;-)

Cheers,

Imar

paulf · January 17th, 2014, 05:28 AM

Doh!

Sorry - sometimes you just can't see the wood for the trees, especially when you know what it ought to say and the brain "auto-corrects". I'll try to come up with something more challenging in future!!

Thank you.

Paul