News Box Manager v3.0.0 [Support Thread]

[gernot]

Hi lat9,
I did a comparison of the files described in the docs folder of the add-on for News Box Manager v3.0.2, and what files are actually included.
I think the main problem might be an incorrectly-named definition file? Not sure about the mismatch of the various files, so will wait for confirmation.

As listed in docs
(I marked as OK if the described files were actually there)
OK: /includes/classes/observers/auto.news_box_manager_metatags.php
Does not exist---wrong Name?: /includes/extra_datafiles/news_box_manager_file_database_names.php
OK: /includes/languages/english/all_articles.php
OK: /includes/languages/english/article.php
OK: /includes/languages/english/extra_definitions/news_box_manager_defines.php
OK: /includes/modules/news_box_format.php
OK: /includes/modules/pages/all_articles/header_php.php
OK: /includes/modules/pages/article/header_php.php
OK: /includes/modules/pages/more_news/header_php.php
OK: /includes/modules/pages/news_archive/header_php.php
OK: /includes/modules/sideboxes/news_box_sidebox.php
OK: /includes/templates/template_default/sideboxes/tpl_news_box_sidebox.php
Does not exist: /includes/templates/template_default/sideboxes/tpl_news_box_sidebox2.php
OK: /includes/templates/template_default/templates/tpl_modules_news_box_format.php
OK: /includes/templates/template_default/templates/tpl_all_articles_default.php
OK: /includes/templates/template_default/templates/tpl_article.php
OK: /includes/templates/YOUR_TEMPLATE/css/stylesheet_news_box_manager.css
OK: /includes/templates/YOUR_TEMPLATE/templates/tpl_index_default.php
OK: /YOUR_ADMIN/news_box_manager.php
OK: /YOUR_ADMIN/includes/auto_loaders/config.news_box_manager.php
OK: /YOUR_ADMIN/includes/extra_datafiles/news_box_manager_file_database_names.php
OK: /YOUR_ADMIN/includes/extra_datafiles/news_box_manager_sanitization.php
OK: /YOUR_ADMIN/includes/init_includes/init_news_box_manager_admin.php
OK: /YOUR_ADMIN/includes/languages/english/news_box_manager.php
OK: /YOUR_ADMIN/includes/languages/english/extra_definitions/news_box_manager_menu_name.php

Files existing but not described in docs:
/YOUR_ADMIN/news_box_manager1.php
/YOUR_ADMIN/news_box_manager2.php
/YOUR_ADMIN/news_box_manager3.php
/YOUR_ADMIN/news_box_manager4.php
/YOUR_ADMIN/includes/classes/observers/NewsBoxManagerAdminObserver.php
/YOUR_ADMIN/includes/functions/news_box_manager_functions.php
/YOUR_ADMIN/includes/init_includes/init_news_box_manager_install.pph
/YOUR_ADMIN/includes/init_includes/init_news_box_manager_update.php
Possibly wrong name?: /includes/extra_datafiles/news_box_file_database_names.php
/includes/modules/sideboxes/news_box_sidebox2.php
/includes/templates/YOUR_TEMPLATE/jscript/jscript_sidebox_slider.js

[lat9]

@gernot, thanks for pointing out the issues with the documentation. I'll be getting that updated.

Since there are no core-file overwrites in the plugin, you can safely copy all the files from the plugin's distribution to apply those updates.

That TABLE_BOX_NEWS_CONTENT value is defined in the admin's /extra_datafiles module.

[gernot]

Hi lat9,
Can you confirm whether the extra_datafiles file has a name issue? As I wrote, I did copy all the files that were in the plugin, but the DB was not properly initialized, and I wonder if that might be caused by the file name not having the word "manager" in its name?

[K1979]

Hi,

Thank you for the mod, I updated from v2 to v3


I got error log as below.

PHP Warning: constant(): Couldn't find constant BOX_NEWS_NAME_TYPE in /home/****/www/***admin***/news_box_manager.php on line 719.

news_box_manager.php line719

<div class="col-sm-2"><?php echo constant("BOX_NEWS_NAME_TYPE$news_content_type"); ?></div>

It will be used in a preview page, if I update or add a news from [All Types], it's OK.
But if I do from [Type1] or [Type2] ,[Type*] menu, it shows nothing in the "News Type:" column.

[lat9]

Those BOX_NEWS_NAME_TYPE[1-4] constants are provided in the distribution's storefront /includes/languages/english/extra_definitions/news_box_manager_defines.php. That file (or your template's version thereof) is used in both the admin and storefront processing.

Check to see if you have an older version of that file present in /includes/languages/english/extra_definitions/YOUR_TEMPLATE that doesn't have those additional defines.

[K1979]

Thank you for your reply.

My explanation was not enough.
I have removed all v2 files when I update for my test website.
I haven't changed the definition in the "news_box_manager_defines.php" yet, but I think it works because I can see the word Type1 not BOX_NEWS_NAME_TYPE1.

If I insert a news from Tools -> News Box Manager[All Types] -> Insert -> Preview, "News Type:" shows its type.


And if I insert a news from Tools -> News Box Manager[Type1(or 2,3,4)] -> Insert -> Preview, "News Type:" shows nothing and I get error log.


PHP Warning: constant(): Couldn't find constant BOX_NEWS_NAME_TYPE in /home/****/www/***admin***/news_box_manager.php on line 719.


I'm right in the middle of learning PHP, so if you have any suggestions please let me know.
Thank you,

[K1979]

Thank you for your reply.

My explanation was not enough.
I have removed all v2 files when I update for my test website.
I haven't changed the definition in the "news_box_manager_defines.php" yet, but I think it works because I can see the word Type1 not BOX_NEWS_NAME_TYPE1.

If I insert a news from Tools -> News Box Manager[All Types] -> Insert -> Preview, "News Type:" shows its type.


And if I insert a news from Tools -> News Box Manager[Type1(or 2,3,4)] -> Insert -> Preview, "News Type:" shows nothing and I get error log.


PHP Warning: constant(): Couldn't find constant BOX_NEWS_NAME_TYPE in /home/****/www/***admin***/news_box_manager.php on line 719.


I'm right in the middle of learning PHP, so if you have any suggestions please let me know.
Thank you,

I will correct the post, when I insert a news, it's OK.
But If I modify a news from "News Box Manager[Type1(or 2,3,4)]", I got the error.

[lat9]

Hi lat9,
Can you confirm whether the extra_datafiles file has a name issue? As I wrote, I did copy all the files that were in the plugin, but the DB was not properly initialized, and I wonder if that might be caused by the file name not having the word "manager" in its name?

It has nothing to do with the filenames, just their locations. Were there any logs generated?

[gernot]

Hi lat9,
No, no logs were generated, which I thought was a bit strange, but strict error checking enabled shows up the error on the screen, as I pasted in my initial post.
I have the file in the admin extra datafiles places as instructed:

test-admin# cat includes/extra_datafiles/news_box_manager_file_database_names.php 
<?php
// -----
// Part of the News Box Manager plugin, re-structured for Zen Cart v1.5.6 and later by lat9.
// Copyright (C) 2015-2019, Vinos de Frutas Tropicales
//
// +----------------------------------------------------------------------+
// | Do Not Remove: Coded for Zen-Cart by geeks4u.com                     |
// | Dedicated to Memory of Amelita "Emmy" Abordo Gelarderes              |
// +----------------------------------------------------------------------+
//
define('TABLE_BOX_NEWS', DB_PREFIX . 'box_news');
define('TABLE_BOX_NEWS_CONTENT', DB_PREFIX . 'box_news_content');

define('FILENAME_NEWS_BOX_MANAGER', 'news_box_manager');
define('FILENAME_NEWS_BOX_MANAGER1', 'news_box_manager1');
define('FILENAME_NEWS_BOX_MANAGER2', 'news_box_manager2');
define('FILENAME_NEWS_BOX_MANAGER3', 'news_box_manager3');
define('FILENAME_NEWS_BOX_MANAGER4', 'news_box_manager4');

Both the box_news and box_news_content tables do exist apparently:

MariaDB [zencartshoptest]> desc box_news;
+--------------------+------------+------+-----+---------------------+----------------+
| Field              | Type       | Null | Key | Default             | Extra          |
+--------------------+------------+------+-----+---------------------+----------------+
| box_news_id        | int(11)    | NO   | PRI | NULL                | auto_increment |
| news_added_date    | datetime   | YES  |     | 0001-01-01 00:00:00 |                |
| news_modified_date | datetime   | YES  |     | NULL                |                |
| news_start_date    | datetime   | YES  |     | NULL                |                |
| news_end_date      | datetime   | YES  |     | NULL                |                |
| news_status        | tinyint(1) | YES  |     | 0                   |                |
| news_content_type  | tinyint(1) | NO   |     | 1                   |                |
+--------------------+------------+------+-----+---------------------+----------------+
7 rows in set (0.014 sec)

MariaDB [zencartshoptest]> desc box_news_content;
+---------------------------+--------------+------+-----+---------+-------+
| Field                     | Type         | Null | Key | Default | Extra |
+---------------------------+--------------+------+-----+---------+-------+
| box_news_id               | int(11)      | NO   | PRI | 0       |       |
| languages_id              | int(11)      | NO   | PRI | 1       |       |
| news_title                | varchar(255) | NO   |     |         |       |
| news_content              | text         | NO   |     | NULL    |       |
| news_metatags_title       | varchar(255) | NO   |     |         |       |
| news_metatags_keywords    | text         | YES  |     | NULL    |       |
| news_metatags_description | text         | YES  |     | NULL    |       |
+---------------------------+--------------+------+-----+---------+-------+
7 rows in set (0.001 sec)

[gernot]

lat9,
It's official: I am retarded. After you asked about logs again, I did look again, and just for, you know, dumb*ss checking, I looked at my production shop logs, and there I see error logs related to my test shop!
So of course I realize I screwed up something... turns out in the storefront includes/configure.php I had the DIR_FS_CATALOG set to my production shop.
All other defines were fine in the storefront configure, just that one was wrong. Ouch! The admin configure file was completely fine.
That explains why the admin side worked but the display did not.
So sorry for the hassle.

Remaining questions:
Are there missing files in the plugin, for sideboxes? Or is it merely the documentation that needs updating?