[CDB] cmBB - Simple Content Management for phpBB

A place for Extension Authors to post and receive feedback on Extensions still in development. No Extensions within this forum should be used within a live environment!
Suggested Hosts
Forum rules
READ: phpBB.com Board-Wide Rules and Regulations

IMPORTANT: Extensions Development rules

IMPORTANT FOR NEEDED EVENTS!!!
If you need an event for your extension please read this for the steps to follow to request the event(s)
Peter77sx
Registered User
Posts: 3253
Joined: Wed Nov 09, 2005 2:51 pm

Re: [BETA][3.2] cmBB - Simple Content Management for phpBB

Post by Peter77sx » Wed Jan 18, 2017 3:48 am

Couple things I found-
1) When Enable URL Rewriting: in server settigns is checked, user avatar on articles/news show broken.


2)Left side bar link to articles/forum post show a double slash after forum directory so site.com/phpbb3//viewtopic.php=etc URL rewriting or not.

probably not a big deal but whatever, thought I'd mention it. :)


Articles seem to be social sharing friendly out-of-the-box... congrats, well done.

User avatar
Ger
Recognised Extension Developer
Posts: 1845
Joined: Wed Jan 02, 2008 7:35 pm
Location: 192.168.1.100
Contact:

Re: [DEV] cmBB - Simple Content Management for phpBB

Post by Ger » Wed Jan 18, 2017 7:58 am

MaFeSa wrote:
Tue Jan 17, 2017 11:56 pm
Hi, I downloaded the latest .zip from github but I have this error

Code: Select all

SQL ERROR [ mysqli ]

Expression #1 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'phpbb.c.category_name' which is not functionally dependent on columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by [1055]

SQL

SELECT category_name, alias, article_id FROM (phpbb_cmbb_category c CROSS JOIN phpbb_cmbb_article a) WHERE std_parent > 1 AND std_parent = article_id AND show_menu_bar = 1 GROUP BY article_id

BACKTRACE

FILE: (not given by php)
LINE: (not given by php)
CALL: msg_handler()

FILE: [ROOT]/phpbb/db/driver/driver.php
LINE: 999
CALL: trigger_error()

FILE: [ROOT]/phpbb/db/driver/mysqli.php
LINE: 193
CALL: phpbb\db\driver\driver->sql_error()

FILE: [ROOT]/phpbb/db/driver/factory.php
LINE: 329
CALL: phpbb\db\driver\mysqli->sql_query()

FILE: [ROOT]/ext/ger/cmbb/cmbb/driver.php
LINE: 74
CALL: phpbb\db\driver\factory->sql_query()

FILE: [ROOT]/ext/ger/cmbb/event/main_listener.php
LINE: 81
CALL: ger\cmbb\cmbb\driver->list_menu_items()

FILE: (not given by php)
LINE: (not given by php)
CALL: ger\cmbb\event\main_listener->page_header_add_menu()

FILE: [ROOT]/vendor/symfony/event-dispatcher/EventDispatcher.php
LINE: 184
CALL: call_user_func()

FILE: [ROOT]/vendor/symfony/event-dispatcher/EventDispatcher.php
LINE: 46
CALL: Symfony\Component\EventDispatcher\EventDispatcher->doDispatch()

FILE: [ROOT]/phpbb/event/dispatcher.php
LINE: 60
CALL: Symfony\Component\EventDispatcher\EventDispatcher->dispatch()

FILE: [ROOT]/phpbb/event/dispatcher.php
LINE: 46
CALL: phpbb\event\dispatcher->dispatch()

FILE: [ROOT]/includes/functions.php
LINE: 4158
CALL: phpbb\event\dispatcher->trigger_event()

FILE: [ROOT]/index.php
LINE: 238
CALL: page_header()
Thanks
Hi, thank you for your report. I seems to come from your sql_mode, which states only_full_group_by. That's default enabled since MySQL 5.7.5 I have discovered.

Can you try this for me, it should fix it in your server:

OPEN ./ext/ger/cmbb/cmbb/driver.php

FIND

Code: Select all

'GROUP_BY'	 => 'article_id',
REPLACE WITH

Code: Select all

'GROUP_BY'	 => 'category_name, alias, article_id',
Save & upload file and purge the cache from ACP index.

Does this solve it for you? If so, I will fix this in the extension.
My extensions:
Simple CMS, Feed post bot, Avatar Resize, Modbreak, Magic OGP, Live topic update, Modern Quote, Quoted Where (GDPR) and Autoresponder.
Newest: FAQ manager for 3.2

Like my work? Buy me a coffee to keep it coming. :ugeek:
-Available for custom work-

User avatar
Ger
Recognised Extension Developer
Posts: 1845
Joined: Wed Jan 02, 2008 7:35 pm
Location: 192.168.1.100
Contact:

Re: [BETA][3.2] cmBB - Simple Content Management for phpBB

Post by Ger » Wed Jan 18, 2017 8:18 am

Peter77sx wrote:
Wed Jan 18, 2017 3:48 am
Couple things I found-
1) When Enable URL Rewriting: in server settigns is checked, user avatar on articles/news show broken.


2)Left side bar link to articles/forum post show a double slash after forum directory so site.com/phpbb3//viewtopic.php=etc URL rewriting or not.

probably not a big deal but whatever, thought I'd mention it. :)


Articles seem to be social sharing friendly out-of-the-box... congrats, well done.
Thanks for reporting this. I have fixed both issues in the master branche on Github.
My extensions:
Simple CMS, Feed post bot, Avatar Resize, Modbreak, Magic OGP, Live topic update, Modern Quote, Quoted Where (GDPR) and Autoresponder.
Newest: FAQ manager for 3.2

Like my work? Buy me a coffee to keep it coming. :ugeek:
-Available for custom work-

MaFeSa
Registered User
Posts: 174
Joined: Wed Feb 11, 2009 7:48 am

Re: [BETA][3.2] cmBB - Simple Content Management for phpBB

Post by MaFeSa » Wed Jan 18, 2017 5:52 pm

Ger wrote:
Wed Jan 18, 2017 7:58 am
Does this solve it for you? If so, I will fix this in the extension.
Yess :D
Thanks Ger ;)

User avatar
Ger
Recognised Extension Developer
Posts: 1845
Joined: Wed Jan 02, 2008 7:35 pm
Location: 192.168.1.100
Contact:

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by Ger » Thu Feb 02, 2017 11:06 am

I upped the version to RC. Submission to the CDB is pending for this issue
My extensions:
Simple CMS, Feed post bot, Avatar Resize, Modbreak, Magic OGP, Live topic update, Modern Quote, Quoted Where (GDPR) and Autoresponder.
Newest: FAQ manager for 3.2

Like my work? Buy me a coffee to keep it coming. :ugeek:
-Available for custom work-

rec0d
Registered User
Posts: 95
Joined: Tue Jan 03, 2017 11:49 am

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by rec0d » Thu Feb 23, 2017 8:44 am

i have two problems. how to remove left side bar for users? and i want the url to be /news insted of article/news

User avatar
Ger
Recognised Extension Developer
Posts: 1845
Joined: Wed Jan 02, 2008 7:35 pm
Location: 192.168.1.100
Contact:

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by Ger » Thu Feb 23, 2017 9:33 am

Why exactly do you want to remove the left sidebar for users? The button for a new article or to edit an article is only visible if you give a user permission to post articles. This is default, but you can simply change this through the permissions tab in the ACP.

If you want to disable the entire sidebar, you should change the file ./ext/ger/cmbb/styles/all/template/event/overall_header_content_before.html to your liking.
Bear in mind that this also holds a loginbox, latest topics, and statistics, as well as the placeholder to append the rightbar in responsive mode if you have that enabled.
rec0d wrote:
Thu Feb 23, 2017 8:44 am
and i want the url to be /news insted of article/news
This is not supported at the moment. You could however add this to the ./ext/ger/cmbb/config/routing.yml:

Code: Select all

ger_cmbb_news:
    path: /news
    defaults: { _controller: ger.cmbb.viewer:handle, alias: "news" }
My extensions:
Simple CMS, Feed post bot, Avatar Resize, Modbreak, Magic OGP, Live topic update, Modern Quote, Quoted Where (GDPR) and Autoresponder.
Newest: FAQ manager for 3.2

Like my work? Buy me a coffee to keep it coming. :ugeek:
-Available for custom work-

rec0d
Registered User
Posts: 95
Joined: Tue Jan 03, 2017 11:49 am

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by rec0d » Thu Feb 23, 2017 5:07 pm

I wanted to remove side menu as i have a custom design and a custom login system. thanks for your answer.
by remove it where can admins edit and create new articles?

User avatar
nero
Registered User
Posts: 321
Joined: Fri Mar 13, 2015 8:18 am

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by nero » Thu Feb 23, 2017 5:12 pm

[Thu Feb 23 12:11:18.463222 2017] [:error] [pid 26669] [client 108.162.241.48:22800] PHP Catchable fatal error: Argument 2 passed to phpbb\\controller\\helper::message() must be of the type array, string given, called in /var/www/travincial.com/ext/ger/cmbb/controller/save.php on line 134 and defined in /var/www/travincial.com/phpbb/controller/helper.php on line 142, referer: http://www.travincial.com/article_edit/_new_

rec0d
Registered User
Posts: 95
Joined: Tue Jan 03, 2017 11:49 am

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by rec0d » Thu Feb 23, 2017 5:19 pm

should i remove

Code: Select all

ger_cmbb_article:
    path: /article/{alias}
    defaults: { _controller: ger.cmbb.viewer:handle, alias: "index" }
or just add your code inside?

User avatar
Ger
Recognised Extension Developer
Posts: 1845
Joined: Wed Jan 02, 2008 7:35 pm
Location: 192.168.1.100
Contact:

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by Ger » Fri Feb 24, 2017 7:33 am

rec0d wrote:
Thu Feb 23, 2017 5:07 pm
I wanted to remove side menu as i have a custom design and a custom login system. thanks for your answer.
by remove it where can admins edit and create new articles?
You should ideally just add your own template files. I provided some basic templating that can be used, but you can create a folder ./ext/ger/cmbb/styles/YOURSTYLE/template
and copy any files you want changed there from the ./ext/ger/cmbb/styles/all/template dir. Change them at will and save them with the same filename. Any files in your specific template dir wil take precedence above the files in the all dir.

About the buttons: you should make sure you have this part somewhere in your code:

Code: Select all

				<!-- IF S_CAN_EDIT -->
					<p><a href="{U_NEW_ARTICLE}" class="button" alt="{L_NEW_ARTICLE}"><span>{L_NEW_ARTICLE}</span> <i class="icon fa-asterisk fa-fw" aria-hidden="true"></i></a></p>
					<!-- IF U_EDIT_ARTICLE --> 
						<p><a href="{U_EDIT_ARTICLE}" class="button" alt="{L_EDIT_ARTICLE}"><span>{L_EDIT_ARTICLE}</span> <i class="icon fa-pencil fa-fw" aria-hidden="true"></i></a></p>
					<!-- ENDIF -->
				<!-- ENDIF -->
				<!-- IF S_CAN_SEE_HIDDEN -->
					<!-- IF S_HIDDEN -->
						<p><a href="index?showhidden=1" class="button" alt="{L_SHOW_HIDDEN}"><span>{L_SHOW_HIDDEN}</span> <i class="icon fa-recycle fa-fw icon-green" aria-hidden="true"></i></a></p>
					<!-- ELSE -->
						<p><span class="button"><span>{L_NO_HIDDEN}</span> <i class="icon fa-recycle fa-fw icon-gray" aria-hidden="true"></i></span></p>
					<!-- ENDIF -->
				<!-- ENDIF -->
You can of course change the markup, but this holds the logic about whether or not to show them (based on permissions), their titles and their URL.
rec0d wrote:
Thu Feb 23, 2017 5:19 pm
should i remove

Code: Select all

ger_cmbb_article:
    path: /article/{alias}
    defaults: { _controller: ger.cmbb.viewer:handle, alias: "index" }
or just add your code inside?
You should just add my code
My extensions:
Simple CMS, Feed post bot, Avatar Resize, Modbreak, Magic OGP, Live topic update, Modern Quote, Quoted Where (GDPR) and Autoresponder.
Newest: FAQ manager for 3.2

Like my work? Buy me a coffee to keep it coming. :ugeek:
-Available for custom work-

User avatar
Ger
Recognised Extension Developer
Posts: 1845
Joined: Wed Jan 02, 2008 7:35 pm
Location: 192.168.1.100
Contact:

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by Ger » Fri Feb 24, 2017 7:55 am

nero wrote:
Thu Feb 23, 2017 5:12 pm
[Thu Feb 23 12:11:18.463222 2017] [:error] [pid 26669] [client 108.162.241.48:22800] PHP Catchable fatal error: Argument 2 passed to phpbb\\controller\\helper::message() must be of the type array, string given, called in /var/www/travincial.com/ext/ger/cmbb/controller/save.php on line 134 and defined in /var/www/travincial.com/phpbb/controller/helper.php on line 142, referer: http://www.travincial.com/article_edit/_new_
Good catch! :)
It only occurred if you didn't provide a title, had missed it earlier. Now fixed on Github!
My extensions:
Simple CMS, Feed post bot, Avatar Resize, Modbreak, Magic OGP, Live topic update, Modern Quote, Quoted Where (GDPR) and Autoresponder.
Newest: FAQ manager for 3.2

Like my work? Buy me a coffee to keep it coming. :ugeek:
-Available for custom work-

rec0d
Registered User
Posts: 95
Joined: Tue Jan 03, 2017 11:49 am

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by rec0d » Fri Feb 24, 2017 11:11 am

Thanks for the upodate

rec0d
Registered User
Posts: 95
Joined: Tue Jan 03, 2017 11:49 am

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by rec0d » Fri Feb 24, 2017 11:18 am

when create a new article this error appear: The provided title is not allowed.

i tryed used name test as a article but if i change that into test test. think you need add a value for one string. right now you need to have it longer then one words.

rec0d
Registered User
Posts: 95
Joined: Tue Jan 03, 2017 11:49 am

Re: [3.2][RC] cmBB - Simple Content Management for phpBB

Post by rec0d » Fri Feb 24, 2017 11:22 am

and read more gets this error : No route found for "GET /test-test" (from "http://127.0.0.1/news")

when press on user avatar it gives the same error. No route found for "GET /test-test" (from "http://127.0.0.1/news")

Locked

Return to “Extensions in Development”