|
Mike Worth made changes - 08/May/09 08:57 PM
Here's a screenshot.
I wasn't aware of that xhtml compliance thing- now I know why 'weird' <br> tags keep showing up everywhere. I suppose I'll have to use some form of mass replace at some point as all my work will be using <br>. Agreed on the dmlib thing, also agreed on the project for another day part.
Mike Worth made changes - 12/May/09 05:43 PM
Mike Worth committed 1 file to 'Contrib CVS' - 03/Jun/09 04:48 PM
Mike Worth made changes - 03/Jun/09 04:49 PM
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Remember that the <br> tag needs to be <br/> for xhtml compliance (http://www.w3schools.com/TAGS/tag_br.asp
).
+ . htmlspecialchars($row[0]) . ($row[1] > 0 ? "($row[1])" : "") . "
+ <br>$row[3]</a></th>";//print the room description as well
Looking at this reminds me of how ugly from a readability perspective this code is (i.e. $row[3]). At some point it would be nice to use the Moodle dmllib functions like get_record and such so that we could do something like $entry->description. Again, a project for another day (or another person or another life time).
Peace - Anthony