Show Posts

This section allows you to view all posts made by this member. Note that you can only see posts made in areas you currently have access to.

Messages - Nao
9406
Off-topic / Re: Post count fever
« on April 6th, 2011, 12:27 PM »
Yeah, that's why I need you to finish the structure changes ;)

BTW I have this 'aeva' field in the members table... Maybe I should rename it to something like 'options'. This is to potentially have an unused field in the DB whose sole purpose is to provide the ability to store non-searchable personal settings for members. (i.e. through serialize.) However, I don't know if it has a point or not. I mean, we could just as well store anything into the proper options table... (I just kinda like serializing stuff. It's in my blood.)
9407
The Pub / Re: Wap2, imode?
« on April 6th, 2011, 12:23 PM »
@Snape> Not worth it I'd say. Platform-specific though, maybe.

Re: using client side to generate pages: no. I can see how it's technically doable, and several sites are doing it already, but accessibility-wise it's just a bag of shit and troubles, disabling JS means the website isn't even viewable at all, and, well... If it were something realistic to consider, don't you think more websites would be doing that? ;)

Re: chat, just because I don't like that doesn't mean I'm not open to implementing one for the vanilla Wedge, but I don't think it'd be very wise either.

There are many techs that allow communication between clients but none of them is really perfect. Even WebSockets has its drawbacks (and believe me, the more HTML5-propaganda tech I could use, the better for Wedge.)
http://en.wikipedia.org/wiki/Push_technology

Problem is, there's nothing preventing the addition of a chat system, as long as it's not updated automatically through any of these techs. Meaning, YOU refresh the page yourself. I think there are better techs to communicate in real time than a website anyway. Facebook does its best but even then, they have yet to release a perfect tool IMHO. This is really why I like Noisen/wedge.org's thought system so well (in Wedge.org it's only for the Friends group, so go to noisen.com if you want to see it in action), it's because it's closer to the type of conversation you get on Twitter -- where, I believe, no one complains that they can't actually chat in real time...?

NB: can't bother to re-read the entire topic to see if I missed anything. Feel free to wake me up in case I forgot something.
9408
Off-topic / Re: Post count fever
« on April 6th, 2011, 08:23 AM »
Pete, if you're gonna rewrite the Db structure, please do it before we even go alpha ;)
9409
Off-topic / Re: If this is "The Pub"...
« on April 6th, 2011, 07:59 AM »
Hi. I'm Nao and I approve of this topic.
9410
You need to save for an iPad3? :whistle:
9411
The Pub / Re: Wap2, imode?
« on April 5th, 2011, 06:49 PM »
I see you've learned your lesson! :niark:
Posted: April 5th, 2011, 06:48 PM

Of course, if it was part of the vanilla package, we could update it along with our other modifications :P
9412
Off-topic / Re: Post count fever
« on April 5th, 2011, 06:47 PM »
See, that's one of the reasons I'm wary of publishing a demo of Wedge right now... I can imagine the conversations at the SMF team.

"My, it's really good... And that's their vanilla release? Uh oh. We can't compete. We need to rework our stuff."

And then 10 years later......... "Wedge? We beat them. We managed to prevent them from releasing it. We won fair and square! Or close to. Well, of course it meant we never released a new version of SMF but it was the price to pay. I wouldn't do it any differently."

-- 4k guy  8-)
9413
You don' have to donate nuthin' ;)
9414
The Pub / Re: Wap2, imode?
« on April 5th, 2011, 06:41 PM »
As for Dragooon-- up to him! He could start now-- or never ;)

:edit: 4k posts!
9415
Archived fixes / Re: PM settings aren't saving properly
« on April 5th, 2011, 10:57 AM »
I'll let you sleep on it... Maybe we'll come up with a better way to do these things at some point. :)
But if we're sure to change it before the final release, we should rename the table now, to something more generic, before we forget.
Posted: April 5th, 2011, 10:57 AM

Back to AeMe now...
9416
The thing with me is that I don't want people to send me money, but everytime I see them give money to others, I say I want my share :lol: Call that jealousy or just plain stupidity but it itches in the wrong place :P
9417
Archived fixes / Re: PM settings aren't saving properly
« on April 5th, 2011, 10:46 AM »
Technically, I can't think of many things that should be a theme option... I'd even tend to say we could remove the id_theme field from that table and rename the table to something else, while we're at it... Theme settings could even be stored in a serialized field in the members table if we really wanted to go through this :P
9418
Archived fixes / Re: PM settings aren't saving properly
« on April 5th, 2011, 10:41 AM »
I believe I found the culprit and fixed it.

In loadThemeOptions (Profile-Modify.php:2125), at the beginning, replace the if block with:

Code: [Select]
if ($context['user']['is_owner'])
{
$context['member']['options'] = $options;
if (isset($_POST['options']))
foreach ($_POST['options'] as $k => $v)
$context['member']['options'][$k] = $v;
}

That makes sense. The original code went through all of the options, and would replace them with the new values only if it was set in the first place. I don't believe there's any reason to do that -- it should insert the options without question.
Posted: April 5th, 2011, 10:41 AM
Quote from Arantor on April 5th, 2011, 10:30 AM
That would imply that it's grabbed the info it's going to use (in $options ?), done what it's going to do but failed to update $options before going to the template.
Yes, that's what I was looking into. It failed to update $context['member']['options'], technically. (See above.)
9419
Archived fixes / Re: PM settings aren't saving properly
« on April 5th, 2011, 10:29 AM »
(I'm not using the tracker these days, because it kinda takes off my happiness considering I already have a long personal to-do list :P)

Okay, I tried this with my admin account and the "Remove inbox label" option, and it seems that it doesn't save settings *the first time*. As soon as I reproduced the problem, it never occurred again.

Confirmed. I just removed the settings from the DB and tried again, it didn't save but only the first time.
Profile-Modify.php is more complicated than it should be, what with the whole validation thing in an array... Why not simply do the validation in the relevant function? I don't know... Probably because it looks geeky that way?

Okay, am a bit lost... Just analyzing the query data should give me the answer, but... No?!

In both cases (first save, subsequent saves), the generated queries are:

REPLACE INTO wedge_themes(`id_member`, `id_theme`, `variable`, `value`)
VALUES
   (1, 1, SUBSTRING('view_newest_pm_first', 1, 255), SUBSTRING('1', 1, 65534)),
   (1, 1, SUBSTRING('popup_messages', 1, 255), SUBSTRING('1', 1, 65534)),
   (1, 1, SUBSTRING('copy_to_outbox', 1, 255), SUBSTRING('1', 1, 65534)),
   (1, 1, SUBSTRING('pm_remove_inbox_label', 1, 255), SUBSTRING('1', 1, 65534))

DELETE FROM wedge_themes
WHERE id_theme != 1
   AND variable IN ('view_newest_pm_first', 'popup_messages', 'copy_to_outbox', 'pm_remove_inbox_label')
   AND id_member = 1

There's no reason why the second query would delete the first round of set options. Not sure exactly... :-/
Posted: April 5th, 2011, 10:13 AM

Oh... The waste of time.

It actually saves the data correctly. It just doesn't show it on the first reload! -_-
9420
Off-topic / Re: Post count fever
« on April 5th, 2011, 10:15 AM »
I never thought THAT would have happened before DNF's release, either! :lol: