Permalink
Browse files

Fixed #126: If a sticky is the only thread in the forum or a sub-foru…

…m it is hidden

With thanks to Sani for noticing and implementing this fix
  • Loading branch information...
1 parent fbe8015 commit f3e42288f7db1bf6105119311546922d2e6bad7b @Kroc committed May 2, 2012
Showing with 8 additions and 9 deletions.
  1. +2 −0 HISTORY.txt
  2. +1 −0 README.txt
  3. +2 −3 index.php
  4. +3 −6 thread.php
View
@@ -5,7 +5,9 @@ v19
- Markup.txt removed, moved to a themed / translatable page
- Privacy policy added (also translatable)
- Translations for 'about.html' can be provided with 'about_en.html' where 'en' is the language code
+* Fixed a long-standing bug that caused page boundaries (i.e. posts per page) to be incorrectly calculated
* Fixed incompaitbility with Windows servers involving page numbers in URLs. '+' is now used instead of ':'
+* Fixed bug with stickies not showing if no non-sticky threads exist (with thanks to "Sani")
* Moved `theme_pageList` from 'theme.config.default.php' and into a new 'theme.php' for storing the theme functions;
this allows each theme to use different types of lists without 'theme.config.php' having to be updated all the time
* Page numbers are now validated / bounded
View
@@ -191,6 +191,7 @@ See LICENCE.txt for licence details
* Richard van Velzen - subfolder support / other fixes,
additional markup implementation
* "Fyra" - UTF-8 fix for templates
+* "Sani" - Bugs
* The users of Camen Design Forum <forum.camendesign.com>
for testing and support
View
@@ -201,7 +201,7 @@
/* threads
---------------------------------------------------------------------------------------------------------------------- */
-if ($threads) {
+if ($threads || $stickies) {
//do the page links (stickies are not included in the count as they appear on all pages)
theme_pageList ($template, PATH_URL, $PAGE, $PAGES);
//slice the full list into the current page
@@ -302,8 +302,7 @@
'#nnf_error-title'=> empty ($_POST) || TITLE
));
-//call the user-defined function in 'theme.config.php' (if it exists), otherwise 'theme.config.default.php'.
-//this function is provided to allow custom themes to do their own additional templating
+//call the theme-specific templating function, in 'theme.php', before outputting
theme_custom ($template);
die ($template->html ());
View
@@ -182,8 +182,7 @@
'#nnf_error-text' => empty ($_POST) || TEXT
));
- //call the user-defined function in 'theme.config.php' (if it exists), otherwise 'theme.config.default.php'.
- //this function is provided to allow custom themes to do their own additional templating
+ //call the theme-specific templating function, in 'theme.php', before outputting
theme_custom ($template);
die ($template->html ());
}
@@ -324,8 +323,7 @@
'#nnf_error-name-delete' => empty ($_POST) || NAME
));
- //call the user-defined function in 'theme.config.php' (if it exists), otherwise 'theme.config.default.php'.
- //this function is provided to allow custom themes to do their own additional templating
+ //call the theme-specific templating function, in 'theme.php', before outputting
theme_custom ($template);
die ($template->html ());
}
@@ -580,8 +578,7 @@
'#nnf_error-text' => empty ($_POST) || TEXT
));
-//call the user-defined function in 'theme.config.php' (if it exists), otherwise 'theme.config.default.php'.
-//this function is provided to allow custom themes to do their own additional templating
+//call the theme-specific templating function, in 'theme.php', before outputting
theme_custom ($template);
die ($template->html ());

0 comments on commit f3e4228

Please sign in to comment.