$entry.last_modified not avail when $view==search

Post by **Don Chambers** » Sat Jul 19, 2008 6:30 am

happened to catch an error in one of my logs... turns out that $entry.last_modified is not an available field in a quicksearch view.... this reminds me of this thread: http://board.s9y.org/viewtopic.php?t=12742

This is confirmed on a live site, my sandbox, and also by Judebert. Any ideas?

judebert · Post by **judebert** » Mon Jul 21, 2008 5:49 pm

I don't see any different SQL being called, so I can't figure out why $entry.last_modified isn't set. (Bump.)

Post by **Don Chambers** » Tue Jul 22, 2008 3:41 pm

Hey Garvin - do you have any input on this issue?

Post by **garvinhicking** » Tue Jul 22, 2008 3:45 pm

Hi!

Don Chambers wrote:Hey Garvin - do you have any input on this issue?

Sadly not right now.

Regards,
Garvin

Post by **Don Chambers** » Tue Jul 22, 2008 3:49 pm

I read elsewhere that an updated release may be necessary - perhaps this bugfix can make its way into that? BTW - this was 1.3.x branch.

I have a template that provides functionality based on that field, so right now, that functionality is broken on search results pages.

judebert · Post by **judebert** » Tue Jul 22, 2008 9:38 pm

Got it.

When searching, we call serendipity_searchEntries(). When just reading, we call the (much more complex and versatile) serendipity_fetchEntries().

Guess which one doesn't have the SQL for last_modified?

In fact, the basic SQL is not significantly different. I'm wondering if perhaps we should have searchEntries() call fetchEntries(), so they both fetch the same data. Garvin?

While checking the two queries to see how far they differed, I found that isdraft and loginname are also not included. However, body, extended, and the "moderate_comments" setting are added.

So, I just added e.isdraft, e.last_modified, and a.username AS loginname to the SQL that starts on line 776 of functions_entries.inc.php. For the 1.4/trunk branch, the query starts on line 781.

I tested it in my sandbox to make sure it works. Since this is a core file, it can't be downloaded from SPARTACUS. You can wait for the next nightly build (within 24 hours) or download the file directory from:
1.3 branches
http://svn.berlios.de/viewcvs/*checkout ... es.inc.php
trunk installations
http://svn.berlios.de/viewcvs/*checkout ... es.inc.php
...and copy it into your serendipity/include/ directory.

Post by **Don Chambers** » Wed Jul 23, 2008 4:13 pm

Thanks a bunch Jude!!!! Works perfectly. Much appreciated!