Development Discussion Board

Should be ready for testing now.

3. You removed this text from documentation strings: "If set to 0 the message is sent immediately and will not be queued for later sending." I'm assuming you did not actually change this behavior, the note that package size of 0 results in immediate mail submission needs to go back in the...

I introduced two configs now. Allowing the mulitple/single appearance of users, as per bantu's post: We have to think about what happens if a user is in multiple groups. It should probably be selectable if the users should show in all groups or only in the highest in the hierarchy. Allowing to view ...

I made some code changes, to view the users by groups. teampage.png I think this makes a lot more sense than the current solution and should als peform a lot better. We've had problems with query performance on the team page quite a few times. If you could create a ticket on the tracker and submit a...

Small note: class_exists and require are more memory efficient than require_once. This is certainly interesting, do you have a url for further reading? Pretty much the same what I said about the include_once you used: http://area51.phpbb.com/phpBB/viewtopic.php?p=207366#p207366 php is faster in loo...

ToonArmy wrote:It's very simple, if new/includes/db_tools.php exists in the updater package, use that.

We also do this many other times, f.e. the language/en/install, the includes/diff/ adm/style/install_* etc.

I'd like bantu's suggestion. For the "Display user in multiple groups" I'd add a simple bool-config.
I think that would be not to hard to write and have lot more functionality than the current thing.

I'd just kick the current behaviour and start new.

Add an option to the groups and then do some kind of mix between the current thing (viewing the rank, primary group and the moderating forums (if any)) and
memberlist.php?mode=group&g=1
only with multiple groups.

Option 1: http://www.phpbb.com/community/viewtopic.php?f=69&t=1020905 The MOD does nearly the same thing I'm suggesting here, it just uses a hackish way to do so ;) Conversion from existing utc + dst offsets to timezone name is not unique. altough this would only help on little percentage of th...

!!value is a pretty common idiom for converting a value to boolean. It has the property of working (and making sense) on all types. What do you suggest instead? (bool) ? Are you really that unsure of operator precedence? I don't believe any self-respecting developer can claim that arithmetic operat...