I don't know if this has ever been pointed out before, but while playing around a bit, I discovered an error in the download code -- specifically, in the GetArticleCount(bool publishedOnly) method in the Article class.

The method in part is:


public static int GetArticleCount(bool publishedOnly)
      {
         if (publishedOnly)
            return GetArticleCount();

...


It actually should be:


public static int GetArticleCount(bool publishedOnly)
      {
         if (!publishedOnly)
            return GetArticleCount();



If publishedOnly is FALSE, you want to use the overload which doesn't check the publishing criteria, which is GetArticleCount().

The way it is written, it will use the GetArticles() overload when publishedOnly is TRUE, and return the count of ALL articles (published or not), which of course is wrong.
Kinda weird I never noticed this before, but better late than never.

__________________
Visit my blog at http://leedumond.com
Follow me on Twitter: http://twitter.com/LeeDumond

You just solved my UI problem..!!!! :)

I came across what obviously is this error working on converting to LINQ, but I was working during the late night with an unclear head I wasn't sure if I was right or not.

I encountered a GetArticleCount() problem not by viewing the code, but by a UI error.

During testing I had 1 article "NOT" listed, and 1 "NOT" approved. I had a total of 12 articles (10 pulishedOnly). In ManageArticles.aspx in the admin area all 12 were viewable, but in BrowseArticles.aspx with "All Categories" selected from the DDL resulted in an error:

     10 Articles Displayed, but with paging enabled to page 2 (page 2 was empty).

I knew there was a count problem, but I haven't gotten around to troubleshooting it yet. Thanks for the heads up on this, you saved me some debugging time.

Thanks Lee,
Ronnie

Hey, I have been working through this too. I see you pointed out a few errors... I found a couple more as well.

I also have a few quibbles with the way the author approached some of the LINQ functionality. I am going to post my thoughts on this soon. I am really looking forward to your opinion on it, as I think you and I are the only ones here who are working on this Blox. :D

Well.. I got through the first part of the BLOX and successfully converted the Articles.cs to LINQ. I took a break here, I figure the rest is repetitive, and I got a primer/intro to LINQ in this project.

Though minor errors, I found the BLOX quite helpful. But keep me posted, I'm still very interested in the Blox and LINQ, I just picked up a LINQ publication. If you'd like to PM me, you're more then welcome to.

Now it's time to design and build my application based on what I've learned in this book and blox. :D I already have a design question . Keep an eye out for a new post, I could use some help on it.

Thanks,
Ronnie