YesNoOk
  1. Welcome!
    EosAlpha BBS is a fork of the popular Simple Machines Forum software. We aim at creating a new forum software, adding several new features and a modern and fresh design on top of the existing SMF code base.

    This software is currently in an early stage of development and this forum acts primarily as a testing platform for the ongoing development.

    Feel free to look around to get an idea about how it feels and looks.
avatar

Bug minor things (Read 16461 times)

Started by Valodim, July 27, 2012, 11:26:30

0 Members and 1 Guest are viewing this.
Share this topic:
#1
minor things |
July 27, 2012, 11:26:30
I tend to find very minor things here and there lately, making this topic so I have some place to note them down and not just forget them later.

Right now I only got the forum search, which says "search in this topic only" in the messagindex.

I'm also getting undefineds from the sidebar template regarding openid. did you drop openid? grep gives a lot of leftovers in the sources.

     Posted: July 27, 2012, 11:51:41
there is an onHover thing in topic_js.js which sets opacity = 1.0 on mouseover in the buttos bar but nothing else. that kinda thing should be done in the css as .post_bottom:hover.

     Posted: July 27, 2012, 11:58:17
the "Unapprove" icon is shown for moderators even when post moderation is disabled

     Posted: July 27, 2012, 13:16:41
http://forum.miranda.or.at/Themes/default/images/collapse.gif
http://forum.miranda.or.at/Themes/default/images/expand.gif

both missing on http://forum.miranda.or.at/stats
#2
Re: minor things |
July 30, 2012, 08:11:44
Regarding onHover, I haven't seen the code (I try to keep my hands off any other fork really ;), a question of principle until Wedge is out at least), but I'd venture into saying that it's Nightwish's way of providing support for IE6, because it only supports :hover on anchor tags.

An alternative way of doing it, and that's how I'm doing it in Wedge, is including an external script to support :hover on any tag. The one I'm using, and it's working nicely, is written by Gilmore Davidson and released under the MIT license.
__
  Wedge
« Everyone knows rock attained perfection in 1974.
It's a scientific fact. » (Homer Simpson)
#3
Re: minor things |
July 30, 2012, 08:17:09
Regarding onHover, I haven't seen the code (I try to keep my hands off any other fork really ;), a question of principle until Wedge is out at least), but I'd venture into saying that it's Nightwish's way of providing support for IE6, because it only supports :hover on anchor tags.
Good point if I were actually supporting IE6, which I don't :)

Honestly, cannot remember why I did this with JavaScript, there probably is (or was) a reason, but the reason does no longer exist. Most likely, this particular code fragment was just forgotten.

__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#4
Re: minor things |
July 31, 2012, 01:48:47
The "when possible, merge..." text should be a <label> so when you click it the checkbox is toggled.

     Posted: July 31, 2012, 12:32:57
posts have no min-height attribute, which makes one-line posts have a bad post-to-signature ratio

     Posted: August 01, 2012, 23:38:20
2012-08-02 01:27:46: (mod_fastcgi.c.2701) FastCGI-stderr: PHP Fatal error:  require_once(): Failed opening required '/var/www/mugenguild.com/forum/Sources/Subs-Compat.php' (include_path='.:/usr/share/php:/usr/share/pear') in /var/www/mugenguild.com/forum/Sources/LogInOut.php on line 346

you missed a spot there

     Posted: August 01, 2012, 23:43:48
pretty urls only work if the separating character is -, if it's _ (which was the default for me) the topic id will still be delimited by a - which won't be recognized

     Posted: August 01, 2012, 23:45:26
btw don't mind my members too much, they can be a mean bunch and they're used to the theme we had for half a decade ;)
#5
Re: minor things |
August 02, 2012, 05:29:57
btw don't mind my members too much, they can be a mean bunch and they're used to the theme we had for half a decade ;)
Change is not always welcome, especially when the look changes so drastically. Remember the various sh1tstorms when big sites like google, youtube or facebook made changes :)

Not that bad, they seem to like most things and only complain about the theme being "too white" (which I pretty much expected anyway).
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#6
Re: minor things |
August 02, 2012, 12:32:17
I see you're working on the white and font complaints. ;) Reply textarea is monospace right now?
#7
Re: minor things |
August 02, 2012, 12:54:21
I see you're working on the white and font complaints. ;) Reply textarea is monospace right now?
It should be consolas (if available), monospace otherwise.

BTW: I'm probably dropping the index_default.css and make index_lightweight.css the new default. Looks better anyway...

BTW2: Seems like many xml responses are broken over there (topic preview, member cards and such) and it's now throwing js errors (timeOffsetMember). base.tpl was changed a bit and needs to be updated in the custom skin.

Also, I'm going to remove all closing tags in php source files - that's what is recommended anyway by Zend, because empty lines after closing tags can disturb xml reponses (if an xml response doesn't start with <?xml in the very first line, it's invalid). That's probably the source of the problems with topic previews...
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#8
Re: minor things |
August 02, 2012, 13:43:16
yeah. I'm using vim to edit files which always adds a \n to the end. and that usually makes sense, too, if not for php's quirky behavior :P

     Posted: August 02, 2012, 13:53:29
monospace in post editor but non-monospace in regular post view is a bad idea, because people will align things or do tables

like this
       ^

or put code and expect it to look ok in the post as well
#9
Re: minor things |
August 02, 2012, 13:55:41
yeah. I'm using vim to edit files which always adds a \n to the end. and that usually makes sense, too, if not for php's quirky behavior :P
Killing all end tags should solve this problems...

Funny to read that Zend's official recommendations *forbids* end tags in files that only contain code and are not supposed to output anything (like most of our stuff in Sources/).

I still believe that this is a major bug (or design flaw) in php itself.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#10
Re: minor things |
August 03, 2012, 00:19:41
Ok it looks like I'll have to do an alternate colorscheme over there, people apparently don't appreciate the professional style too much. And the have a point, it's a gaming forum, it shouldn't look "sterile". I still like the professional thing and the brighter colors were fine, don't let those guys influence you too much for your own design.

The "merge post with last one" option doesn't show up in my place, and I couldn't find an option for that. I'm probably just missing it?

Oh one more thing, inline-modifying and quick-replying should be mutually exclusive in thread view, and when modifying it would be neat if the quote button added quotes inline.
#11
Re: minor things |
August 03, 2012, 09:14:54
Ok it looks like I'll have to do an alternate colorscheme over there, people apparently don't appreciate the professional style too much. And the have a point, it's a gaming forum, it shouldn't look "sterile". I still like the professional thing and the brighter colors were fine, don't let those guys influence you too much for your own design.
Nah, I'm not going to change this style much, just adding a couple of things like alternating row classes and such. The basic color scheme will stay.
Quote
The "merge post with last one" option doesn't show up in my place, and I couldn't find an option for that. I'm probably just missing it?
This is a "per board" option, you'll find it when creating or modifying a board.[/quote]
Quote
Oh one more thing, inline-modifying and quick-replying should be mutually exclusive in thread view, and when modifying it would be neat if the quote button added quotes inline.
Yup, there should be some locking.

BTW: there is now a way to easily override css. In theme_support.php, you can define a css file name (one per theme variant), put that file in the theme's css folder and it will be loaded *after* the main style sheet.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#12
Re: minor things |
August 04, 2012, 11:58:16
you probably already noticed, but you used "on" instead of "bind" in bbc_refresh, which breaks spoilers and other things :)

Quote
BTW: there is now a way to easily override css. In theme_support.php, you can define a css file name (one per theme variant), put that file in the theme's css folder and it will be loaded *after* the main style sheet.

yeah I hope to get there some time, right now I need a couple of template changes as well. ah well, diff-updating is fairly painless at this point :)

     Posted: August 04, 2012, 12:14:44
fetch a preview when user clicks "go advanced"?
#13
Re: minor things |
August 04, 2012, 12:33:45
you probably already noticed, but you used "on" instead of "bind" in bbc_refresh, which breaks spoilers and other things :)
Ups... because the bundled jQuery is still an older one. You can enable the option to load jQuery from Google's CDN which has already been told to fetch 1.7.2. It will also save you some traffic :)

The bundled was still at 1.6, probably forgot to update it.
Quote
Quote
BTW: there is now a way to easily override css. In theme_support.php, you can define a css file name (one per theme variant), put that file in the theme's css folder and it will be loaded *after* the main style sheet.

yeah I hope to get there some time, right now I need a couple of template changes as well. ah well, diff-updating is fairly painless at this point :)
You could use template overrides (should already work).. See Themes/default/tpl/overrides/readme, so in theory, you could simply use the default theme and customize it via css- and template overrides. Depending on the scale of template changes, merging could still be a pain, I know.

Also, some heads up because of this, though it should not break things. Unless you explicitly enable this in Membergroups->Settings, it will dynamically build an inline style sheet using the old colors.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#14
Re: minor things |
August 04, 2012, 13:02:01
it would, if the styles were set as "a.member .group_1" instead of "a.member group_1" ;)
actually it doesn't that way either :P
 

      Posted: August 04, 2012, 13:02:58
oh and I already use template overrides, that works just fine
Last Edit: August 04, 2012, 13:12:14 by Valodim
#15
Re: minor things |
August 04, 2012, 14:11:26
it would, if the styles were set as "a.member .group_1" instead of "a.member group_1" ;)
actually it doesn't that way either :P
The inline styles are regenerated when you modify a member group in the admin panel (for a fresh install, they would be pre-populated in the db).

(Yes, I know, a updater script would be awesome :) )
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#16
Re: minor things |
August 04, 2012, 14:17:28
suggestion for short user agent: shorten_subject(substr($url['USER_AGENT'], 0, strpos($url['USER_AGENT'], ' ')), 25) that will shorten to something like "Mozilla/5.0" or "Opera/9.8"

About the group colors, I actually meant that the css identifier is wrong, it targets the element group_1 instead of class .group_1

     Posted: August 04, 2012, 14:33:01
here's a bit for your wiki, I just used the second query to convert the msn profile field. first is just for reference.

Code:
INSERT INTO `yabbse_custom_fields` (`col_name`, `field_name`, `field_desc`, `field_type`, `field_length`, `field_options`, `mask`, `show_reg`, `show_display`, `show_profile`, `private`, `active`, `bbc`, `can_search`, `default_value`, `enclose`, `placement`) VALUES ('cust_msn','MSN','MSN Messenger address','text',255,'','email',0,1,'forumprofile',0,1,0,1,'','<a href=\"msnim:chat?contact={INPUT}\"><img src=\"{IMAGES_URL}/msn.png\" alt=\"{INPUT}\" /></a>',1);

REPLACE INTO yabbse_themes (id_member, id_theme, variable, value) SELECT id_member, 1, 'cust_msn', msn FROM yabbse_members WHERE msn IS NOT NULL AND msn LIKE '%@%' ;
#17
Re: minor things |
August 04, 2012, 14:52:03
suggestion for short user agent: shorten_subject(substr($url['USER_AGENT'], 0, strpos($url['USER_AGENT'], ' ')), 25) that will shorten to something like "Mozilla/5.0" or "Opera/9.8"
Done (and when you click on the shortened version, you see the full one).
Quote
About the group colors, I actually meant that the css identifier is wrong, it targets the element group_1 instead of class .group_1
Will be in the next commit.
Quote
here's a bit for your wiki, I just used the second query to convert the msn profile field. first is just for reference.
Thanks. Guess that would also work for the other removed profile fields...
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#18
Re: minor things |
August 05, 2012, 00:56:40
love the "new" indicator.

huge timestamps right now with GMT+2 stuff, probably just a temporary state during development~

do you have plans to show more events in a user's activity stream? most significantly, topic reply notifications?
#19
Re: minor things |
August 05, 2012, 01:07:25
huge timestamps right now with GMT+2 stuff, probably just a temporary state during development~
Screenshots, browser version?
It's now using toLocaleTimeString() method and I'm seeing normal timestamps over there...
Quote
do you have plans to show more events in a user's activity stream? most significantly, topic reply notifications?
Yup, activity stream can be used for all kind of things, you already can get notifications when someone posts a reply (but by default, this kind of notification is disabled). Also, members can opt-out from creating activities (I think, that's a privacy thing and should be allowed).

More interesting would be a 'your message has been quoted' notification...

BTW: There is one important fix in one of the last commits to load theme_support.php from the default theme if the custom theme doesn't have one.

And I still get xml responses with empty lines at the beginning - weird, ob_sessrewrite() now does a ltrim() on the buffer - have tested it here on my server and it works, at least when I deliberately insert blank lines in the xml templates.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#20
Re: minor things |
August 05, 2012, 09:17:22
I use luakit so it's libwebkit rendering, here's a shot.

the newline thing got me stumped as well :\

     Posted: August 05, 2012, 10:57:49
the pm layout gets really cramped at 1024 pixels width, which is a resolution that should still be supported. maybe put the messages on the full screen width below the menu and pm list?

I didn't give it a shot yet because the change is non-trivial with how the template is stacked right now (and it's going to be converted soon anyways I guess)
#21
Re: minor things |
August 05, 2012, 11:48:38
I use luakit so it's libwebkit rendering, here's a shot.
What I expected - it's browser specific what toLocaleTimeString() returns. Guess, I've to find another way to get a locale-aware time format (not so easy in JavaScript).

      Posted: August 05, 2012, 10:57:49
Quote
the pm layout gets really cramped at 1024 pixels width, which is a resolution that should still be supported. maybe put the messages on the full screen width below the menu and pm list?

I didn't give it a shot yet because the change is non-trivial with how the template is stacked right now (and it's going to be converted soon anyways I guess)
PMs will just get a different postbit layout anyway (one without a wide left column, because it's the left column that eats too much space.

BTW: This fix in Load.php is important to avoid tons of warnings about undefined in the log when using a custom theme.

     Posted: August 05, 2012, 14:06:50
Ok...

New time formatting code should be in here...

It should now recognize the member's PHP-style time format (set in the profile) and output either 24hr or 12hr AM/PM time stamps without seconds or time zone information.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#22
Re: minor things |
August 05, 2012, 14:11:59
yup, looks good now. nice work.

     Posted: August 05, 2012, 16:20:44
found the cause for the broken preview - my report borad modification added an empty line after the ?> in Error.english.php :)

also, for the record, php ignores a newline after ?> at the end of files automagically. that's not really consistent behavior, but it allowed xml to work even when there were all those ?>s around

the description for merging of posts in the admin panel is wrong: it says the number of minutes determines the maximum age of the last post for the merge. in the implementation, it's actually the maximum age for the last post to perform merging whether the user checked the box or not. both ways make sense I guess, but they should be coherent ;)
#23
Re: minor things |
August 05, 2012, 16:34:30
found the cause for the broken preview - my report borad modification added an empty line after the ?> in Error.english.php :)
also, for the record, php ignores a newline after ?> at the end of files automagically. that's not really consistent behavior, but it allowed xml to work even when there were all those ?>s around
For some reason, it doesn't always work - sometimes, the new lines sneak into the output and ruin xml responses. This is a mess and the worst thing is that these empty lines sneak in before the output buffer is sent, so trimming the buffer doesn't help either.
Quote
the description for merging of posts in the admin panel is wrong: it says the number of minutes determines the maximum age of the last post for the merge. in the implementation, it's actually the maximum age for the last post to perform merging whether the user checked the box or not. both ways make sense I guess, but they should be coherent ;)
There should be a permission to prevent users from un-checking the merge box and force them into merging.

BTW:

https:/..../forum/Themes/mfg/css/index_lightweight.css?v=1572

returns 404?

mfg theme broken for me...
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#24
Re: minor things |
August 05, 2012, 16:43:30
eh, that one is laziness on my part, I basically did mv index_lightweight.css index_default.css to make lightweight the default (and only) variant :P
Last Edit: August 06, 2012, 22:08:15 by Valodim
#25
Re: minor things |
August 06, 2012, 22:08:08
PersonalMessages.php line 710, that one is a huge bug because notifications link to pms which are not the head of a conversation. Also, why shouldn't a user be allowed to see a conversation if the selected pm is not the head of it?
#26
Re: minor things |
August 06, 2012, 22:17:24
PersonalMessages.php line 710, that one is a huge bug because notifications link to pms which are not the head of a conversation. Also, why shouldn't a user be allowed to see a conversation if the selected pm is not the head of it?
Need to investigate this, there might very well be a good reason for this code. This code is in SMF 2.0 and 2.1 (development branch) as well, but I can see how it conflicts with the notifications.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#27
Re: minor things |
August 06, 2012, 22:42:40
The query above it looks fine to me. It does a GROUP BY pm.id_pm_head and only selects this id_pm_head and the MAX(pm.id_pm), which means it is not relevant which pm is selected out of all which have the same id_pm_head.

The INNER JOIN also has a check for the correct member id no matter what, so it's not possible to get other users' pms in there.

I don't think that check is necessary.

      Posted: August 06, 2012, 23:25:36
the facebook like button should not require an app id anymore since version 1.2, an url should be sufficient.
Last Edit: August 06, 2012, 23:26:04 by Valodim
#28
Re: minor things |
August 07, 2012, 08:30:48
I don't think that check is necessary.
Still not 100% sure, but you're probably right. I tried to break in with all kind of scary pm requests - no success even with admin privileges. What bothers me a bit is that this code was explicitly written for the conversation view, so it's not a leftover from old code.

About today's commit(s): Lots of stuff again, mostly in the pm system. Template conversion goes fine and there are only 2 really big ones left - the profile stuff and the post.template (+a couple of smaller ones).

You may want to take care when merging Subs-Editor.php because of the new buttons.


     Posted: August 10, 2012, 18:36:33
Some heads for upcoming changes...

I've started splitting (and somewhat cleaning up) the main style sheets. To make things easier for theme designers, the main css will be split into two files:
  • base.css - contains only layout-related things, like margins, positions, list layouts and similar.
  • skin_themevariantname.css - contains all the fonts, colors, gradients, images and such.
The base.css is loaded first, so the skin stylesheet can override things in the base if needed. This should it make easier to create color variants without affecting the layout too much and will support the concept of theme variants (as defined in theme_support.php) which act as sub-styles of the current theme.

Basically, you could leave everything in base.css and create a empty skin_default.css (or do it vice versa and keep everything in skin_default.css and create an empty base.css), if you have already customized the css too much.

Other things implemented
  • Auto-sizing textareas for quick reply and inline-modify. It will initially start small and automatically expand or shrink wile typing a message.
  • New profile option for setting the initial size of the full post editor that replaces the stupid resizing bar which didn't work on all browsers and did not remember its setting.

This stuff should be committed later this day (or maybe on the weekend).
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
1 Useful 
Last Edit: August 10, 2012, 18:36:33 by Nightwish
#29
Re: minor things |
August 11, 2012, 21:17:01
fyi: not slacking off, but I need to prepare a presentation for uni for monday that's why I'm MIA right now.

Restructuring the stylesheets is a great idea. While you're doing that, have you ever looked at SASS? Whole new dimension of stylesheets :)

     Posted: August 11, 2012, 21:43:24
quickreply and edit should have a bigger min-height
#30
Re: minor things |
August 11, 2012, 23:57:28
fyi: not slacking off, but I need to prepare a presentation for uni for monday that's why I'm MIA right now.
Yeah, have read it over there...
Quote
Restructuring the stylesheets is a great idea. While you're doing that, have you ever looked at SASS? Whole new dimension of stylesheets :)
Yup and I *might* be using that at some point, just not now.
Quote
quickreply and edit should have a bigger min-height
They should now auto-expand (and shrink) as needed and only start with a small height (if not, force-reloading might help, you may still have the old scripts in the cache).
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#31
Re: minor things |
August 17, 2012, 11:26:58
alright, I cleaned up my css and all changes are now in an overriding css file while leaving the others untouched. much better now. also incorporated color changes by popular demand :P

Still in the process of cleaning up my working tree and moving mods into plugins. That really is a lot cleaner than the direct software hacks :)
#32
Re: minor things |
August 17, 2012, 13:37:21
Ok fine...

Just some notes about the changes from the last couple of days. Nothing ground-breaking, just template stuff mainly. PM templates are done, profile templates about 90% done.

Plans for the near future: finish the template stuff, then get the mobile theme into a usable state (the first version will probably be very basic and bare-bone, supporting only the most important pages - board index, message index and topic display).

     Posted: August 18, 2012, 17:19:43
About today's commit:

Make sure Profile.template.php does NOT get removed from Themes/default as it's still needed in Register(). I already moved it to Deprecated/, but this was in error. The last commit should revert this anyway, just to make sure...
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#33
Re: minor things |
August 19, 2012, 16:43:36
Today, I've added SphinxQL SearchAPI support. This commit shouldn't conflict with anything search-related, but some testing is needed before I can deprecate the old Sphinx search support which is really no longer needed, because SphinxQL was implemented around version 1.0 quite some time ago.

SphinxQL should improve search performance even further and also fix the bugs with some result sorting methods not being available with the old Sphinx API.

SphinxQL requires a minimal change to the existing sphinx.conf. The port for incoming SphinxQL queries must be specified and must be different from the standard port.
Code: (text)
searchd
{
        listen = 3312
        listen = 3313:mysql41   << this one
        log = /var/sphinx/log/searchd.log
        query_log = /var/sphinx/log/query.log
        read_timeout = 5
        max_children = 30
        pid_file = /var/sphinx/data/searchd.pid
        max_matches = 1000
}

Sphinx settings were overhauled as well, all strings should now be translatable and the "Test connection" button should also perform a test for SphinxQL connections.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#34
Re: minor things |
August 24, 2012, 19:07:23
sadly, I can't update my sphinxsearch. it requires a newer libc, which is a showstopper :(

     Posted: August 24, 2012, 22:01:42
the sphinx guys had a package with a newer version, but compiled for older ubuntu! all working now, yay :)
#35
Re: minor things |
August 25, 2012, 13:25:42
Problem with entering text in Cyrillic - for exapmle when i try to add Topic prefix or Post ratings. In Latin letters are okay
#36
Re: minor things |
August 26, 2012, 13:49:31
Problem with entering text in Cyrillic - for exapmle when i try to add Topic prefix or Post ratings. In Latin letters are okay
Should be fixed with the last commit from today, but please note that the admin UI for both features (topic prefixes and ratings) is very basic at the moment and some features are missing.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#37
Re: minor things |
August 30, 2012, 18:51:10
Incoming changes:
  • Some styling changes in the default theme (there is now only one theme_variant - default).
  • Topic bans. Moderators can now ban members from specific topics only without changing their permissions. A member who has been banned from a topic can not participate in it (no post/reply/modify), but can still read it normally. Topic bans can expire automatically after x days (done in daily maintenance) or be permanent until lifted by a moderator. A member who has been banned from a topic will see a notice explaining it.

    Right now, the UI is sparse - moderators will see a new button next to the warning/report button in the bottom left corner of a post. This will lead to the moderation center where a ban can be set or a existing ban can be lifted.

    Viewing existing topic bans by topic or member will be added to the moderation center soon and the topic ban system will be integrated into the warning system allowing a moderator who can issue warnings to set a topic ban together with a warning.

    This requires a db scheme change with the new table topicbans.

Edit:

@Valodim: Does the SphinxQL work over there? Because if it does, I'm going to deprecate the old sphinx code soon. It's buggy anyway, throwing errors when using certain sorting methods and a bit slower than SphinxQL. Also, it's a bit confusing to have Sphinx twice in the search method administration UI.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
1 Like It 
Last Edit: August 30, 2012, 19:33:10 by Nightwish
#38
Re: minor things |
August 30, 2012, 23:57:49
Neat, topic bans. I know some mods will be happy about that :)

And yeah, SphinxQL works all fine :)

     Posted: August 31, 2012, 00:02:07
come to think of it, shouldn't something like a topic ban be fairly easy implementable as an addon?
#39
Re: minor things |
August 31, 2012, 01:26:47
And yeah, SphinxQL works all fine :)
Ok, then I can remove the old code soon.
Quote
come to think of it, shouldn't something like a topic ban be fairly easy implementable as an addon?
Maybe later. Right now, the mod system isn't able to perform db changes and such. Apart from this, it's not really difficult to convert it into a plugin (except for the warning system integration maybe).
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#40
Re: minor things |
September 22, 2012, 21:33:43
back from some absence (again)

In Profile-View.php in showPosts(), is the $range_limit yours? It seems to be a performance thing, but on my forum it hides huge numbers of posts for some members, in one extreme case all but two out of 1500. Ditched it for now.
#41
Re: minor things |
September 23, 2012, 14:52:27
back from some absence (again)

In Profile-View.php in showPosts(), is the $range_limit yours? It seems to be a performance thing, but on my forum it hides huge numbers of posts for some members, in one extreme case all but two out of 1500. Ditched it for now.
Hm, cannot remember implementing such a thing, so it's probably a SMF feature. I can vaguely remember a discussion about performance problems of the find content feature a while ago (during 2.0 development), so maybe that feature is just buggy. Will have to check this against current smf 2.1 code (which has most of the known 2.0 bugs already fixed).
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#42
Re: minor things |
September 23, 2012, 16:44:29
doesn't seem particularly slow to me even for members with huge amounts of posts. oh well.
#43
Re: minor things |
September 23, 2012, 23:08:19
doesn't seem particularly slow to me even for members with huge amounts of posts. oh well.
No, it's not slow. Checked the queries and they're all fine - no file sorts or other evil things. The range limit is strange, but it's definitely from SMF as it's still in the 2.1 code base in unmodified form.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#44
Re: minor things |
September 25, 2012, 01:53:05
git blame -> contact the developer? or maybe just get rid of it :P
#45
Re: minor things |
September 28, 2012, 19:05:01
fyi, I'm in the process of doing a css rewrite using sass and compass of the default layout right now.

At this point, I'm simply porting the css into cleaner sass syntax without changing templates. But ultimately, I aim to have the entire html use exclusively semantic descriptions, shifting all layouting into the css. If I'm doing this right, at the end of this there should be no style attributes left besides ajax things like "display: none" or dynamic ones, and no layouting class names like "floatleft".

     Posted: September 29, 2012, 20:47:59
install.php:

Quote
Some of the queries were not executed properly. This could be caused by an unsupported (development or old) version of your database software.

Technical information about the queries:
Line #1966: Column count doesn't match value count at row 29
Line #2217: Incorrect table definition; there can be only one auto column and it must be defined as a key
#46
Re: minor things |
September 30, 2012, 14:38:04
fyi, I'm in the process of doing a css rewrite using sass and compass of the default layout right now.
At this point, I'm simply porting the css into cleaner sass syntax without changing templates. But ultimately, I aim to have the entire html use exclusively semantic descriptions, shifting all layouting into the css. If I'm doing this right, at the end of this there should be no style attributes left besides ajax things like "display: none" or dynamic ones, and no layouting class names like "floatleft".
All style attributes in the templates are just temporary hacks or leftovers from the old templates. The plan is to kill them anyway (except for the very few that are needed). Layouting classes like floatleft do make sense though, because they easily allow to make the theme RTL aware (not that the current theme is fully RTL compatible, but the basic idea still stands).

Quote
Some of the queries were not executed properly. This could be caused by an unsupported (development or old) version of your database software.
I'm surprised the installer doesn't throw more serious errors :) Install.php (and especially upgrade.php) are totally untested. The last test with install.php was a couple of months back, before some database changes were made.

FYI: I'm also pretty busy at the moment, so activity was a bit low in the last couple of days. It's just temporary :)

Additionally, this forum may be go down for a short while in the next couple of days, since the server will be deactivated at the end of october, I need to switch to a different host.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
Last Edit: September 30, 2012, 14:44:43 by Nightwish
#47
Re: minor things |
October 11, 2012, 15:49:24
No sweat :)

Hey um, I wanted to add more activities, and I was wondering about the software design there. Why do you use constant numbers with defines there? isn't that a bad idea for extensibility? Is this just a temporary thing?
#48
Re: minor things |
October 11, 2012, 16:14:53
Hey um, I wanted to add more activities, and I was wondering about the software design there. Why do you use constant numbers with defines there? isn't that a bad idea for extensibility? Is this just a temporary thing?
These are only the predefined ones that must always exist and are part of the default population in the activity_types table. It should be possible (by plugins and hooks) to add custom activity types. The hooks are not yet implemented, but yeah, it's planned to support custom activity types.

Ultimately, the activity_types table defines what activities are available.

     Posted: October 26, 2012, 16:22:35
BTW: Don't wonder about the style looking different every couple of days. I'm just playing around to see how easy/not-so-easy it is to change various parts of the look.

I've also started to play around with SASS but I still prefer the SCSS syntax.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#49
Re: minor things |
October 26, 2012, 16:31:20
Also, the geoip plugin is now functional and can hook into the profile summary page and show location info to admins and profile owners (normal users cannot see geoip location for other members though).

It's probably not so useful, just an experiment to see where more hooks are badly needed for the user profile system.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#50
Re: minor things |
November 07, 2012, 18:44:50
I've added the current action a user is doing to the profile summary view. If the member is logged in, his last activity field will not only show the timestamp but also the last action he was performing (similar to what the who's online page displays).

If a member has set his online status to "invisible", this information will only be visible to administrators, of course. All others see "hidden".
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#51
Re: minor things |
January 24, 2013, 14:41:56
Been busy for the last couple of weeks, so there wasn't much activity.

I merged https://github.com/silvercircle/SMF/pull/7 which was open for about 5 months now - no idea why, never seen it and never got a notification for this pull request which seems strange, but github sometimes does act a bit weird. It's now merged with the member cards stuff being the biggest change.

Also. smarty was updated to the latest version that includes at least one important bugfix.

Plus, some other minor fixes. Compiled templates are now purged after changing certain theme settings, and the location for compiled themes is now "per theme" (e.g. $FORUMHOME/template_cache/theme_1/ is now the location for compiled templates for the theme with the id 1).
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork
#52
Re: minor things |
February 18, 2013, 23:58:24
hey there. just wanted to send a keepalive. I'm also quite busy with uni stuff at the moment, but still around  :P

we had a couple of minor bugs which I quickfixed in the meantime. I'll send patches at some point when I have some time :)
#53
Re: minor things |
February 19, 2013, 07:07:15
Just another note about templates.

There is now a new maintenance option in the admin panel (Forum Maintenance -> Routine -> Clean all compiled templates). It will do like instructed and purge the entire cache of compiled templates. In some cases, smarty may fail to properly track all the dependencies and may use already compiled templates, instead of re-compiling them when the source file has been changed.
__
Every program has at least one bug and can be shortened by at least one instruction -- from which, by induction, one can deduce that every program can be reduced to a single instruction that doesn't work.
My SMF-based forum fork