Ascending

For questions and problems with the CMS core. This board is NOT for any 3rd party modules, addons, PHP scripts or anything NOT distributed with the CMS made simple package itself.
Locked
TNTH

Ascending

Post by TNTH »

I've got a problem with the "Sort Ascending:" button. When put it on, it put's itselfes off when pressing the apply- or submitbutton. I've got the problem in FF and in IE and I'm using the 0.9-release.

Can someone help me out???
TNTH

still...

Post by TNTH »

I still haven't got an answer and I still have the problem in version 0.9.2 en 0.8.2.

Anyone any idea???
Ted
Power Poster
Power Poster
Posts: 3329
Joined: Fri Jun 11, 2004 6:58 pm

Ascending

Post by Ted »

Sorry. Sometimes I miss some posts.

I'm not completely sure where the Sort Ascending button is...
TNTH

Screenshot

Post by TNTH »

No problem, it's getting busy here. Even I'm not able anymore to read all the posts :)

The Sort Ascending button is at Content Management >> News (if you'll have a newspage and at Content Type chose News)

Here is a screenshot (49 kb, 1024x660px, gif).

Hope you can help me, I currently am using 0.8.2 at the website.
TNTH

Re: Ascending

Post by TNTH »

Jeah... i solved my problem....here's what i changed

Within the cmsmadesimple\modules\news\News.module.php file
CMS version: 0.9.2

old    274:

Code: Select all

if (isset($params["sortasc"]))
new  274:

Code: Select all

if ($params["sortasc"]==0)
old  1010:

Code: Select all

if (isset($params['sortasc']) && $params['sortasc'] == 1)
new 1010:

Code: Select all

if (isset($params['sortasc']))
I looked at a similar option(Show Category)...at least i think it workes the same, i edited the above lines while thinking of the similar option.
and it worked for me.
I would like to have feedback on this code if anyone has problems.

In version 0.8.2 the following file has to be changed: cmsmadesimple\modules\news\modulefunctions.php

old    134:

Code: Select all

if (isset($params["sortasc"]))
new  134:

Code: Select all

if ($params["sortasc"]==0)
It should work now in 0.8.2 too.

good luck ;)
Last edited by TNTH on Thu Mar 03, 2005 11:12 am, edited 1 time in total.
Ted
Power Poster
Power Poster
Posts: 3329
Joined: Fri Jun 11, 2004 6:58 pm

Re: Ascending

Post by Ted »

Thanks very much for the fix.  I'll put it in later today.
TNTH

Re: Ascending

Post by TNTH »

You're welcome :-)

I'll hope that the other users have profit of it.
Sjors

Re: Ascending

Post by Sjors »

I am using version 0.9.2 and I was able to find the code
if (isset($params["sortasc"]))
(although only once and not at the same line number), but not
if (isset($params['sortasc']) && $params['sortasc'] == 1)
. So I changed the first part, but this doesn't have any effect.

Sjors
Nicke

Re: Ascending

Post by Nicke »

The bug still hasnt been fixed!

I have the lates version and the code looks like this:

Code: Select all

                if (isset($params["sortasc"])) 
                { 
                    $query .= "asc"; 
                } 
                else 
                { 
                    $query .= "desc"; 
                }
Could someone fix this soon?
Locked

Return to “CMSMS Core”