CommentFileSizeAuthor
#9 advagg-2752147-9-fix-undefined-indexes.patch2.45 KBmikeytown2
#8 advagg-2752147-8-fix-undefined-indexes.patch2.45 KBmikeytown2
#7 undefined_index-2752147-7.patch4.4 KBmarkdorison
#7 interdiff-2752147-4-7.txt3.45 KBmarkdorison
#4 advagg-2752147-4-fix-undefined-index-weight.patch1.48 KBmikeytown2
#2 Undefined_index_weight-2752147-2.patch1.52 KBsergeimalyshev

Comments

kpacaba11 created an issue. See original summary.

sergeimalyshev’s picture

sergeimalyshev commented
StatusFileSize
new Undefined_index_weight-2752147-2.patch1.52 KB
mikeytown2’s picture

mikeytown2 commented
Status: Active » Needs review
mikeytown2’s picture

mikeytown2 commented
Version: 7.x-2.14 » 7.x-2.x-dev
StatusFileSize
new advagg-2752147-4-fix-undefined-index-weight.patch1.48 KB

Re-worked a couple minor points and made this patch

Status: Needs review » Needs work

The last submitted patch, 4: advagg-2752147-4-fix-undefined-index-weight.patch, failed testing.

mikeytown2’s picture

mikeytown2 commented
Status: Needs work » Needs review
markdorison’s picture

markdorison
he/him
commented
StatusFileSize
new interdiff-2752147-4-7.txt3.45 KB
new undefined_index-2752147-7.patch4.4 KB

I am still seeing a number of these warnings in my testing. New patch and interdiff attached.

mikeytown2’s picture

mikeytown2 commented
StatusFileSize
new advagg-2752147-8-fix-undefined-indexes.patch2.45 KB

Thinking this will fix all the issues. Still wondering how all the defaults got blown out see https://api.drupal.org/api/drupal/includes%21common.inc/function/drupal_... and https://api.drupal.org/api/drupal/includes!common.inc/function/drupal_ad...

mikeytown2’s picture

mikeytown2 commented
StatusFileSize
new advagg-2752147-9-fix-undefined-indexes.patch2.45 KB

Fixes a typo

  • mikeytown2 committed 31fdc90 on 7.x-2.x 
    Issue #2752147 by mikeytown2, markdorison, kpacaba11: Undefined index:...
mikeytown2’s picture

mikeytown2 commented
Status: Needs review » Fixed

#9 has been committed

Status: Fixed » Closed (fixed)

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