11
Mamba
Re: Problems with Lexicon
  • 2017/2/6 12:32

  • Mamba

  • Moderator

  • Posts: 11366

  • Since: 2004/4/23


Quote:
this is the most recent version of lexicon. updated 18 days ago

just another xoops module that doesn't work

Updated, but not released, it's still a Beta:
https://github.com/XoopsModules25x/lexikon/blob/master/xoops_version.php

As in every Beta, you should expect bugs, which you report, and hopefully somebody will fix them.

And everybody appreciates your testing and bug reporting.
Support XOOPS => DONATE
Use 2.5.10 | Docs | Modules | Bugs

12
aerograf
Re: Problems with Lexicon
  • 2017/2/6 13:51

  • aerograf

  • Quite a regular

  • Posts: 214

  • Since: 2017/1/7 1


Quote:

Bleekk wrote:
@aerograf

maybe it helps if you redirect to the index page if a user calls a non-existent page

go to entry.php line 81
change this
// verify result
    
if ($xoopsDB->getRowsNum($result) <= 0) {
        
redirect_header('javascript:history.go(-1)'2_MD_LEXIKON_UNKNOWNERROR);
        exit();
    }


to
// verify result
    
if ($xoopsDB->getRowsNum($result) <= 0) {
        
redirect_header(XOOPS_URL.'/modules/lexikon/index.php'2_MD_LEXIKON_UNKNOWNERROR);
        exit();
    }

Thank you!!! Less errors from robots will.
The same should be done in category.php line 117, reduced to the form:
// verify ID
    
if ($xoopsDB->getRowsNum($catdata) <= 0) {
        
redirect_header(XOOPS_URL.'/modules/lexikon/index.php'2_MD_LEXIKON_UNKNOWNERROR);
        exit();
    }

But why do we need /include/notification.inc.php? Like he should do a redirect? Or am I wrong?

13
Bleekk
Re: Problems with Lexicon
  • 2017/2/6 14:04

  • Bleekk

  • Theme Designer

  • Posts: 940

  • Since: 2002/12/14


you are welcome

14
Zap_English
Re: Problems with Lexicon

Quote:

Mamba wrote:


And everybody appreciates your testing and bug reporting.


ok, so if anyone can fix the bug i'm having in publisher, i can let my site go live

15
Zap_English
Re: Problems with Lexicon

Quote:

Bleekk wrote:
@Zap_English
Be part of the community and create the bootstrap templates for the module or wait till someone do the work for you! We area community and do our best.

Have you tried to use the module first without porting your old database? Does it work if you create entries on your own first?


I'm not a programmer

yes. I tried it with test data i inputted into the module and it still appears the same way in xbootstrap

Other modules I tested that don't have the xbootstrap template worked fine and didn't show the "cover" photo as their background

16
Mamba
Re: Problems with Lexicon
  • 2017/2/6 16:06

  • Mamba

  • Moderator

  • Posts: 11366

  • Since: 2004/4/23


Please test Lexikon 1.52 Beta 2

https://github.com/mambax7/lexikon/releases
Support XOOPS => DONATE
Use 2.5.10 | Docs | Modules | Bugs

17
Zap_English
Re: Problems with Lexicon

That fixes the background problem. I'll let you know later if it fixes the import problem

18
Mamba
Re: Problems with Lexicon
  • 2017/2/6 17:17

  • Mamba

  • Moderator

  • Posts: 11366

  • Since: 2004/4/23


Quote:
I'll let you know later if it fixes the import problem

I didn't test any imports.

What are you importing from?
Support XOOPS => DONATE
Use 2.5.10 | Docs | Modules | Bugs

19
aerograf
Re: Problems with Lexicon
  • 2017/2/6 17:24

  • aerograf

  • Quite a regular

  • Posts: 214

  • Since: 2017/1/7 1


Quote:

Mamba wrote:
Please test Lexikon 1.52 Beta 2

https://github.com/mambax7/lexikon/releases

First, it is necessary to replace category.php line:
// verify ID
    
if ($ xoopsDB-> getRowsNum ($ catdata) <= 0) {
        
redirect_header 'javascript: history.go (-1)'2_MD_LEXIKON_UNKNOWNERROR);
    }

on
// verify ID
    
if ($ xoopsDB-> getRowsNum ($ catdata) <= 0) {
        
redirect_header (XOOPS_URL '/ modules / lexikon / index.php'2_MD_LEXIKON_UNKNOWNERROR.);
    }

otherwise proper operation category.php? categoryID = true unless specified.

The rest look. Thank you!

20
Mamba
Re: Problems with Lexicon
  • 2017/2/6 17:37

  • Mamba

  • Moderator

  • Posts: 11366

  • Since: 2004/4/23


Quote:
First, it is necessary to replace category.php line:

Done:https://github.com/mambax7/lexikon/commit/8b54ba2420ef70cee4daf504fb0de7472067b18c
Support XOOPS => DONATE
Use 2.5.10 | Docs | Modules | Bugs

Login

Who's Online

169 user(s) are online (93 user(s) are browsing Support Forums)


Members: 0


Guests: 169


more...

Donat-O-Meter

Stats
Goal: $100.00
Due Date: Mar 31
Gross Amount: $0.00
Net Balance: $0.00
Left to go: $100.00
Make donations with PayPal!

Latest GitHub Commits