Displaying images in rows of 5

CFPARAM is a way to define and set a value to a variable IF the variable does not already exist.  If it does exist, nothing happens, the value does not change.  It's akin to:

<cfif NOT StructKeyExists(variables,'thisVar')>

<cfset thisVar = "foo" />

</cfif>

HTH,

^ _ ^

Actually, Dave's offering was pretty close to what you are looking for, except the opening TR needs to be before the CFOUTPUT and the closing TR should be after the /CFOUTPUT.

<table border="0" width="582" cellspacing="0" cellpadding="0" align="center">
	<tr>
	<cfoutput query="List">
		<td align="center" valign="top" width="542" style="padding-left:2em;text-indent:-2em">
			<a href="#request.BaseURL#/book.cfm?-#URL_Title#-&BookID=#BookID#"><img src="#request.BaseURL#/images/#Image#" width="75"></a>
		</td><cfif List.currentrow % 5 eq 0 AND List.currentrow neq List.recordCount>
	</tr>
	<tr>
	</cfif>
	</cfoutput>
	</tr>
</table>

This will start a TR, insert 5 TDs, then close and open another TR.  (% 5 is modulous of 5; when 0, the current row is evenly divisible by 5 and will start a new row as long as it's not the last record in the set.)

But Dave is right about reducing your CFOUTPUT.. since you're now ordering by serial_no first, the grouping is unnecessary, esp since you're not actually displaying anything about the serial_no.

HTH,

^ _ ^

Hi, 123proust,

I would like to offer some constructive criticism, if I may.  In your select, you are using * to get everything.  I can be a bit pedantic, and would like to suggest that you definitively name each column in your Inventory table, as using * forces the database to reference an internal table to get the column names.  Additionally, if you do not require every column in that table, then you are pushing unneeded data across the network and adding unnecessary processing to the CPU.

If you are grouping by series number, then that should be reflected in your query using the GROUP BY clause.

As far as listing only the first 100, that will depend upon what flavor of database server you are using, as MS-SQL is slightly different from Oracle, which (I believe) is slightly different than MySQL, and so on.  Some people will just grab every record and use CF code for pagination.  This is not recommended.  At least not by me.  One should set the limits within the query and pass a page number or some such to the database in the query, and get only the desired records.

Now that you are grouping by series number, do you plan to display the series number in the output?  Or is there a related name that you would display?  Or is it just for grouping and the value not important?

V/r,

^ _ ^

UPDATE:  Thank you for marking my answer as correct.  I do appreciate it.

Thank you, that worked great! You pointed out that I'm grouping by author. I meant to group, or order, the 500 book cover images by their Series Number (the "Series_no" Table in Inventory). This is as far as I got. Can you please help me do this, and teach me how to limit the list to the first 100 in the series? Here's as far as I got with your code and code that belonged to another page. 

<CFPARAM NAME="attributes.action" DEFAULT="">
<CFPARAM NAME="attributes.dsn" DEFAULT="">
<cfparam name="browseBy" default="Author">
<cfparam name="attributes.AuthorFirstName" default="">

<CFQUERY DATASOURCE="#request.MainDSN#" NAME="List">
SELECT Inventory.*,
Authors.AuthorID,
Authors.AuthorFirstName,
Authors.AuthorLastName,
Authors.Pseudonym
FROM Inventory
INNER JOIN
Authors
ON Authors.AuthorID=Inventory.AuthorID
ORDER BY AuthorLastName, AuthorFirstName, Title
</CFQUERY>

<TABLE BORDER="0" WIDTH="582" CELLSPACING="0" CELLPADDING="0" ALIGN="center">
<CFOUTPUT QUERY="List" group="Series_no">
<TR>
<CFOUTPUT>
<TD ALIGN="center" VALIGN="top" WIDTH="542" style="padding-left:2em;text-indent:-2em">
<A HREF="#request.BaseURL#/book.cfm?-#URL_Title#-&BookID=#BookID#"><img src="#request.BaseURL#/images/#Image#" width="75"></A>
</TD><cfif List.currentrow % 5 eq 0 AND List.currentrow neq List.recordCount>
</tr>
<tr>
</cfif>
</CFOUTPUT>
</TR>
</CFOUTPUT>

</TABLE>

<CFOUTPUT QUERY="List" GROUP="AuthorID">
<!--- You should have something here for author, since you're grouping by author --->
    <TR>
<CFOUTPUT>
        <TD ALIGN="left" VALIGN="top" WIDTH="542" style="padding-left:2em;text-indent:-2em">
            <A HREF="#request.BaseURL#/book.cfm?-#URL_Title#-&BookID=#BookID#"><img src="#request.BaseURL#/images/#Image#" width="50"></A>
        </TD><cfif List.currentrow % 5 eq 0 AND List.currentrow neq List.recordCount>
    </tr>
    <tr>
</cfif>
</CFOUTPUT>
</TR>
</CFOUTPUT>

HTH,

^ _ ^