Hey there,

I tried to recreate a feature in the backend.
Since I do not have the permission to "generate features" I did not have the option to "generate" the feature.

Due to this a PHP notice occurred on submit.

Notice: Undefined index: generate in features_export_build_form_submit() (line 868 of /path/to/drupal/sites/all/modules/contrib/features/features.admin.inc).

I have fixed that and two little coding standard failures in line endings in the features.admin.inc.

Patch will be up in the second comment.

func0der

Comments

func0der created an issue. See original summary.

func0der’s picture

Status: Active » Needs review
StatusFileSize
new1.34 KB
func0der’s picture

Issue summary: View changes

Status: Needs review » Needs work

The last submitted patch, 2: features-fix_php_notice_during_recreate-2603308-2.patch, failed testing.

func0der’s picture

StatusFileSize
new1.2 KB

Wrong paths, sorry.

func0der’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 5: features-fix_php_notice_during_recreate-2603308-2.patch, failed testing.

func0der’s picture

StatusFileSize
new1.17 KB

....

func0der’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 8: features-fix_php_notice_during_recreate-2603308-2.patch, failed testing.

mpotter’s picture

Make sure you are creating your patch files correctly as per the drupal.org documentation. Looks like your patch file contains header information that is causing it to fail the testing.

func0der’s picture

StatusFileSize
new1019 bytes

I know how to create patches for Drupal modules, but my IDE removed the trailing spaces from the patch. That's why it does not work properly.

func0der’s picture

StatusFileSize
new1019 bytes

OKay, this is going to leave a bad mark on my reputation.

func0der’s picture

Status: Needs work » Needs review
alesr’s picture

StatusFileSize
new713 bytes

Fixed patch with a single line.

  • mpotter committed 5bce5df on 7.x-2.x authored by alesr
    Issue #2603308 by func0der, alesr: PHP notice during recreating a...
mpotter’s picture

Status: Needs review » Fixed

OK, Committed this to 5bce5df.

alesr: Not sure why you posted the same patch as in #13. I committed the one from #13 because fixing the trailing whitespace is still useful.

Unfortunately I screwed up the patch attribution credit...func0der I'm really sorry I didn't give you the author credit for the patch. DrEditor's git command gave credit to alesr and I was too fast and didn't notice that clicking the different radio buttons to change the credit doesn't actually update the git message on the fly.

Status: Fixed » Closed (fixed)

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