[phpws-comm-auto] [ phpwebsite-comm-Support Requests-771650 ] Module Article: expiring articles fails

[SourceForge.net <no...@so...>]

Support Requests item #771650, was opened at 2003-07-15 11:05
Message generated for change (Comment added) made by adarkling
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=563509&aid=771650&group_id=81360

Category: Module: article
Group: Second Level Support
>Status: Closed
>Resolution: Fixed
Priority: 6
Submitted By: David M Walker (datamgmt)
Assigned to: Eloi George (adarkling)
Summary: Module Article: expiring articles fails

Initial Comment:
Expiring articles fails with message:

Error: sqlUpdate() statement was incomplete:
update mod_article set expiration_date='2003-07-15 
14:59:11' where id = NULL

Notably the address now says
http://www.datamgmt.com/index.php#6
and I was trying to expire article 6 so I guess the
command line is getting hosed in some way



----------------------------------------------------------------------

>Comment By: Eloi George (adarkling)
Date: 2003-07-15 13:37

Message:
Logged In: YES 
user_id=619893

I am continually amazed at how many bugs can stay hidden
until someone else looks at it!

Thanks for pointing this out.  I'll release a new bugfix
tomorrow morning, but until then you can fix it by doing this:

In /mod/article/class/Article.php change line 966 from:

$GLOBALS['core']->sqlUpdate($query_data, 'mod_article',
'id', $_SESSION['SES_ART_article']->id);

to:

$GLOBALS['core']->sqlUpdate($query_data, 'mod_article',
'id', $this->id);


The command line is OK.  It's just telling the browser to
position the screen at the article (#6) you just expired so
you don't have to scroll down the list to find it again.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=563509&aid=771650&group_id=81360