Five weeks ago, I became a Co-Maintainer of the Field conditional state module: https://drupal.org/project/field-conditional-state
The current version 2.x is my own creation to almost 100%, but I also contributed to 1.x.

With the Field conditional state module you can set the #state form field attribute based on value selected in other form fields.
The 1.x version is somehwat limited, but the target of the version 2.x is, to create an FAPI states UI for fields, supporting all its possibilities.

Repository:
git clone --branch 7.x-2.x http://git.drupal.org/project/field-conditional-state.git

Now we would like to "rename" the module from field-conditional-state to field_conditional_state (#1905376: Rename your module? - drush issues and #2101553: Roadmap 7.x-1.1 and beyond #7 and following), but according to #1875910: change project URL and git repo path the only possibility is, to create a new project.

Because of that I would like to be able to create a new full project for the 2.x branch of the Field conditional states module.

Comments

PA robot’s picture

Status: Needs review » Needs work

Link to the project page and git clone command are missing in the issue summary, please add them.

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

Tobias Xy’s picture

Issue summary: View changes

git clone added

Tobias Xy’s picture

Issue summary: View changes

Changed clone command

Tobias Xy’s picture

Status: Needs work » Needs review

I added the git clone command, but the link to the project page already exists (it's a full project; no sandbox)!

nitesh pawar’s picture

There are some errors reported by automated review tools, did you already check them? See
http://pareview.sh/pareview/httpgitdrupalorgprojectfield-conditional-sta...

Tobias Xy’s picture

Yeah, I did.

But the remaining Errors / Warnings are only about "Line too long" and "Short description in one line".
I don't understand why the short description has to be in a single line? It's sometimes really hard to describe a function within one line and a maximum of 80 characters.
I'll try to fix that, if necessary but I can't see the benefit of this.

bneil’s picture

Status: Needs review » Needs work

Hello,

Please provide a README.txt file https://drupal.org/node/447604

This sounds like a really useful module. I was testing it and I'm wondering how you delete conditional states from a field? For example, I no longer want conditionality anymore and I wasn't able to remove any settings from the field.

When deleting a field that another field depended on, I received the following error:

Notice: Undefined index: field_test in field_conditional_state_settings_form() (line 119 of /Users/bneil/drupal/drupal-7.23/sites/all/modules/field-conditional-state/field_conditional_state.admin.inc).

Perhaps there's a graceful way to handle removing a field.

Tobias Xy’s picture

Status: Needs work » Needs review

I completely rewrote the settings form, changed the database design (without update hooks, so please make a clean install!) and fixed the "Undefined index" issue.
Also I added a README.txt.

Should be fine now! :-)

Edit: The only problem reported by the automatic review tool is a false positive, I guess...

masoud561’s picture

Status: Needs review » Needs work

Hi, I reviewed your module manually and I found some Issues:

1- in conditional states tab when a user deletes two or more conditions without refreshing the page,
second delete function doesn’t work correctly.
Tip: Probably because, “$state_id” variable in “field_conditional_state_delete_condition”
Function, doesn’t change for each delete button pressing.

2- name of functions which isn’t public and isn’t hook, like some functions in
“field_conditional_state.admin.inc” file, would be better to begin with an Underscore “_”.
Tip: See drupal.org/node/1118210

3- in conditional states tab after a user clicks on save configurations button, first vertical tab is shown
But it would be better to show last modified tab.

hass’s picture

Status: Needs work » Reviewed & tested by the community

Tobias asked for a project rename and to create a new project without hyphens. Nobody asked for reviews and coder bullsh** warnings.

hass’s picture

Status: Reviewed & tested by the community » Closed (won't fix)

This is the wrong place here. Tobi, go into https://drupal.org/project/user and click in the project table the link "Add a new project"... Be careful with the names and tripple check them all before saving finally... Copy over the git code later.

Tobias Xy’s picture

Status: Closed (won't fix) » Needs work

Since this is only way I know to get the permission to create full projects (I don't want FCS 2.x to be a sandbox project), it's wrong to close this issue.
I'm not happy about this and as long as nobody can present a solution to rename our project, this seems to be the only way.

Tobias Xy’s picture

Issue summary: View changes

Extended project description

hass’s picture

You mean the link creates a sandbox project only?

hass’s picture

No, I can. I could create the project myself. You should also be able to do, too. https://drupal.org/project/add and select module project.

Tobias Xy’s picture

Issue summary: View changes
Status: Needs work » Needs review

The issues mentioned in #7 are fixed now.

@hass: I'm not able to create full projects following your link. I guess you already have the right to create full projects.

hass’s picture

No idea why. I can create it for you and give you full access if someone else is not able to grant you the propper permissions for module maintainers.

PA robot’s picture

Status: Needs review » Needs work

Git clone command for the sandbox is missing in the issue summary, please add it.

I'm a robot and this is an automated message from Project Applications Scraper.

geek-merlin’s picture

Status: Needs work » Fixed

Looks like this is fixed now.
Reopen if i got that wrong.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.