MODX 3.0.0 Language string not found: "visibility"

stormhike · April 7, 2022, 10:56pm

Summary

After a fresh install of 3.0.0 (which is going pretty well) I keep getting the log message:
[2022-04-07 22:22:10] (DEBUG @ … /core/src/Revolution/modLexicon.php : 526) Language string not found: “visibility”

Step to reproduce

No idea.

Environment

I have set my language to ‘en’ which shows when I add extra debugging to the process() function at modLexicon.php : 521. And if it wasn’t set then that routine would use ‘en’ anyway.
I have renamed the connectors and the manager folders.
I don’t have any extras installed yet.

Any ideas?

halftrainedharry · April 8, 2022, 7:35am

This language string should exist:

github.com

modxcms/revolution/blob/fe438bf3dd4a9863bc479e21945d05215141d853/core/lexicon/en/default.inc.php#L242

      
        
            $_lang['install'] = 'Install';
            $_lang['installed'] = 'Installed';
            $_lang['integer'] = 'Integer';
            $_lang['introtext'] = 'Introtext';
            $_lang['invalid_data'] = 'Invalid data';
            $_lang['json_grid'] = 'JSON Grid';
            $_lang['key'] = 'Key';
            $_lang['keyword'] = 'Keyword';
            $_lang['keywords'] = 'Keywords';
            $_lang['last_modified'] = 'Last Modified';
            $_lang['visibility'] = 'Visibility';
            $_lang['launch_site'] = 'View Site';
            $_lang['language'] = 'Language';
            $_lang['lexicon'] = 'Lexicon';
            $_lang['list'] = 'List';
            $_lang['load_headers'] = 'Load Headers';
            $_lang['loading'] = 'Loading...';
            $_lang['locked'] = 'Locked';
            $_lang['locked_by'] = 'locked by [[+username]]';
            $_lang['lock_msg'] = '[[+name]] is currently editing this [[+object]]. Please wait until the other user has finished and try again.';
            $_lang['logged_in_as'] = 'logged in as [[+username]]';

Maybe try clearing the cache.

To you see any output if you put the lexicon tag [[%visibility]] in your content?

markh · April 8, 2022, 3:09pm

Your log_level setting is set to show debug messages; while it’s weird it’s not seeing an existing key, debug messages are not meant to indicate a problem per se.

stormhike · April 8, 2022, 10:03pm

Thanks, yes the string does exist in that file at line 242.
When I add [[%visibility]] it produces ‘Visibility’.
And I’m worried I might wear my cache out I’ve emptied it so many times

I have a lot more problems that I need to work out. I’m struggling with creating my model classes from my hand coded schema. But that’s a different topic.

stormhike · April 8, 2022, 10:13pm

Thanks Mark.
What’s even more weird is that when I add [[%visibility]] it shows the value of the string ‘Visibility’ yet I still get those messages for the same page.

I have no idea why it’s looking up the language string for visibility, I don’t knowingly ask for that in my content or code.

This is not so serious, and I know you guys are super busy, it’s only a debug message and I can live with it for now

markh · April 9, 2022, 12:37am

Fwiw, I think that’s part of the right click menu in the files tab.

halftrainedharry · April 9, 2022, 9:03am

Maybe the extra ExtraBuilder can help you with that.

https://modx.com/extras/package/extrabuilder

stormhike · April 9, 2022, 9:31pm

That didn’t work for me.
I tried it by importing my schema, it showed the tables (albeit without default values), but when I tried to create the class files"Error, Validate ‘bootstrap.php’: Class still does not exist after autoloader registration:". I had changed the schema baseClass to “xPDO\Om\xPDOObject” and the version to 3.0.

Unfortunately ExtraBuilder is poorly documented and the error message didn’t explain why it failed.

I already have my schema and DB tables, all I need is the classes.

I copied my model classes from my 2.8.3 installation to the 3.0.0 one (with the map files) and everything seems to work. Although that’s probably not ideal.

I had previously searched around for ways to create the classes from the schema, like the one at Reverse Engineer from Database - Generating the Model | MODX Documentation but they all seem to be unable to find XPDO, I guess it has moved since 2.x.

halftrainedharry · April 10, 2022, 10:28am

Yes, this doesn’t work anymore:

$xpdo_path = strtr(MODX_CORE_PATH . 'xpdo/xpdo.class.php', '\\', '/');
include_once ( $xpdo_path );
...
$xpdo = new xPDO("mysql:host=$database_server;dbname=$dbase", $database_user, $database_password, $table_prefix);

MODX3 now uses namespaces and autoloader.
You probably have to change it to

require_once MODX_CORE_PATH . "vendor/autoload.php";
...
$xpdo = new xPDO\xPDO("mysql:host=$database_server;dbname=$dbase", $database_user, $database_password, $table_prefix);

stormhike · April 10, 2022, 6:16pm

Thank you halftrainedharry for helping.
I’ve decided to give up on xPDO and use PDO instead for my custom tables, it has decent documentation and is stable.

digitalpenguin · April 11, 2022, 2:03am

I had previously searched around for ways to create the classes from the schema, like the one at Reverse Engineer from Database - Generating the Model | MODX Documentation but they all seem to be unable to find XPDO, I guess it has moved since 2.x.

If you’re looking to build the xPDO model classes from the schema, then you’d need a build schema script. Rather than reverse-engineering from the database, that is.

It might be helpful to have a look at the Articles build schema script as it’s just been refactored for MODX 3.0 (and thus xPDO 3 too).

Here’s the build schema script for Articles on MODX 2.x:

github.com

modxcms/Articles/blob/506c07202f268a966884164b0b41f96a856d41f8/_build/build.schema.php

<?php
/**
 * Articles
 *
 * Copyright 2011-12 by Shaun McCormick <shaun+articles@modx.com>
 *
 * Articles is free software; you can redistribute it and/or modify it under the
 * terms of the GNU General Public License as published by the Free Software
 * Foundation; either version 2 of the License, or (at your option) any later
 * version.
 *
 * Articles is distributed in the hope that it will be useful, but WITHOUT ANY
 * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
 * A PARTICULAR PURPOSE. See the GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License along with
 * Articles; if not, write to the Free Software Foundation, Inc., 59 Temple
 * Place, Suite 330, Boston, MA 02111-1307 USA
 *
 * @package articles

This file has been truncated. show original

And here’s the one for MODX 3.x:

github.com

modxcms/Articles/blob/83ecf47e0299ceb152f24a2730a9e52bc159d97e/_build/build.schema.php

<?php
/**
 * Articles
 *
 * Copyright 2011-12 by Shaun McCormick <shaun+articles@modx.com>
 *
 * Articles is free software; you can redistribute it and/or modify it under the
 * terms of the GNU General Public License as published by the Free Software
 * Foundation; either version 2 of the License, or (at your option) any later
 * version.
 *
 * Articles is distributed in the hope that it will be useful, but WITHOUT ANY
 * WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
 * A PARTICULAR PURPOSE. See the GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License along with
 * Articles; if not, write to the Free Software Foundation, Inc., 59 Temple
 * Place, Suite 330, Boston, MA 02111-1307 USA
 *
 * @package articles

This file has been truncated. show original

halftrainedharry · April 11, 2022, 9:21am

There is also this “Using Custom Database Tables” tutorial in the documentation, that has recently been updated for MODX3 and contains a simple example of a schema and a built script.

digitalpenguin · April 11, 2022, 9:29am

Fantastic! I wasn’t aware of that.

stormhike · April 12, 2022, 6:59pm

Murray,
That looks good, thank you. Unlike Articles I don’t have a pressing need to make the move to 3.0.
I’m switching my custom tables from xPDO to PDO, and moving my tables into a database separate from MODX. I’m not creating an extra so I don’t need to use xPDO.