Some minor fixes to 1.6.7

The place to talk about things that are related to CMS Made simple, but don't fit anywhere else.
Post Reply
shtepawski
New Member
New Member
Posts: 7
Joined: Sun Mar 07, 2010 8:49 am

Some minor fixes to 1.6.7

Post by shtepawski »

Hello.

Maybe someone is interested in the fixes i've made to Teremba Bay release.
Here they are:

1. fix plugins/function.cms_selflink.php

434: $result .= '/>';

in order to have closing `/>' instead of "".

2. simple tagging plugin fix
(server complaints "Notice: Use of undefined constant content - assumed 'content' in [SKIPPED]")

328: 329: 335:
Just put [content] and [Tags] in '', so it turns to ['content'] and ['Tags'].

3. Minor fix to visitor stats

I can't remember the line, but there is one typo in the code..

4. Also i couldn't get Glossary work with pretty urls correctly.
Maybe it's possible for community to improve the module and represent it, since it's of great value for site owners.

Thank you.

P.s. sorry if i confused a category to post this stuff in..
User avatar
Dr.CSS
Moderator
Moderator
Posts: 12711
Joined: Thu Mar 09, 2006 5:32 am

Re: Some minor fixes to 1.6.7

Post by Dr.CSS »

This is invalid code
shtepawski
New Member
New Member
Posts: 7
Joined: Sun Mar 07, 2010 8:49 am

Re: Some minor fixes to 1.6.7

Post by shtepawski »

Thank you! You're right.
How could i mess href tag with those who must have ending `/>' %-\

I don't know should we bother about one space left between the title and closing `>'. Have got to take a look at the specification.

Well, after the changes to simpletagging module there is still one warning left
"Notice: Undefined index: Tags in /home/lg-consult/lg-consult.ru/docs/modules/simpletagging/simpletagging.module.php on line 329"

It happens after the changes to the content page is done which doesn't have a tag field, because this page wasn't intended to work with this feature anyway.

The following code solves the issue:

                                if (isset($params['content']->mProperties->mPropertyValues['Tags']))
                                {
                                  $tags = $params['content']->mProperties->mPropertyValues['Tags'];
                                }
                                else
                                {
                                  $tags = '';
                                }

However it doesn't seem to be a problem, since the value of Tags is automatically assigned to ''. It's just a matter of observing warning message each time a content is edited.
User avatar
Dr.CSS
Moderator
Moderator
Posts: 12711
Joined: Thu Mar 09, 2006 5:32 am

Re: Some minor fixes to 1.6.7

Post by Dr.CSS »

You do realize that if you change core module code or the code of any module you render yourself to being non-supported in the core or the module, if you know how to hack code you must know enuf to do your own support...
Post Reply

Return to “The Lounge”