vb.org Archive

vb.org Archive (https://vborg.vbsupport.ru/index.php)
-   vBulletin.org Site Feedback (https://vborg.vbsupport.ru/forumdisplay.php?f=7)
-   -   To enforce a valid coding style in the community! (https://vborg.vbsupport.ru/showthread.php?t=93400)

Boofo 08-02-2005 11:13 AM

That passes! Good job! ;)

Revan 08-02-2005 11:15 AM

Quote:

Originally Posted by sabret00the
how does it sit right with you that you're willing to say "you're code is ++++, fix it" but you're not willing to say "oh to do that, try this" come on, if any system like the one you're backing was put into place, i'd suggest that all 'validators' had a record or helping needy coders before they even thought about validating code.

Impossible, as you already know that will diminish the team to 0. Just because I don't spend all my time helping other people in the PHP forum, doesn't make my opinion as a validator any less valuable. I know most of the standards by heart, and code from them the best I can (Im actively refusing to capitalise 'as' in foreach() statements though :p), so in what way does the fact that Im not seen in the PHP forum make me less worthy of a Validator position?
I am quite capable of offering a segment of code recoded/optimised/cleaned up.
In fact, Im offended by the wording you are using to describe how we would do our job. In no way did we intend to call the code shide, or ORDER a coder to fix it. We would send a notice to the coder sounding like so:
Quote:

Hello, $username

Thank you for participating in the Validation system here on vBulletin.org. Your participation helps improve the community as a whole, and helps the end-user feel more secure about what s/he is installing.

We have reviewed your modification, and it has been marked as $validation_result.
For more information on the different levels, and guidelines on how to improve your code, please check this page.
If you require further assistance, feel free to reply to this PM or contact any other member of the Validation Team.

Again, thank you for your participation, and thank you for sharing your hard work.

Sincerely,
vBulletin.org Validation Team
Now will you please shut the **** up about the whole "your gonna rip the piss outta the hacks hence this is ++++" deal?
Quote:

Originally Posted by sabret00the
and while you're not saying you're hounding em, didn't i read up above that you'd like to filter out un-standardized code and even possibly disallow un-standardised code? if that's not hounding people i don't know what is.

I doubt this is possible with regex, and we never said anything about disallowing un-compliant code. We would only disallow code that had serious flaws (such as security holes, noticeable performance drops, etc).

calorie 08-02-2005 11:16 AM

Quote:

Originally Posted by used as an example
1. Valid - The hack complies with the vB Coding Standards both in terms of PHP and Source Code Formatting.
2. Cleared - The hack's PHP is optimised, but it has obvious flaws in Coding Style
3. Warning - Neither the hack's PHP or the hack's Coding Style complies
4. Pending - Pending evaluation (default for new hacks)

Passes depending on what? ;)

sabret00the 08-02-2005 11:16 AM

PHP Code:

if ($a)
{
    echo 
"A";


:p

calorie 08-02-2005 11:17 AM

Exactly...

Boofo 08-02-2005 11:20 AM

Well, my coding days are over. :(

;)

Revan 08-02-2005 11:20 AM

Check my above post, it contains a good idea and a specification :P

sabret00the 08-02-2005 11:20 AM

1. https://vborg.vbsupport.ru/showpost....0&postcount=10 (first paragraph under the second quote)

2.
Quote:

Originally Posted by Revan
Just because I don't spend all my time helping other people in the PHP forum, doesn't make my opinion as a validator any less valuable.

nah it not only does. it should be the standard that's met by validators ;)

Revan 08-02-2005 11:21 AM

DV hasn't seen my post, who the hell said his idea was written in stone?

sabret00the 08-02-2005 11:21 AM

Quote:

Originally Posted by Revan
I doubt this is possible with regex, and we never said anything about disallowing un-compliant code. We would only disallow code that had serious flaws (such as security holes, noticeable performance drops, etc).

i actually think it could be done but we'd need matt in here as he's handy with regex's

Quote:

Originally Posted by Revan
DV hasn't seen my post, who the hell said his idea was written in stone?

it's his thread, i assumed he was the idea leader.

i hope my postcount is going up for all of this :nervous:


All times are GMT. The time now is 09:50 AM.

Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2025, vBulletin Solutions Inc.

X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.01374 seconds
  • Memory Usage 1,745KB
  • Queries Executed 10 (?)
More Information
Template Usage:
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (1)bbcode_php_printable
  • (7)bbcode_quote_printable
  • (1)footer
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (6)option
  • (1)pagenav
  • (1)pagenav_curpage
  • (4)pagenav_pagelink
  • (1)pagenav_pagelinkrel
  • (1)post_thanks_navbar_search
  • (1)printthread
  • (10)printthreadbit
  • (1)spacer_close
  • (1)spacer_open 

Phrase Groups Available:
  • global
  • postbit
  • showthread
Included Files:
  • ./printthread.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/class_bbcode_alt.php
  • ./includes/class_bbcode.php
  • ./includes/functions_bigthree.php 

Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • printthread_start
  • pagenav_page
  • pagenav_complete
  • bbcode_fetch_tags
  • bbcode_create
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • printthread_post
  • printthread_complete