xoops forums

aerograf

Quite a regular
Posted on: 2017/5/3 19:29
aerograf
aerograf (Show more)
Quite a regular
Posts: 213
Since: 2017/1/7 1
#1

xsitemap alternative theme

I want to share a topic for xsitemap. Maybe someone will like it:
Demo (temporary): http://xoops.shmel.org/modules/xsitemap/
Download: http://xoops.shmel.org/dw/xsitemap_theme.rar

Bleekk

Theme Designer
Posted on: 2017/5/3 19:43
Bleekk
Bleekk (Show more)
Theme Designer
Posts: 935
Since: 2002/12/14
#2

Re: xsitemap alternative theme

Have you changed the module or only the templates of the Module?

aerograf

Quite a regular
Posted on: 2017/5/3 20:16
aerograf
aerograf (Show more)
Quite a regular
Posts: 213
Since: 2017/1/7 1
#3

Re: xsitemap alternative theme

So far, only a template. But still in front.

Bleekk

Theme Designer
Posted on: 2017/5/3 20:48
Bleekk
Bleekk (Show more)
Theme Designer
Posts: 935
Since: 2002/12/14
#4

Re: xsitemap alternative theme

Thank you!

zyspec

Module Developer
Posted on: 2017/5/3 22:34
zyspec
zyspec (Show more)
Module Developer
Posts: 1075
Since: 2004/9/21
#5

Re: xsitemap alternative theme

@aerograf / @Bleekk,

There was a new version of xsitemap uploaded today (v1.54 Beta) that you might want to look at before you modify anything other than templates. There were quite a few changes/improvements so it would be better to start modifying that version so we don't have to merge your new changes from an older version.

aerograf

Quite a regular
Posted on: 2017/5/4 5:58
aerograf
aerograf (Show more)
Quite a regular
Posts: 213
Since: 2017/1/7 1
#6

Re: xsitemap alternative theme

Thank you. I'll take advice.

aerograf

Quite a regular
Posted on: 2017/5/4 13:06
aerograf
aerograf (Show more)
Quite a regular
Posts: 213
Since: 2017/1/7 1
#7

Re: xsitemap alternative theme

Quote:

zyspec wrote:
@aerograf / @Bleekk,

There was a new version of xsitemap uploaded today (v1.54 Beta) that you might want to look at before you modify anything other than templates. There were quite a few changes/improvements so it would be better to start modifying that version so we don't have to merge your new changes from an older version.

Revealed during the proceedings:
Include / functions.php to bring line 205 to this form:
Xml_url-> appendChild ($ xml-> createComment (htmlentities (ucwords ($ mod ['name'])). ""));

Fixes the problem of creating an xml file if there is a category name -
-----
If you use the xml_google.php file in the root of the site, you should add at the beginning:
If (! Defined ('SITEMAP_ROOT_CONTROLLER_LOADED'))) {
If (! 
File_exists (dirname (__ FILE __). '/ Modules / xsitemap / xml_google.php')) {
Die (
"Do not call this file directly");
}
If (! Empty ($ 
_SERVER ['REQUEST_URI']))) {
_SERVER ['REQUEST_URI'] = str_replace ('xml_google.php''modules / xsitemap / xml_google.php', $ _SERVER ['REQUEST_URI']);
} Else {
_SERVER ['REQUEST_URI'] = '/modules/xsitemap/xml_google.php';
}
_SERVER ['PHP_SELF'] = $ _SERVER ['REQUEST_URI'];
Define ('SITEMAP_ROOT_CONTROLLER_LOADED'1);
Real_xml_google_path dirname (__ FILE __). '/ Modules / xsitemap / xml_google.php';
Chdir ('./modules/xsitemap/');
Require $ 
real_xml_google_path;
Exit;
} Else {
Require_once 
__DIR__'../../mainfile.php';
}

aerograf

Quite a regular
Posted on: 2017/5/4 13:11
aerograf
aerograf (Show more)
Quite a regular
Posts: 213
Since: 2017/1/7 1
#8

Re: xsitemap alternative theme

It is necessary to understand still with correct generation of a file xsitemap.xml
Search engines react poorly to reading subcategories from a file. Issue a warning.

zyspec

Module Developer
Posted on: 2017/5/5 14:49
zyspec
zyspec (Show more)
Module Developer
Posts: 1075
Since: 2004/9/21
#9

Re: xsitemap alternative theme

EDIT: Can I ask if you've tried turning off subcategories in the module's Preferences to see how the search engines react?

I think I see the problem with sub-categories... the XML isn't being formed correctly when there are subcategories.

zyspec

Module Developer
Posted on: 2017/5/5 15:45
zyspec
zyspec (Show more)
Module Developer
Posts: 1075
Since: 2004/9/21
#10

Re: xsitemap alternative theme

I just issued a pull request to fix the sub-category problem with the XML rendering. You can see the changes at https://github.com/XoopsModules25x/xsi ... 9301436b6101e11ead630dbc3

The changes that effect the sub-cateogory issue are contained in the ./include/function.php file. The changes in the xoops_version.php file aren't required, I just updated the date so we could tell that the BETA 1 had been updated.