Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
There are a lot of errors and warnings but they mostly look simple enough, such as class names name needs to start with a capital letter, comments longer than 80 characters etc. Have a go at speeding through the issues list on the automated review. I have done it for one of my modules and found it was a lot easier and quicker todo that I had expected.
Some points noted there are really important. Also indentation issues are quite necessary to be fixed as this code will be used to collaborate with other contributors some other times and hence its very important to have a common standard.
Here are some points that should be taken care of:
1.Remove the translations folder, translations are done on http://localize.drupal.org
2.Remove "version" from the info file, it will be added by drupal.org packaging automatically.
Highlights of the indentation and styling issues that needs to be taken care of:
1. Line should not exceed 80 characters
2. Files must end in a single new line character
3. Function comment short description must be on a single line
4. No whitespace at the end of a line
I would suggest you to go through the output of the automated code review and fix as many errors as you can.
If you can work through all the errors and warnings you can then let us know when the repo is up to date I would be happy to look at the code again and see what I can help out with.
Fixed:
1. Removed the translations folder.
2. Files are ending in a single new line character.
3. No whitespace at the end of a line.
Most lines not exceed 80 characters excluding text stings. Mentioned indentation errors have simple text string which I formatted for best readability and I leaved it as is.
Convert comments of functions in single lines will worsen readability, imho.
The code is looking better but the automated review is still bringing up a few issues that can be fixed quickly. All the issues in views_hst_filter.module are the same spacing issue with arrays for example.
With views_hst_filter_handler_filter.inc a lot of lines seem to generate 2 errors/warnings as only 27 lines are affected with issues.
You will get more people interested in doing manual reviews once the automated review is coming up clean. Otherwise a manual review would just pick up all the same issues.
So, code is now worse than it been, but "my friendly project application review script" report less errors. Whereas Coder on first stage readiness of module reported no errors, no warnings...
This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.
Severity minor, Drupal Commenting Standards, Internationalization, Drupal Security Checks, Drupal SQL Standards, Drupal Coding Standards
sites/all/modules/pareview_temp/./test_candidate/views_hst_filter_handler_filter.inc:
+168: [minor] use lowercase html tags to comply with XHTML
Status Messages:
Coder found 3 projects, 3 files, 1 minor warnings, 0 warnings were flagged to be ignored
FILE: ...odules/pareview_temp/test_candidate/views_hst_filter_handler_filter.inc
--------------------------------------------------------------------------------
FOUND 34 ERROR(S) AFFECTING 21 LINE(S)
--------------------------------------------------------------------------------
12 | ERROR | Class name must begin with a capital letter
12 | ERROR | Class name must use UpperCamel naming without underscores
15 | ERROR | Class property $is_alone should use lowerCamel naming without
| | underscores
20 | ERROR | No scope modifier specified for function "init"
34 | ERROR | Method name "views_hst_filter_handler_filter::admin_summary" is
| | not in lowerCamel format, it must not contain underscores
34 | ERROR | No scope modifier specified for function "admin_summary"
48 | ERROR | Method name "views_hst_filter_handler_filter::option_definition"
| | is not in lowerCamel format, it must not contain underscores
48 | ERROR | No scope modifier specified for function "option_definition"
61 | ERROR | Function comment short description must be on a single line
64 | ERROR | Method name "views_hst_filter_handler_filter::options_form" is
| | not in lowerCamel format, it must not contain underscores
64 | ERROR | No scope modifier specified for function "options_form"
84 | ERROR | Method name "views_hst_filter_handler_filter::expose_form" is
| | not in lowerCamel format, it must not contain underscores
84 | ERROR | No scope modifier specified for function "expose_form"
120 | ERROR | Method name "views_hst_filter_handler_filter::expose_validate"
| | is not in lowerCamel format, it must not contain underscores
120 | ERROR | No scope modifier specified for function "expose_validate"
126 | ERROR | Method name "views_hst_filter_handler_filter::expose_submit" is
| | not in lowerCamel format, it must not contain underscores
126 | ERROR | No scope modifier specified for function "expose_submit"
134 | ERROR | Method name "views_hst_filter_handler_filter::exposed_info" is
| | not in lowerCamel format, it must not contain underscores
134 | ERROR | No scope modifier specified for function "exposed_info"
140 | ERROR | Method name "views_hst_filter_handler_filter::exposed_form" is
| | not in lowerCamel format, it must not contain underscores
140 | ERROR | No scope modifier specified for function "exposed_form"
233 | ERROR | Function comment short description must be on a single line
238 | ERROR | Method name "views_hst_filter_handler_filter::exposed_validate"
| | is not in lowerCamel format, it must not contain underscores
238 | ERROR | No scope modifier specified for function "exposed_validate"
290 | ERROR | No scope modifier specified for function "query"
324 | ERROR | Missing parameter type at position 1
331 | ERROR | Method name "views_hst_filter_handler_filter::is_valid_tid" is
| | not in lowerCamel format, it must not contain underscores
331 | ERROR | No scope modifier specified for function "is_valid_tid"
347 | ERROR | Method name "views_hst_filter_handler_filter::get_term_child" is
| | not in lowerCamel format, it must not contain underscores
347 | ERROR | No scope modifier specified for function "get_term_child"
363 | ERROR | Function comment short description must be on a single line
367 | ERROR | Missing parameter type at position 1
370 | ERROR | Method name "views_hst_filter_handler_filter::check_for_alone"
| | is not in lowerCamel format, it must not contain underscores
370 | ERROR | No scope modifier specified for function "check_for_alone"
--------------------------------------------------------------------------------
yannickoo, thank you for your message. I am sorry, but I will not more nothing to do. I will likely simple remove project through some time, than I will be make class name and functions in lowerCamel format.
Interesting, sorry ser_house, didn't know that... It's heavy from kervi do take money for code which isn't written by him. Kervi then you should delete your sandbox right?
So basically the CodeCanyon version would have thrown up these errors listed above?
It's sad that Kervi made no mention of purchasing/commissioning this himself and set up a sandbox effectively to stop others working on a solution... all the while selling somebody else's work on CC. I appreciate entrepreneurship, but basically it's held up the community.
1. i have bough the module from ser_house.
2. he give me the right to do with it all that i want.
3. with 2 new features, i have published the module on codecanyon and started to sell.
4. i have added another feature.
5. yannickoo have bough my version of module, and have created a sandbox on d.org. GPL... :(
6. i have reclaimed the authorship, and i have created a new sandbox,for at least, mantain the module.
7. the original author of module have appeared. with new sandbox.
8. i have removed my sandbox to not interferr.
9. the original author have some problems
yannickoo, thank you for your message. I am sorry, but I will not more nothing to do. I will likely simple remove project through some time, than I will be make class name and functions in lowerCamel format.
10. for now, both sandboxes are removed. If somebody will, he can start again (not me). then new sandbox will be promoved to full project, i'll remove the module from codecanyon.
That is right. The views maintainer dawehner told me that this is absolutely okay and they cannot change the views api because of the coding standards.
Hey guys, this feature is really needed and if it has to exist outside of the hierarchical select module at least for now, fine.
The Drupal way is collaboration and I;m not seeing much here.
What would ideally happen:
- We start working collaboratively in this sandbox: http://drupal.org/sandbox/ser_house/1832320
- kervi adds an issue and a patch with the new features his version of the module has
- yannickoo adds an issue and a patch to clean up the code
- seb_house commits those patches and gives git attribution to kervi and yanickoo (easy to do with dreditor)
- the module gets promoted to a full module or a patch id done for hierarchical select
- everyone has attribution on thier d.o profile for their work on the modules
- possibly with time co-maintainers?
So here is the patch for the code clean up. The maintainer should remove the master branch and create a 7.x-1.x branch instead. The README.txt file should be written in english and not in russian.
As ipwa mentioned in #27 the git commit message should be git commit -m "Issue #1841548 by yannickoo: Clean up code." --author="yan_nick <yan_nick@531118.no-reply.drupal.org>".
@ser_house: Please don't take this personally. I think your code is good and most importantly it does work. I know the review process can be daunting but we are just trying to do our best so your application is successfull. If you don't address the issues about the Git branch (you need to add a 7.x-1.x branch), and the issues mentioned in the automated review for views_hst_filter_handler_filter.inc from #9, the Git maintainers won't approve your project. If it was up to me you would get your application approved and we could start working on this issues on the live project through the issue queue, but unfortunately that's not how it works, all those little issues have to be solved before the application can be successfull. If you want to discuss more about this I'm happy to do an irc chat or an email conversation if you use my contact form, it's not that the Git maintainers don't want you to promote the project it's just that since Drupal get's so many project applications the standards for approval are quite high, but I'm sure we'll get there if we do some minor changes. Look at the guidelines to review applications to get an idea: http://drupal.org/node/894256
I will add issues to the sandbox and report back so we can start adding patches.
The @yannickoo issue already has a aptch so we are just waiting for @kervi to add his patch. Once those two issues get resolved we can mark this as 'reviewed & tested by the community" so the Git maintainers can review it.
Thanks to everyone specially @seb_house for starting this useful sub-module and to @kervi and @yanickoo for helping out.
I hope this code will be removed from codecanyon if not now at least when it's approved and goes on to be a full project on d.o
The Git issue was addressed and the code clean up, so we are nearly there.
You are right those features don't necessarily have to be in, in order for this to be a full project (although it might help when the git admins do their review)
I will do a full review tomorrow and if I find everything ok, I'll mark it as 'reviewed and tested'.
There are even multiple code issues. See pareview. You can ignore the classes stuff but the indentation, function descriptions and the max. 80 characters are important.
Review of the 7.x-1.x branch:
Run coder to check your style, some issues were found (please check the Drupal coding standards). See attachment.
This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.
Severity minor, Drupal Commenting Standards, Internationalization, Drupal Security Checks, Drupal SQL Standards, Drupal Coding Standards
sites/all/modules/pareview_temp/./test_candidate/views_hst_filter_handler_filter.inc:
+124: [critical] Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.
+217: [minor] use lowercase html tags to comply with XHTML
Status Messages:
Coder found 3 projects, 3 files, 1 critical warnings, 1 minor warnings, 0 warnings were flagged to be ignored
FILE: ...al-7-pareview/sites/all/modules/pareview_temp/test_candidate/README.txt
--------------------------------------------------------------------------------
FOUND 0 ERROR(S) AND 1 WARNING(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
20 | WARNING | Line exceeds 80 characters; contains 87 characters
--------------------------------------------------------------------------------
FILE: .../sites/all/modules/pareview_temp/test_candidate/views_hst_filter.module
--------------------------------------------------------------------------------
FOUND 10 ERROR(S) AFFECTING 10 LINE(S)
--------------------------------------------------------------------------------
67 | ERROR | Array indentation error, expected 4 spaces but found 0
68 | ERROR | Array indentation error, expected 4 spaces but found 0
69 | ERROR | Array indentation error, expected 4 spaces but found 0
70 | ERROR | Array indentation error, expected 4 spaces but found 0
71 | ERROR | Array indentation error, expected 4 spaces but found 0
72 | ERROR | Array indentation error, expected 4 spaces but found 0
73 | ERROR | Array indentation error, expected 4 spaces but found 0
74 | ERROR | Array indentation error, expected 4 spaces but found 0
75 | ERROR | Array indentation error, expected 4 spaces but found 0
76 | ERROR | Array indentation error, expected 4 spaces but found 0
--------------------------------------------------------------------------------
FILE: ...odules/pareview_temp/test_candidate/views_hst_filter_handler_filter.inc
--------------------------------------------------------------------------------
FOUND 81 ERROR(S) AND 2 WARNING(S) AFFECTING 54 LINE(S)
--------------------------------------------------------------------------------
11 | ERROR | Class name must begin with a capital letter
11 | ERROR | Class name must use UpperCamel naming without underscores
14 | ERROR | Class property $is_alone should use lowerCamel naming without
| | underscores
19 | ERROR | No scope modifier specified for function "init"
33 | ERROR | Method name "views_hst_filter_handler_filter::admin_summary"
| | is not in lowerCamel format, it must not contain underscores
33 | ERROR | No scope modifier specified for function "admin_summary"
47 | ERROR | Method name
| | "views_hst_filter_handler_filter::option_definition" is not in
| | lowerCamel format, it must not contain underscores
47 | ERROR | No scope modifier specified for function "option_definition"
65 | ERROR | Function comment short description must be on a single line
68 | ERROR | Method name "views_hst_filter_handler_filter::options_form" is
| | not in lowerCamel format, it must not contain underscores
68 | ERROR | No scope modifier specified for function "options_form"
112 | ERROR | Method name
| | "views_hst_filter_handler_filter::has_extra_options" is not in
| | lowerCamel format, it must not contain underscores
112 | ERROR | No scope modifier specified for function "has_extra_options"
120 | ERROR | Method name
| | "views_hst_filter_handler_filter::extra_options_form" is not
| | in lowerCamel format, it must not contain underscores
120 | ERROR | No scope modifier specified for function "extra_options_form"
156 | ERROR | Method name "views_hst_filter_handler_filter::expose_validate"
| | is not in lowerCamel format, it must not contain underscores
156 | ERROR | No scope modifier specified for function "expose_validate"
156 | ERROR | There should be no white space after an opening "{"
156 | ERROR | There should be no white space before a closing "}"
162 | ERROR | Method name "views_hst_filter_handler_filter::expose_submit"
| | is not in lowerCamel format, it must not contain underscores
162 | ERROR | No scope modifier specified for function "expose_submit"
170 | ERROR | Method name "views_hst_filter_handler_filter::exposed_info" is
| | not in lowerCamel format, it must not contain underscores
170 | ERROR | No scope modifier specified for function "exposed_info"
176 | ERROR | Method name "views_hst_filter_handler_filter::exposed_form" is
| | not in lowerCamel format, it must not contain underscores
176 | ERROR | No scope modifier specified for function "exposed_form"
219 | ERROR | Expected "if (...) {\n"; found "if (...){\n"
254 | WARNING | Line exceeds 80 characters; contains 87 characters
271 | WARNING | Line exceeds 80 characters; contains 83 characters
284 | ERROR | Function comment short description must be on a single line
289 | ERROR | Method name
| | "views_hst_filter_handler_filter::exposed_validate" is not in
| | lowerCamel format, it must not contain underscores
289 | ERROR | No scope modifier specified for function "exposed_validate"
342 | ERROR | No scope modifier specified for function "query"
380 | ERROR | Method name "views_hst_filter_handler_filter::is_valid_tid" is
| | not in lowerCamel format, it must not contain underscores
380 | ERROR | No scope modifier specified for function "is_valid_tid"
394 | ERROR | Method name "views_hst_filter_handler_filter::get_term_child"
| | is not in lowerCamel format, it must not contain underscores
394 | ERROR | No scope modifier specified for function "get_term_child"
417 | ERROR | Data type of return value is missing
419 | ERROR | Expected 3 space(s) before asterisk; 2 found
420 | ERROR | Method name
| | "views_hst_filter_handler_filter::get_vocabulary_levels" is
| | not in lowerCamel format, it must not contain underscores
420 | ERROR | No scope modifier specified for function
| | "get_vocabulary_levels"
435 | ERROR | Last parameter comment requires a blank newline after it
440 | ERROR | Method name
| | "views_hst_filter_handler_filter::is_single_filter" is not in
| | lowerCamel format, it must not contain underscores
440 | ERROR | No scope modifier specified for function "is_single_filter"
455 | ERROR | Expected 3 space(s) before asterisk; 2 found
455 | ERROR | Function comment short description must be on a single line
455 | ERROR | Function comment short description must end with a full stop
456 | ERROR | Expected 3 space(s) before asterisk; 2 found
457 | ERROR | Expected 3 space(s) before asterisk; 2 found
458 | ERROR | Expected 3 space(s) before asterisk; 2 found
458 | ERROR | Doc comment for var Term does not match actual variable name
| | $tid at position 1
458 | ERROR | Parameter comment must be on the next line at position 1
459 | ERROR | Expected 3 space(s) before asterisk; 2 found
460 | ERROR | Expected 3 space(s) before asterisk; 2 found
460 | ERROR | Doc comment for var TRUE does not match actual variable name
| | $child_count at position 2
460 | ERROR | Parameter comment must be on the next line at position 2
461 | ERROR | Expected 3 space(s) before asterisk; 2 found
462 | ERROR | Expected 3 space(s) before asterisk; 2 found
463 | ERROR | Expected 3 space(s) before asterisk; 2 found
464 | ERROR | Method name
| | "views_hst_filter_handler_filter::get_term_nodes_count" is not
| | in lowerCamel format, it must not contain underscores
464 | ERROR | No scope modifier specified for function
| | "get_term_nodes_count"
467 | ERROR | Expected "if (...) {\n"; found "if (...){\n"
480 | ERROR | No space before comment text; expected "//
| | $query->condition('n.type', 'node_type', '=');" but found
| | "//$query->condition('n.type', 'node_type', '=');"
489 | ERROR | Expected 3 space(s) before asterisk; 2 found
490 | ERROR | Expected 3 space(s) before asterisk; 2 found
491 | ERROR | Expected 3 space(s) before asterisk; 2 found
491 | ERROR | Last parameter comment requires a blank newline after it
491 | ERROR | Missing parameter type at position 1
492 | ERROR | Expected 3 space(s) before asterisk; 2 found
493 | ERROR | Expected 3 space(s) before asterisk; 2 found
494 | ERROR | Expected 3 space(s) before asterisk; 2 found
495 | ERROR | Expected 3 space(s) before asterisk; 2 found
496 | ERROR | Method name
| | "views_hst_filter_handler_filter::term_get_children_ids" is
| | not in lowerCamel format, it must not contain underscores
496 | ERROR | No scope modifier specified for function
| | "term_get_children_ids"
500 | ERROR | Expected "if (...) {\n"; found "if (...){\n"
501 | ERROR | Expected "foreach (...) {\n"; found "foreach(...) {\n"
513 | ERROR | Last parameter comment requires a blank newline after it
518 | ERROR | Method name
| | "views_hst_filter_handler_filter::build_options_list" is not
| | in lowerCamel format, it must not contain underscores
518 | ERROR | No scope modifier specified for function "build_options_list"
533 | ERROR | Expected "if (...) {\n"; found "if(...){\n"
539 | ERROR | Expected "if (...) {\n"; found "if(...){\n"
556 | ERROR | Method name "views_hst_filter_handler_filter::build_widget_id"
| | is not in lowerCamel format, it must not contain underscores
556 | ERROR | No scope modifier specified for function "build_widget_id"
558 | ERROR | String concat is not required here; use a single string
| | instead
--------------------------------------------------------------------------------
@ChristopheDG: I don't understand, ser_house has a sandbox project called Views hst filter: http://drupal.org/sandbox/ser_house/1832320
It's a very useful module, and can be used on D7. But it's still a sandbox project yet, but it shouldn't scare you.
ser_house: you marked this application as "won't fix". Does this mean you do not want this project be promoted to a full project and do not want to get full git access?
If you still want to continue with your application, please get some review bonus by reviewing other applications.
Btw., the project application can last a long time. You create this issue about 9 weeks ago, which is not very long. Just have a look at all the other applications waiting for input. We are quite busy at the moment (which is unfortunately a permanent state) so keep patient and don't give up ...
you marked this application as "won't fix". Does this mean you do not want this project be promoted to a full project and do not want to get full git access?
Want I? Now quite a bit.
On the other hand...
Hi,
How can I download this wonderful module of yours.
Is it useful module for people? Yes.
Can people simply download this module? No.
Will I review other applications? No. I want make review, really. But when I did decide to do it, not when "If you still want to continue with your application".
9 weeks is not very long, I agree. And I believe that people are busy.
@Wim Leers:
Why? This module has no relation to your module. Moreover, it is hardly compatible with your linear mode. So, I don't see any point in this too.
@ser_house :
:( why won't your module be available here? I think your module is great, and should be continued to be discussed and developed here, it would even be better if it got to a full release... :)
@Sk8erPeter:
But it not going to the full release. And if it so, my module will be on my site, as other my modules. All are downloaded every day, and I do not need to drupal.org for this. Just it will be people who know russian language in most case.
I don't want and I don't wait anything more from here.
@ser_house :
But what made you angry with drupal.org? :( This way we can not easily participate in creating patches, helping your efforts, etc. :(
And I think many users don't speak Russian (e.g. unfortunately I don't).
I would be really sad if you removed your project.
I'm surprised at your negative approach. I read the comment you got offended at, where #1410826: [META] Review bonus was linked and where stBorchert said they were really busy currently "so keep patient and don't give up"... this comment didn't contain any offensive stuffs, and just stated that it can take a long time for your module to get reviewed and for your module to be promoted to a full release, because the queue is really huge, but you can speed up this process by getting some review bonus. Noone said that your project will be deleted if you don't review other modules.
I don't understand why you didn't simply keep this module here in your own sandbox if you don't have time to deal with other modules' reviews (that's absolutely understandable). This way you helped other people and improved your reputation! But now you deleted this useful module... :((
I have enabled the above module,but I don't know what to do next,can someone help me?
I wan to limit the options on a taxonomy exposed filter view to level 2 terms.How can I do that?thanks
@pinkonomy: you need to add Simple hierarchical select filter to views filters. This is a new kind of filter and you can find it by simply typing hierarchical in Search field while adding filter. Then you'll get all the settings you need hopefully.
Comments
Comment #1
adamelleston CreditAttribution: adamelleston commentedHi,
I ran your code through this automated tool to check against coding standard, number of spaces in tab, syntax, newline at end of file etc
http://ventral.org/pareview/httpgitdrupalorgsandboxserhouse1832320git
There are a lot of errors and warnings but they mostly look simple enough, such as class names name needs to start with a capital letter, comments longer than 80 characters etc. Have a go at speeding through the issues list on the automated review. I have done it for one of my modules and found it was a lot easier and quicker todo that I had expected.
Comment #2
ser_house CreditAttribution: ser_house commentedThank you for your check. Some of those errors and warnings really useful, but not all. As example the class name style for views_handler_filter.
Comment #3
aritra.ghosh CreditAttribution: aritra.ghosh commentedHii,
Did you fix the code according to the suggestions of the automated code review yet?
http://ventral.org/pareview/httpgitdrupalorgsandboxserhouse1832320git
Some points noted there are really important. Also indentation issues are quite necessary to be fixed as this code will be used to collaborate with other contributors some other times and hence its very important to have a common standard.
Here are some points that should be taken care of:
1.Remove the translations folder, translations are done on http://localize.drupal.org
2.Remove "version" from the info file, it will be added by drupal.org packaging automatically.
Highlights of the indentation and styling issues that needs to be taken care of:
1. Line should not exceed 80 characters
2. Files must end in a single new line character
3. Function comment short description must be on a single line
4. No whitespace at the end of a line
I would suggest you to go through the output of the automated code review and fix as many errors as you can.
Thanks,
Aritra
Comment #4
adamelleston CreditAttribution: adamelleston commentedIf you can work through all the errors and warnings you can then let us know when the repo is up to date I would be happy to look at the code again and see what I can help out with.
Comment #5
ser_house CreditAttribution: ser_house commentedHi,
Fixed:
1. Removed the translations folder.
2. Files are ending in a single new line character.
3. No whitespace at the end of a line.
Most lines not exceed 80 characters excluding text stings. Mentioned indentation errors have simple text string which I formatted for best readability and I leaved it as is.
Convert comments of functions in single lines will worsen readability, imho.
Thanks.
PS: sorry for my writing mistakes.
Comment #6
adamelleston CreditAttribution: adamelleston commentedThe code is looking better but the automated review is still bringing up a few issues that can be fixed quickly. All the issues in views_hst_filter.module are the same spacing issue with arrays for example.
With views_hst_filter_handler_filter.inc a lot of lines seem to generate 2 errors/warnings as only 27 lines are affected with issues.
You will get more people interested in doing manual reviews once the automated review is coming up clean. Otherwise a manual review would just pick up all the same issues.
Comment #7
yannickooIt seems that you have the module from kervi. I already uploaded the code. You can see a cleaner version of this module here.
Comment #8
ser_house CreditAttribution: ser_house commentedSo, code is now worse than it been, but "my friendly project application review script" report less errors. Whereas Coder on first stage readiness of module reported no errors, no warnings...
Comment #9
yannickooYou have to work on "your" code so don't mark it as "needs review".
It appears you are working in the "vental_org" branch in git. You should really be working in a version specific branch. The most direct documentation on this is Moving from a master branch to a version branch. For additional resources please see the documentation about release naming conventions and creating a branch in git.
There is still a master branch, make sure to set the correct default branch: http://drupal.org/node/1659588 . Then remove the master branch, see also step 6 and 7 in http://drupal.org/node/1127732
Review of the vental_org branch:
This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.
Source: http://ventral.org/pareview - PAReview.sh online service
Comment #10
ser_house CreditAttribution: ser_house commentedyannickoo, thank you for your message. I am sorry, but I will not more nothing to do. I will likely simple remove project through some time, than I will be make class name and functions in lowerCamel format.
Comment #11
yannickooI would advise you to delete your sandbox and wait for kervi's module because he is the original author of this module.
Let's close this issue because this is not your code and this is not the sense behind project applications.
Comment #12
ser_house CreditAttribution: ser_house commented:-) I'm afraid you're mistaken: I am is original author of this module. Kervi ordered it in September 2012 and I made this module.
I don't know, how such case are resolving here, but I am know exactly what I wrote it.
Comment #13
ser_house CreditAttribution: ser_house commentedI am open this issue again.
Comment #14
yannickooBut kervi is the author from the module, take a look here.
Comment #15
dgastudio CreditAttribution: dgastudio commentedyes, you are the original author.
but you give me all the rights about this module, remember?
but, no problem. i don't have time to mantain it, so you are the boss.
Comment #16
ser_house CreditAttribution: ser_house commentedkervi, it was GPLv3, remember? You can do all you wish, but author? See, me accused what I stole your code, it right?
I think such things need coordinate in advance.
Comment #17
ser_house CreditAttribution: ser_house commentedTake a look here.
Comment #18
yannickooInteresting, sorry ser_house, didn't know that... It's heavy from kervi do take money for code which isn't written by him. Kervi then you should delete your sandbox right?
Comment #19
ser_house CreditAttribution: ser_house commentedIt's okay, I hope.
I think kervi has right to take money for it, but not call himself original author of code. Just we didn't coordinate with him such case, sorry.
Comment #20
dgastudio CreditAttribution: dgastudio commentedReally sorry for all this story. Never again.
Sandbox deleted.
Comment #21
yannickooKervi should also stop selling it! Crazy story...
And ser_house should get a review bonus so that this module can get faster reviewed.
Comment #22
ipwa CreditAttribution: ipwa commentedCross posting relevant HS issue: http://drupal.org/node/1170192
Comment #23
choitz CreditAttribution: choitz commentedSo basically the CodeCanyon version would have thrown up these errors listed above?
It's sad that Kervi made no mention of purchasing/commissioning this himself and set up a sandbox effectively to stop others working on a solution... all the while selling somebody else's work on CC. I appreciate entrepreneurship, but basically it's held up the community.
Comment #24
dgastudio CreditAttribution: dgastudio commentedno, not exactly.
1. i have bough the module from ser_house.
2. he give me the right to do with it all that i want.
3. with 2 new features, i have published the module on codecanyon and started to sell.
4. i have added another feature.
5. yannickoo have bough my version of module, and have created a sandbox on d.org. GPL... :(
6. i have reclaimed the authorship, and i have created a new sandbox,for at least, mantain the module.
7. the original author of module have appeared. with new sandbox.
8. i have removed my sandbox to not interferr.
9. the original author have some problems
10. for now, both sandboxes are removed. If somebody will, he can start again (not me). then new sandbox will be promoved to full project, i'll remove the module from codecanyon.
Comment #25
ser_house CreditAttribution: ser_house commentedMaybe I have some problems, but anyway class, inherited from views_handler_filter mustn't be in lowerCamel format.
Comment #26
yannickooThat is right. The views maintainer dawehner told me that this is absolutely okay and they cannot change the views api because of the coding standards.
Comment #27
ipwa CreditAttribution: ipwa commentedHey guys, this feature is really needed and if it has to exist outside of the hierarchical select module at least for now, fine.
The Drupal way is collaboration and I;m not seeing much here.
What would ideally happen:
- We start working collaboratively in this sandbox: http://drupal.org/sandbox/ser_house/1832320
- kervi adds an issue and a patch with the new features his version of the module has
- yannickoo adds an issue and a patch to clean up the code
- seb_house commits those patches and gives git attribution to kervi and yanickoo (easy to do with dreditor)
- the module gets promoted to a full module or a patch id done for hierarchical select
- everyone has attribution on thier d.o profile for their work on the modules
- possibly with time co-maintainers?
Let's make it happen!
Comment #28
Sk8erPeter CreditAttribution: Sk8erPeter commented+1 for @ipwa! :)
Comment #29
ser_house CreditAttribution: ser_house commentedWell, I don't quite understand what is being said.
1. What are new features?
2 .What sort of clean up need for code? Again about lowerCamel format?
I have impression what I published very bad code and without collaboration just impossible promote it to full project.
Why? I'm just curious.
Comment #30
dgastudio CreditAttribution: dgastudio commentednew features:
1. counter of related nodes for each term
2. exclude empty term
3. unique id for each widget.
Comment #31
yannickooSo here is the patch for the code clean up. The maintainer should remove the master branch and create a 7.x-1.x branch instead. The README.txt file should be written in english and not in russian.
As ipwa mentioned in #27 the git commit message should be
git commit -m "Issue #1841548 by yannickoo: Clean up code." --author="yan_nick <yan_nick@531118.no-reply.drupal.org>"
.Comment #32
ipwa CreditAttribution: ipwa commented@ser_house: Please don't take this personally. I think your code is good and most importantly it does work. I know the review process can be daunting but we are just trying to do our best so your application is successfull. If you don't address the issues about the Git branch (you need to add a 7.x-1.x branch), and the issues mentioned in the automated review for views_hst_filter_handler_filter.inc from #9, the Git maintainers won't approve your project. If it was up to me you would get your application approved and we could start working on this issues on the live project through the issue queue, but unfortunately that's not how it works, all those little issues have to be solved before the application can be successfull. If you want to discuss more about this I'm happy to do an irc chat or an email conversation if you use my contact form, it's not that the Git maintainers don't want you to promote the project it's just that since Drupal get's so many project applications the standards for approval are quite high, but I'm sure we'll get there if we do some minor changes. Look at the guidelines to review applications to get an idea: http://drupal.org/node/894256
I will add issues to the sandbox and report back so we can start adding patches.
Comment #33
ipwa CreditAttribution: ipwa commentedGreat! We are geting somewhere these are the two issues:
#1841548: Clean up code
#1841560: New features from @kervi
The @yannickoo issue already has a aptch so we are just waiting for @kervi to add his patch. Once those two issues get resolved we can mark this as 'reviewed & tested by the community" so the Git maintainers can review it.
Thanks to everyone specially @seb_house for starting this useful sub-module and to @kervi and @yanickoo for helping out.
I hope this code will be removed from codecanyon if not now at least when it's approved and goes on to be a full project on d.o
Comment #34
ser_house CreditAttribution: ser_house commentedVery good.
#1841548: Clean up code fixed.
#1841560: New features from @kervi: It's really need for full project at this time? Users must wait while we have add it?
@kervi: Would you like to contact me for discuss about your status co-maintainer (icq or gtalk)?
Comment #35
ipwa CreditAttribution: ipwa commentedAwesome.
The Git issue was addressed and the code clean up, so we are nearly there.
You are right those features don't necessarily have to be in, in order for this to be a full project (although it might help when the git admins do their review)
I will do a full review tomorrow and if I find everything ok, I'll mark it as 'reviewed and tested'.
Comment #36
yannickooThere are even multiple code issues. See pareview. You can ignore the classes stuff but the indentation, function descriptions and the max. 80 characters are important.
Review of the 7.x-1.x branch:
This automated report was generated with PAReview.sh, your friendly project application review script. You can also use the online version to check your project. You have to get a review bonus to get a review from me.
Source: http://ventral.org/pareview - PAReview.sh online service
Comment #37
ser_house CreditAttribution: ser_house commentedI don't understand (NetBeans):
@yannickoo: This is code after applying an your patch (http://drupal.org/node/1841548)
Comment #38
ser_house CreditAttribution: ser_house commentedCoder
Options:
Results:
Comment #39
ser_house CreditAttribution: ser_house commentedДумаю, достаточно.
Comment #40
ChristopheDG CreditAttribution: ChristopheDG commentedThis is sad news. So many people are waiting for this functionality in D7.
Are there any decent alternatives out there?
Comment #41
Sk8erPeter CreditAttribution: Sk8erPeter commented@ChristopheDG: I don't understand, ser_house has a sandbox project called Views hst filter:
http://drupal.org/sandbox/ser_house/1832320
It's a very useful module, and can be used on D7. But it's still a sandbox project yet, but it shouldn't scare you.
Comment #42
ipwa CreditAttribution: ipwa commentedGit maintainers, this application has been reviewed and tested by the community, please grant full Git access to @ser_house.
Comment #43
stBorchertser_house: you marked this application as "won't fix". Does this mean you do not want this project be promoted to a full project and do not want to get full git access?
If you still want to continue with your application, please get some review bonus by reviewing other applications.
Btw., the project application can last a long time. You create this issue about 9 weeks ago, which is not very long. Just have a look at all the other applications waiting for input. We are quite busy at the moment (which is unfortunately a permanent state) so keep patient and don't give up ...
Comment #44
ser_house CreditAttribution: ser_house commentedWant I? Now quite a bit.
On the other hand...
And next:
Is it useful module for people? Yes.
Can people simply download this module? No.
Will I review other applications? No. I want make review, really. But when I did decide to do it, not when "If you still want to continue with your application".
9 weeks is not very long, I agree. And I believe that people are busy.
Postponed? Ok.
Comment #45
Wim LeersWhat is holding this one up?
Let's get this into Hierarchical Select proper. See #1170192-239: Port Hierarchical Select Taxonomy Views (for exposed filters in Views) to Drupal 7.
Comment #46
ymakux CreditAttribution: ymakux commentedComment #47
ser_house CreditAttribution: ser_house commentedOk, I don’t see any point in this more.
The module will be available on my site (http://serho.ru/custmodules/filtr-dlya-views-s-ierarhicheskim-vyborom-terminov), not here.
@Wim Leers:
Why? This module has no relation to your module. Moreover, it is hardly compatible with your linear mode. So, I don't see any point in this too.
Comment #48
Sk8erPeter CreditAttribution: Sk8erPeter commented@ser_house :
:( why won't your module be available here? I think your module is great, and should be continued to be discussed and developed here, it would even be better if it got to a full release... :)
Comment #49
ser_house CreditAttribution: ser_house commented@Sk8erPeter:
But it not going to the full release. And if it so, my module will be on my site, as other my modules. All are downloaded every day, and I do not need to drupal.org for this. Just it will be people who know russian language in most case.
I don't want and I don't wait anything more from here.
Comment #50
Sk8erPeter CreditAttribution: Sk8erPeter commented@ser_house :
But what made you angry with drupal.org? :( This way we can not easily participate in creating patches, helping your efforts, etc. :(
And I think many users don't speak Russian (e.g. unfortunately I don't).
I would be really sad if you removed your project.
Comment #51
ser_house CreditAttribution: ser_house commented@Sk8erPeter:
I am so sorry :(
Comment #52
yannickooser_house, what is the problem now? Why you don't want to promote your sandbox as full project? I don't understand that.
BTW thank you for mention us on your page ;)
Comment #53
ser_house CreditAttribution: ser_house commented@yannickoo:
Should I to dance for this?
So, I have deleted the module. Please, remove this post.
Comment #54
yannickooI also can put this on drupal.org, what do you think?
Comment #55
ser_house CreditAttribution: ser_house commentedI don't want my code was on drupal.org. Don't, under any circumstances.
Comment #56
yannickooOkay, that is our opinion. All other users can use the new Simple hierarchical select.
Comment #57
Sk8erPeter CreditAttribution: Sk8erPeter commented@ser_house :
I'm surprised at your negative approach. I read the comment you got offended at, where #1410826: [META] Review bonus was linked and where stBorchert said they were really busy currently "so keep patient and don't give up"... this comment didn't contain any offensive stuffs, and just stated that it can take a long time for your module to get reviewed and for your module to be promoted to a full release, because the queue is really huge, but you can speed up this process by getting some review bonus. Noone said that your project will be deleted if you don't review other modules.
I don't understand why you didn't simply keep this module here in your own sandbox if you don't have time to deal with other modules' reviews (that's absolutely understandable). This way you helped other people and improved your reputation! But now you deleted this useful module... :((
Comment #58
ymakux CreditAttribution: ymakux commented@yannickoo: the Simple hierarchical select was developed from scratch or forked from ser_house's module?
Comment #59
stBorchertThe module has been built from scratch and takes a different approach (no dependency to Hierarchical select, for taxonomy only).
Comment #60
Sk8erPeter CreditAttribution: Sk8erPeter commented@stBorchert : thanks for your efforts, I will check out your module, I hope this will be a good alternative. :))
Comment #61
ymakux CreditAttribution: ymakux commented@stBorchert: Ok, thanks, looks greate
Comment #62
pinkonomy CreditAttribution: pinkonomy commentedI have enabled the above module,but I don't know what to do next,can someone help me?
I wan to limit the options on a taxonomy exposed filter view to level 2 terms.How can I do that?thanks
Comment #63
philipz CreditAttribution: philipz commented@pinkonomy: you need to add Simple hierarchical select filter to views filters. This is a new kind of filter and you can find it by simply typing hierarchical in Search field while adding filter. Then you'll get all the settings you need hopefully.
Comment #64
marcoka CreditAttribution: marcoka commentedgreat alternative: http://drupal.org/project/shs
Comment #64.0
marcoka CreditAttribution: marcoka commentedEnough.
Comment #65
smartsystems160 CreditAttribution: smartsystems160 commentedshs works for nodes, not users. its not a complete solution for the hs view exposed filters issue