This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[13:00:04] <imkingdavid> alright, meeting starts now. First item on the agenda is the status of events since last week's meeting. | |
[13:00:21] <nickvergessen> I updated some of the events with docs | |
[13:00:32] <nickvergessen> but need to finish the rest | |
[13:00:58] <imkingdavid> alright | |
[13:01:04] <nickvergessen> progress is here https://github.com/phpbb/phpbb3/compare/develop...nickvergessen:feature/php-events-4 | |
[13:01:30] <nickvergessen> will try to fix the rest over the weekend | |
[13:01:39] <imkingdavid> alright | |
[13:01:40] <nickvergessen> so that oleg or anyone else can merge it | |
[13:01:57] <imkingdavid> does anyone have questions about events? | |
[13:02:04] <nickvergessen> then next step is to document the stuff correctly |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[17:37:26] <nn-> imkingdavid: i am aware of that but i believe what i said remains true | |
[17:37:26] <igorw> nn-: please reply to event dispatcher PR | |
[17:37:34] <igorw> about PHPBB_NO_AUTOLOAD naming | |
[17:37:51] <nn-> do you have some issue with keeping both constants at top level? | |
[17:38:10] <nn-> i'm probably ok with renaming to no composer autoload | |
[17:39:09] --> unknownbliss1 (~unknownbl@phpbb/website/Unknown-Bliss) has joined #phpBB-dev | |
[17:39:09] *** Mode #phpBB-dev +o unknownbliss1 by ChanServ | |
[17:39:16] <imkingdavid> nn-: the first reason to hit the button is met by the second reason. So what is the issue with using the patch as is, or even with keeping the "Full Editor" text but making the button go to the preview page? | |
[17:39:39] <nn-> which reason is first again? | |
[17:39:58] <imkingdavid> first: getting to posting screen, second: preview what you have written, imo? |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
<?php | |
/** | |
* | |
* @package testing | |
* @copyright (c) 2011 phpBB Group | |
* @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2 | |
* | |
*/ | |
/** |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[20:23] == imkingdavid [ad4242aa@phpbb/developer/imkingdavid] has joined #phpBB-dev | |
[20:23] == mode/#phpBB-dev [+o imkingdavid] by ChanServ | |
[20:25] <@imkingdavid> can another developer review https://github.com/phpbb/phpbb3/pull/545 ? it's pretty much ready (I tested it some time ago and it worked) but I'd like a second opinion before I merge it | |
[20:30] == nw- [~pie@72.245.42.19] has joined #phpBB-dev | |
[20:48] <nw-> imkingdavid: rfc for http://tracker.phpbb.com/browse/PHPBB3-10576 ? | |
[20:51] <nw-> we can have a prosilver/lang/en.php.sample possibly regarding comments in https://github.com/phpbb/phpbb3/pull/530 | |
[20:53] == Marshalrusty [~Marshalru@phpbb/manager/pdpc.21for7.marshalrusty] has quit [Disconnected by services] | |
[20:53] == YuriyRusko [~Marshalru@phpbb/manager/pdpc.21for7.marshalrusty] has joined #phpBB-dev | |
[20:53] == mode/#phpBB-dev [+o YuriyRusko] by ChanServ | |
[20:54] <nw-> &$lang = array() - that looks very suspicious |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[11:16] <@imkingdavid> naderman: to use templates in extensions, is there a $template->set_extension_filenames or something? How does it know to look in the ext/x/styles/prosilver/template/ folder rather than the main styles/prosilver/template folder? | |
[11:19] <@imkingdavid> should I be using $template->set_ext_dir_prefix($phpbb_root_path . 'ext/x/'); ? | |
[11:25] <@nn-> imkingdavid i don't know if that has been implemented yet | |
[11:26] <@imkingdavid> nn-: lovely | |
[11:26] <@imkingdavid> nn-: what about adding language from the ext/x/language/en/ directory? $user->add_lang_ext() ? or is that for something else? | |
[11:26] <@nn-> isn't it one of those things that i asked for a decision on? | |
[11:26] <@imkingdavid> ? | |
[11:27] * imkingdavid will have a hard time making a good example extension without such things | |
[11:27] <@nn-> https://github.com/p/phpbb3-ext-overall-header-addition/tree/master/ext/phpbb3-ext-overall-header-addition/styles | |
[11:27] <@nn-> note "universal" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
static function get_blog_data($blog_id) | |
{ | |
global $db; | |
if(!$blog_id) | |
{ | |
return false; | |
} | |
$sql_ary = array( | |
'SELECT' => 'b.*,c.*,u.*', | |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[16:21] <imkingdavid> http://tracker.phpbb.com/browse/PHPBB3-10539 | |
[16:21] <imkingdavid> odd | |
[16:22] <imkingdavid> no clue why it only happens sometimes | |
[16:22] <imkingdavid> but it's consistent when it does happen | |
[16:22] <unknownbliss> its bizarre | |
[16:22] <unknownbliss> indeed | |
[16:23] <imkingdavid> maybe something to do with the reverse loading of topics https://github.com/phpbb/phpbb3/blob/develop/phpBB/viewforum.php#L444 | |
[16:23] <imkingdavid> but i haven't had time to look too far into it | |
[16:23] <imkingdavid> since I can't get it to happen locally | |
[16:23] <unknownbliss> i got it to happen locally |
NewerOlder