Improving quality of submitted hacks (code)

[Hornstar]

I'd like to hear what others think could be done to improve the quality of submitted hacks here.

A couple weeks back on vb.com in the licensed area people were discussing areas about the quality of code in some of the hacks submitted on vb.org and how they can severely affect big boards etc.

-----------
One idea I had was that there could be a team set up to approve hacks before they are shown. Exactly the same way the articles forum is set up: https://vborg.vbsupport.ru/forumdisplay.php?f=184
This way will ensure hacks that are submitted are safer for our sites. Currently anyone could submit a hack where it could potentially delete/screwup our whole database.
It could also give the chance for this 'team' to give feedback and suggestions to the coder before the hack are approved. Kind of like one final pre beta test.

What other things do you think could be done to improve this area on vb.org?

[Marco van Herwaarden]

This is also a topic that comes up once in a while. We have been experimenting with a "Quality Ensurance" team in the past, but that just don't work. Inspecting the underlying code of all submitted modifications is simply not feasible and we also do not want to put too many barrieres for people to submit their work to the community.

The only way to improve the quality is to provide feedback to the coder and educate them in better ways of solving the problem. If and what a coder does with such information is up to him/her.

[Paul M]

Originally Posted by Marco van Herwaarden

Inspecting the underlying code of all submitted modifications is simply not feasible and we also do not want to put too many barrieres for people to submit their work to the community.

Just to put a figure on things, approx 90 code modifications were released over the last month, anything from simple 1 plugin mods, to quite complicated, multi file, multi plugin products - and at the moment we are in a quiet period as many coders etc are waiting for vb4.

That figure does not include updates to existing modifications either, just new threads.

[Regs]

Just throwing this idea out there, but what if the mod that was voted "mod of the month" not only received a shiny icon/badge but also a code review by a dedicated team here?

Or, for now, looking at major mods with the most installs (that work with the latest release) and code reviewing those?

I dunno. Just throwing stuff out there I guess. There is so much released here now that it can be difficult in knowing what the "best" really is.

[Marco van Herwaarden]

What would be the benefit of inspecting 1 modification, and then a modification that already is in the spot lights.

PS It could turn into a deception if winners of the BOTM would often be denied aproval as the coding doesn't pass such a quality control. And history shows that it is not always the "best" modification that wins, more often it is simply the most used or best promoted (even though vote rallying is against our rules).

[Regs]

I don't understand your question?

Maybe I wasn't very clear in the idea... the team isn't "approving" a mod, they are "helping" a mod author improve what the community has decided is popular.

If the code isn't good (i.e., your definition of the "best" mod), wouldn't it be advantageous to both the mod author AND the community to help make it the best it could be?

Furthermore, if you are already internally (whether just in your head or amongst staff) ranking what are "best" mods, then why not share with the community what these are?

In my mind, now is the time to really give this topic a serious thought. If most are awaiting vB4 to start releasing new code, why not put something together and give it a trial run before the you-know-what hits the fan?

[Marco van Herwaarden]

We (both staff and members) already spend a lot of time educating users (including coders), and that is what we can do.

There are too many modifications released or updated each month to inspect them all. Also staff is here to moderate the site, not to inspect modifications. But if a (larger) group members with coding experience would commit for a longer time to taking part in a Quality Assurance team, then we might consider giving it a go again, but i doubt it will work as history has proven that either not enough members want to spend their time on inspecting other peoples work and or interest faded after some time and the whole project collapses.

It will also not be easy to have objective guidelines on when to accept a submission and when not.

PS Yes i have my personal list of "good" modifications. But my criterium (being a coder myself) is simply: If i didn't write it myself, i don't trust it. So hardly worthwhile to post such a list.

PS There is an older thread on the same topic: https://vborg.vbsupport.ru/showthrea...hlight=quality

[TheLastSuperman]

Originally Posted by Marco van Herwaarden

What would be the benefit of inspecting 1 modification, and then a modification that already is in the spot lights.

PS It could turn into a deception if winners of the BOTM would often be denied aproval as the coding doesn't pass such a quality control. And history shows that it is not always the "best" modification that wins, more often it is simply the most used or best promoted (even though vote rallying is against our rules).

Glad I saw that post because I missed it... I was supporting a users mod the other month for MOTM via my sig so since I did not hear anything hopefully that WAS within the rules but know I know for future reference.

How about this... it's simple but any new mod that just jumps through the roof w/ installs should be checked? I dunno as this is going to be a touchy subject w/ lots of opinions so IMO you vb admins decide what is best and we will just have to go w/ the flow otherwise this will be knit-picked to death and we will still be posting months from now just like many other suggestions that come back up every few months .

S-MAN

[Paul M]

Personally, 95% of the mods I release are written for my forum, and therefore Im not interested in whether they follow some arbitary coding standard on a 3rd party site. I release them here simply because i think they will be useful to other people (simply doing that, and then supporting, them takes up chunks of my limited free time).

If it became the case that they had to follow a standard, or not be released, then I would take the second option, and simply not release them any more - Im pretty sure this is what a lot of people would do.

[Carnage]

I was thinking about something a few weeks ago that could help here.

Improve the ratings system for mods; have several categories eg user friendlyness ease of install quality of code etc allow people to rank in all the categories

Additionally, allow any reply to a mod thread to be given a 'this is a review of the mod' flag So that members could post reviews as well as just support requests/general comments. These reviews could be then linked from the first post in the mods information pannel.