Error with old block configdata

Bug #1982520 reported by Dianne Tennent
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Mahara
21.04
Fix Released
High
Unassigned
21.10
Fix Released
High
Unassigned
22.04
Fix Released
High
Unassigned
22.10
Fix Released
High
Dianne Tennent

Bug Description

This error surfaced when creating a new account on a client site based on a template:

[WAR] 57 (blocktype/text/lib.php:437) Undefined index: instructions

Call stack (most recent first):

    log_message("Undefined index: instructions", 8, true, true, "/htdocs/blocktype/text/...", 437) at /htdocs/lib/errors.php:515
    error(8, "Undefined index: instructions", "/htdocs/blocktype/text/...", 437, array(size 7)) at /htdocs/blocktype/text/lib.php:437
    PluginBlocktypeText::rewrite_blockinstance_extra_config(object(View), object(BlockInstance), array(size 3), array(size 0)) at /htdocs/lib/mahara.php:1914
    call_static_method("PluginBlocktypeText", "rewrite_blockinstance_extra_config", object(View), object(BlockInstance), array(size 3), array(size 0)) at /htdocs/blocktype/lib.php:2709
    BlockInstance->copy(object(View), object(View), array(size 0)) at /htdocs/lib/view.php:6298
    View->copy_contents(object(View), array(size 0)) at /htdocs/lib/view.php:497
    View::create_from_template(array(size 4), "8", 27590, false, false, array(size 0)) at /htdocs/auth/user.php:1681
    User->copy_views(array(size 8), false) at /htdocs/auth/user.php:1783
    User->copy_site_views_collections_to_new_user() at /htdocs/lib/user.php:2777
    create_user(object(stdClass), array(size 0), object(Institution), null) at /htdocs/auth/lib.php:2310
    login_submit(object(Pieform), array(size 5)) at /htdocs/lib/pieforms/pieform.php:577
    Pieform->__construct(array(size 10)) at /htdocs/lib/mahara.php:5733
    pieform_instance(array(size 10)) at /htdocs/auth/lib.php:600
    auth_setup() at /htdocs/init.php:424
    require("/htdocs/init.php") at /htdocs/index.php:16

Happens when copying old blocks that don't have $configdata['instructions']

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review
Changed in mahara:
status: New → Confirmed
assignee: nobody → Dianne Tennent (dianne-t)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/c/mahara/+/13008
Committed: https://git.mahara.org/mahara/mahara/commit/1f32844b81c17adef8ee80c58b3603010099bef3
Submitter: "Robert Lyon <email address hidden>"
Branch: main

commit 1f32844b81c17adef8ee80c58b3603010099bef3
Author: Dianne Tennent <email address hidden>
Date: Fri Jul 22 11:40:07 2022 +1200

Bug 1982520: Error with old block configdata

Ensure $configdata['instructions'] exists before trying to rewrite it

Change-Id: I21f9643a778ccd80f52910b27f711d287fce005b

Robert Lyon (robertl-9)
Changed in mahara:
status: Confirmed → Fix Committed
milestone: none → 22.10.0
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "22.04_DEV" branch: https://reviews.mahara.org/c/mahara/+/13020

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "21.10_DEV" branch: https://reviews.mahara.org/c/mahara/+/13021

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "21.04_DEV" branch: https://reviews.mahara.org/c/mahara/+/13022

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/c/mahara/+/13020
Committed: https://git.mahara.org/mahara/mahara/commit/d5264fe50825bbe27416973d2874604a3014bafd
Submitter: "Robert Lyon <email address hidden>"
Branch: 22.04_DEV

commit d5264fe50825bbe27416973d2874604a3014bafd
Author: Dianne Tennent <email address hidden>
Date: Fri Jul 22 11:40:07 2022 +1200

Bug 1982520: Error with old block configdata

Ensure $configdata['instructions'] exists before trying to rewrite it

Change-Id: I21f9643a778ccd80f52910b27f711d287fce005b
(cherry picked from commit 1f32844b81c17adef8ee80c58b3603010099bef3)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/c/mahara/+/13021
Committed: https://git.mahara.org/mahara/mahara/commit/7f622a42d9c132eb1eaed9ea118328c2ac04b52d
Submitter: "Robert Lyon <email address hidden>"
Branch: 21.10_DEV

commit 7f622a42d9c132eb1eaed9ea118328c2ac04b52d
Author: Dianne Tennent <email address hidden>
Date: Fri Jul 22 11:40:07 2022 +1200

Bug 1982520: Error with old block configdata

Ensure $configdata['instructions'] exists before trying to rewrite it

Change-Id: I21f9643a778ccd80f52910b27f711d287fce005b
(cherry picked from commit 1f32844b81c17adef8ee80c58b3603010099bef3)

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/c/mahara/+/13022
Committed: https://git.mahara.org/mahara/mahara/commit/7ff7755bdbe3c139203790ce57ccf38e5e15d0b6
Submitter: "Robert Lyon <email address hidden>"
Branch: 21.04_DEV

commit 7ff7755bdbe3c139203790ce57ccf38e5e15d0b6
Author: Dianne Tennent <email address hidden>
Date: Fri Jul 22 11:40:07 2022 +1200

Bug 1982520: Error with old block configdata

Ensure $configdata['instructions'] exists before trying to rewrite it

Change-Id: I21f9643a778ccd80f52910b27f711d287fce005b
(cherry picked from commit 1f32844b81c17adef8ee80c58b3603010099bef3)

no longer affects: mahara
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.