Why build_datastore with no arguments?

[futureal]

While tinkering around in vB3's code, I noticed many places that do something like (in pseudo-pseudocode):

$DB_site->query("UPDATE datastore SET ... ");
build_datastore();

Since the build_datastore function takes two arguments (title and data) and does nothing if the arguments aren't given...what's the point?

Were these calls added for stylistic reasons, or...? Perhaps future development?

I know this is an oddball question, just looking for an answer to satisfy my curiosity.

[Link14716]

Originally Posted by futureal

While tinkering around in vB3's code, I noticed many places that do something like (in pseudo-pseudocode):

$DB_site->query("UPDATE datastore SET ... ");
build_datastore();

Since the build_datastore function takes two arguments (title and data) and does nothing if the arguments aren't given...what's the point?

Were these calls added for stylistic reasons, or...? Perhaps future development?

I know this is an oddball question, just looking for an answer to satisfy my curiosity.

That's an interesting question. It doesn't make a whole lot of sense, does it?

[Xenon]

very good question.

maybe it was planned to update the datastore chache when running it without parameters, but they forgot it or decided otherwise..

[Revan]

Um, have you all been smoking crack or something? build_datastore without arguments CLEARLY rebuilds the $vboptions/$vbulletin->options array.
I've been using that in my installers since I learned of the function, and the array has always been updated just like it should.

[Andreas]

At least in 3.0.7 and 3.5.0 Beta 2 a call to build_datastore() without arguments does nothing.

[Paul M]

Originally Posted by KirbyDE

At least in 3.0.7 and 3.5.0 Beta 2 a call to build_datastore() without arguments does nothing.

It does nothing in any version of 3.0.x I have (3.0.1 +) the code is the same in all - basically ;

PHP Code:

function build_datastore($title = '', $data = '')
{

    if ($title != '')
    {
        * DO STUFF *
    }

} 


So if no parameters are supplied, it does nothing.

[futureal]

Yea, pretty sure I'm not smoking crack.

I was writing a quick hack involving the datastore and happened to take a look at that function, just thought it was weird. My best guess is that it used to have some sort of debugging trace in there that they removed for the actual release.

[Link14716]

I checked the 3.0.0 Beta 3 code and the function didn't even exist then. In fact, it didn't appear until 3.0.0 Release Candidate 4. The function is the same then as it is in 3.0.7. It looks to have changed a bit in 3.5.0, though, but not much.

[Paul M]

Originally Posted by Link14716

I checked the 3.0.0 Beta 3 code and the function didn't even exist then. In fact, it didn't appear until 3.0.0 Release Candidate 4. The function is the same then as it is in 3.0.7. It looks to have changed a bit in 3.5.0, though, but not much.

Yep - but the change in 3.5.0 is still inside the IF - so it still will do nothing if no arguments are supplied.

[tamarian]

Note that there are a couple of functions called build_datastore, so keep an eye for the require_once line.