Log of /contributions/modules/commentrss/commentrss.module
Parent Directory
| 
Revision Log
| 
Revision Graph
Revision
1.12.2.2.2.6.2.1 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Wed Oct 1 14:09:17 2008 UTC
(13 months, 4 weeks ago)
by
goba
Branch:
DRUPAL-6--2
CVS Tags:
DRUPAL-6--2-0
Changes since
1.12.2.2.2.6: +76 -225 lines
Diff to
previous 1.12.2.2.2.6
Forward porting changes to commentrss from 5.x-1.x to 5.x-2.x, including:
- #214958 by Junyor and myself: add context to comment feeds, so people can identify where comments are coming from
- #161150 report by twohills, patch by myself: Add different options for site
front page feed exposure, and prevent all site feed from appearing on
other pages. Setting backwards compatible.
- Update help text to describe feed features better.
- Moved settings page under 'RSS publishing' setting in admin > content.
- #198151 report by ragaskar, patch by myself: avoid node comment RSS feed
from appearing on other pages.
- Removed lots of dead code from node comment feed generation code.
- #189829 reported by zostay, mostly fixed already, but one missing piece:
Built in check for comment support on node, so that a node feed will not
be accessible at all if the node does not support comments.
- Remove vocabulary feed as vocabulary_list module is not supported for
years. Will look into Views integration later.
- Added lots of phpdoc to API functions.
- Align namespace handling in channel feed to core patch suggested at #157709
- Rethink commentrss_format_items() to join on {users} instead of doing a
user_load() on all comments handled. This leads to much better performance.
- Better descriptions on settings.
Also used Drupal 6 features to split the module into pages.inc, admin.inc and the main module file only containing core hook callbacks, minimizing memory footprint.
Revision
1.12.2.2.2.3 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Fri Sep 5 09:29:11 2008 UTC
(14 months, 3 weeks ago)
by
goba
Branch:
DRUPAL-6--1
CVS Tags:
DRUPAL-6--1-1
Changes since
1.12.2.2.2.2: +24 -22 lines
Diff to
previous 1.12.2.2.2.2
, to
branch point 1.12.2.2
- Fix admin settings so that it actually displays.
- Display vocabulary option but disabled, so people are aware of the feature.
- Pass on type (was missing) and id (was not nicely implemented) from menu.
- Keep the defaults for site and node feeds in the RSS handler, so it works with zero config.
- Found a bug in vocabulary feed with bogus variable name, fixed.
Revision
1.12.2.2.4.1 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Tue Apr 29 13:30:55 2008 UTC
(19 months ago)
by
goba
Branch:
DRUPAL-5--2
Changes since
1.12.2.2: +263 -165 lines
Diff to
previous 1.12.2.2
Generic UI cleanup and bugfixing.
- Clean up install and uninstall process. Use consistent variable defaults
instead of defaults set on installation.
- #161150 report by twohills, patch by myself: Add different options for site
front page feed exposure, and prevent all site feed from appearing on
other pages. Setting backwards compatible.
- Update help text to describe feed features better.
- Moved settings page under 'RSS publishing' setting in admin > content.
- #198151 report by ragaskar, patch by myself: avoid node comment RSS feed
from appearing on other pages.
- Simplified callbacks for feeds so that we don't need internal wrapper.
- #187595 report by Boobaa: tie to 'access comments' permission instead of
'access content' for access to the comments feeds.
- Removed lots of dead code from node comment feed generation code.
- #189829 reported by zostay, mostly fixed already, but one missing piece:
Built in check for comment support on node, so that a node feed will not
be accessible at all if the node does not support comments.
- Remove vocabulary feed as vocabulary_list module is not supported for
years. Will look into Views integration later.
- Added lots of phpdoc to API functions.
- Align namespace handling in channel feed to core patch suggested at #157709
- Rethink commentrss_format_items() to join on {users} instead of doing a
user_load() on all comments handled. This leads to much better performance.
- Better descriptions on settings.
Revision
1.6.2.5 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Sat Aug 19 18:48:55 2006 UTC
(3 years, 3 months ago)
by
goba
Branch:
DRUPAL-4-7
Changes since
1.6.2.4: +18 -44 lines
Diff to
previous 1.6.2.4
, to
branch point 1.6
stop my silly practice of doing two SQL selects for a comment rss page
do one bigger one instead, which also has the benefit that the joins could be optimized by the DB engine,
while the previous version had the node join obscured in the query and then the nid IN (...) list
Tested on a 4.7 site and seems to be working fine (although some caching might help, the database
caches the results quite well as it seems for requests close enough.)
Revision
1.11 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Sat Aug 19 18:48:00 2006 UTC
(3 years, 3 months ago)
by
goba
Branch:
MAIN
Changes since
1.10: +18 -44 lines
Diff to
previous 1.10
stop my silly practice of doing two SQL selects for a comment rss page
do one bigger one instead, which also has the benefit that the joins could be optimized by the DB engine,
while the previous version had the node join obscured in the query and then the nid IN (...) list
Tested on a 4.7 site and seems to be working fine (although some caching might help, the database
caches the results quite well as it seems for requests close enough.)
Revision
1.1 -
(
view)
(
download)
(
as text)
(
annotate)
-
[select for diffs]
Sun Jul 4 16:24:50 2004 UTC
(5 years, 4 months ago)
by
goba
Branch:
MAIN
Branch point for:
DRUPAL-4-4
Provides comment feeds for:
- whole site
- per vocabulary
- per term
- per node
Also needs vocabulary_list.module to not print broken links.
There is nowhere to link to if that module is not set up.
This form allows you to request diffs between any two revisions of this file.
For each of the two "sides" of the diff,
select a symbolic revision name using the selection box, or choose
'Use Text Field' and enter a numeric revision.
