Difference between revisions of "Issue:Createdby not working"

From FollowTheScore
Jump to: navigation, search
m
 
(2 intermediate revisions by one other user not shown)
Line 22: Line 22:
  
 
What am I doing wrong or is this a bug?
 
What am I doing wrong or is this a bug?
 +
 +
== Reply ==
 +
 +
Obviously this is a bug.
 +
 +
changing the "=" in source lines 2653 and 2657 to ".=" should solve the problem.
 +
So these lines should read:
 +
 +
        if ($bAddAuthor && $sSqlRevisionTable =='') {
 +
            $sSqlRevisionTable = $sRevisionTable . ' AS rev, ';
 +
    #2653 ----> $sSqlCond_page_rev .= ' AND '.$sPageTable.' .....
 +
        }
 +
        if ($bAddLastEditor && $sSqlRevisionTable =='') {
 +
            $sSqlRevisionTable = $sRevisionTable . ' AS rev, ';
 +
    #2657 ----> $sSqlCond_page_rev .= ' AND '.$sPageTable.'.page_id= ....
 +
        }
 +
 +
Can you please check this? [[User:Gero|Gero]] 23:29, 20 November 2008 (UTC)
 +
 +
==Answer==
 +
 +
Thanks Gero, I have now applied that fix and it works, fine,
 +
 +
Many thanks
 +
 +
Mark.

Latest revision as of 10:42, 21 November 2008

I have just upgraded to DPL 1.7.4 running on

- Windows Server 2003 (in VM-ware, 2GB RAM) - Apache 2.2.4 - php 5.2.4 - Mediawiki 1.9.2 - MySQL 5.0.24a - mod_perl 2.0.3 - Perl 5.8.8 - APC 3.0.15-dev

But when I try and run

<DPL> createdby=Andygray addauthor=true </DPL>

I am getting all users content, i.e. it does not seem to work

What am I doing wrong or is this a bug?

Reply

Obviously this is a bug.

changing the "=" in source lines 2653 and 2657 to ".=" should solve the problem.
So these lines should read:
       if ($bAddAuthor && $sSqlRevisionTable ==) {
           $sSqlRevisionTable = $sRevisionTable . ' AS rev, ';
   #2653 ----> $sSqlCond_page_rev .= ' AND '.$sPageTable.' .....
       }
       if ($bAddLastEditor && $sSqlRevisionTable ==) {
           $sSqlRevisionTable = $sRevisionTable . ' AS rev, ';
   #2657 ----> $sSqlCond_page_rev .= ' AND '.$sPageTable.'.page_id= ....
       }

Can you please check this? Gero 23:29, 20 November 2008 (UTC)

Answer

Thanks Gero, I have now applied that fix and it works, fine,

Many thanks

Mark.