many fixes for various problems

leop - February 12, 2008 - 15:24
Project:Recent Changes
Version:5.x-1.1
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed
Description

Here is a patch that fixes the following problems and introduces new functionality:

  • Extensive options on administrative page.
    http://drupal.org/node/138442
  • A list of allowed content types.
    http://drupal.org/node/167312
  • Show only publised nodes and comments for published nodes.
    http://drupal.org/node/145282
  • Extensive access checks for recent changes page and feeds on all levels: content types, nodes and revisions. Inaccessible content will not be listed on the recent changes page, while headers of inaccessible content might be listed in recent changes feeds, depending on the settings.
    http://drupal.org/node/136465
    http://drupal.org/node/156103
    http://drupal.org/node/217262
    http://drupal.org/node/133676
  • RSS feeds per content type.
    5.x-1.x-dev
  • Filter allows for multiple content type selection, depending on settings in administrative page.
  • Filter contains 'comments' option, instead of providing it as an additional checkbox. To select comments for certain content types, multiple selection can be enabled in the administrative page.
  • Human-friendly content type names instead of machine-readable content types.
  • Links to older revisions (if necessary) instead of current revisions.
  • Content type page filter shows only allowed and accessible content types.
  • Added options 'updated' for changed content and 'revised' for new revisions. The options 'moved' and 'new' are also still available.
  • The users in the 'user' column are the users that performed the 'post' or 'update' actions. Owners (authors) are shown if the changes were made by another user than the owner of that node.
  • Speed up of SQL queries (no queries inside for or foreach loops).
  • Many other fixes and features
AttachmentSize
recent_changes.patch45.78 KB

#1

leop - February 12, 2008 - 19:43

This new version of the patch also formats comments, and uses the node_teaser function to make short teasers of long comments. Also fixed a problem with comments for inaccessible content.

AttachmentSize
recent_changes_#2.patch 46.05 KB

#2

leop - February 14, 2008 - 17:45

hehe, '#'s in filenames should not be allowed. Here is the real patch.

AttachmentSize
recent_changes2.patch 46.05 KB

#3

rötzi - February 20, 2008 - 23:47
Status:needs review» fixed

Thank you very much for your patch. I commited it now into the DRUPAL-5 branch and will do a new release with it.

#4

fuzzy_texan - February 21, 2008 - 01:18

@leop: If this is half as good as it claims to be, you've just made my day. Thanks! I'll test this out on my test instance and report back as to whether it works.

#5

reikiman - February 21, 2008 - 04:00

Issue: In the Recent Changes settings.. if I uncheck 'Show diffs in feeds' then save the form, the form refreshes with that setting rechecked.

#6

reikiman - February 21, 2008 - 04:18

Issue: A non-logged-in browser cannot access the recent_changes page. Instead the message is

Access denied
You are not authorized to access this page.

"access content" and "access revisions" is turned on for anonymous. Most of the content on this site is inside a forum and I have Forum Access in use.

#7

leop - February 21, 2008 - 09:17

#5 I can reproduce this. It is possibly a typo in the variable name.

#6 the recent_changes page needs at least the "access content" permission. Since I cannot reproduce, it might be a problem with Forum Access, which I am not familiar with. Please post a separate issue (since the latest release includes this patch), and let's see if anybody else encounters this problem.

I'll have a look at #5, in the afternoon (Netherlands).

#8

rötzi - February 21, 2008 - 17:58

leop: If you are interested in (co-)maintaining the 'recent_changes' module, please contact me at http://drupal.org/user/73064/contact

#9

leop - February 21, 2008 - 19:54

A patch for #5 is posted at: http://drupal.org/node/225009

#10

leop - February 27, 2008 - 16:38

#11

Anonymous (not verified) - March 12, 2008 - 16:41
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

#12

christefano - June 11, 2008 - 03:40

#133676 has been marked as a duplicate of this issue.

 
 

Drupal is a registered trademark of Dries Buytaert.