Errors generated when visiting /admin/hosting/backup_manager 

Notice: Undefined index: #type in theme_hosting_backup_gc_intervals_table() (line 877 of /var/aegir/hostmaster-7.x-3.x/sites/keylime.illuminateweb.org.uk/modules/hosting_site_backup_manager/hosting_site_backup_manager.module).
Warning: array_merge(): Argument #1 is not an array in _form_set_class() (line 4273 of /var/aegir/hostmaster-7.x-3.x/includes/form.inc).
Warning: array_merge(): Argument #1 is not an array in _form_set_class() (line 4273 of /var/aegir/hostmaster-7.x-3.x/includes/form.inc).
Warning: array_merge(): Argument #1 is not an array in _form_set_class() (line 4273 of /var/aegir/hostmaster-7.x-3.x/includes/form.inc).
Warning: array_merge(): Argument #1 is not an array in _form_set_class() (line 4273 of /var/aegir/hostmaster-7.x-3.x/includes/form.inc).
Notice: Undefined index: #type in theme_hosting_backup_gc_intervals_table() (line 877 of /var/aegir/hostmaster-7.x-3.x/sites/keylime.illuminateweb.org.uk/modules/hosting_site_backup_manager/hosting_site_backup_manager.module).
Warning: array_merge(): Argument #1 is not an array in _form_set_class() (line 4273 of /var/aegir/hostmaster-7.x-3.x/includes/form.inc).
Warning: array_merge(): Argument #1 is not an array in _form_set_class() (line 4273 of /var/aegir/hostmaster-7.x-3.x/includes/form.inc).
Warning: array_merge(): Argument #1 is not an array in _form_set_class() (line 4273 of /var/aegir/hostmaster-7.x-3.x/includes/form.inc).
Warning: array_merge(): Argument #1 is not an array in _form_set_class() (line 4273 of /var/aegir/hostmaster-7.x-3.x/includes/form.inc).
CommentFileSizeAuthor
#9 classes-must-be-in-arrays-2545976-9.patch799 bytesformatC'vt
#2 hsbm_fapi_class_bug.patch2.05 KBgboudrias
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mshepherd created an issue. See original summary.

gboudrias’s picture

gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commented
Parent issue: » #2483233: Finish porting of hosting_backup_queue
FileSize
hsbm_fapi_class_bug.patch2.05 KB

Pushed this patch to dev, please review. Should probably release if this fixes it.

gboudrias’s picture

gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commented
Status: Active » Needs review

  • gboudrias committed f62054c on 7.x-3.x 
    Issue #2545976 by gboudrias, mshepherd: Fix GC FAPI class bug on...
gboudrias’s picture

gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commented
Issue tags: +Aegir 3.0.2
gboudrias’s picture

gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commented
Issue tags: -Aegir 3.0.2 +Aegir 3.1
helmo’s picture

helmo CreditAttribution: helmo at Initfour websolutions commented
Status: Needs review » Reviewed & tested by the community

looks good, could even be called fixed as it's committed.

gboudrias’s picture

gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commented
Status: Reviewed & tested by the community » Fixed
formatC'vt’s picture

formatC'vt CreditAttribution: formatC'vt as a volunteer commented
Status: Fixed » Needs review
FileSize
classes-must-be-in-arrays-2545976-9.patch799 bytes

formatC_vt gboudrias> about this bug #2545976 i think http://cgit.drupalcode.org/hosting_site_backup_manager/tree/hosting_site... and #n883 should be arrays to
formatC_vt https://api.drupal.org/api/drupal/includes!theme.inc/function/theme_table/7 look examples for rows
gboudrias formatC_vt: I think you're right
gboudrias Could you open an issue just as a reminder?

  • gboudrias committed 0dadceb on 7.x-3.x authored by formatC'vt 
    Issue #2545976 by gboudrias, formatC'vt: Undefined index and array_merge...
gboudrias’s picture

gboudrias CreditAttribution: gboudrias at Praxis Labs Coop commented
Status: Needs review » Fixed

Reviewed and pushed, thanks :)

Status: Fixed » Closed (fixed)

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