Follow up for #1986140: User 1 profile Access Denied (view and edit) -> Warning Illegal offset type in isset or empty

Problem/Motivation

(why the issue was filed, steps to reproduce the problem, etc.)

Proposed resolution

(description of the proposed solution, the rationale behind it, and workarounds for people who cannot use the patch)

Remaining tasks

(reviews needed, tests to be written or run, documentation to be written, etc.)

User interface changes

(new or changed features/functionality in the user interface, modules added or removed, changes to URL paths, changes to user interface text)

API changes

(API changes/additions that would affect module, install profile, and theme developers, including examples of before/after code if appropriate)

Original report by [username]

// Text of original report here.
(for legacy issues whose initial post was not the issue summary)

Comments

rootwork’s picture

Assigned: Unassigned » rootwork

at Portland Drupalcon

rootwork’s picture

StatusFileSize
new827 bytes
rootwork’s picture

Component: layout.module » tracker.module
Status: Active » Needs review
phl3tch’s picture

You've still got the underscore in there. Should be tracker_check_user_access.

phl3tch’s picture

Status: Needs review » Needs work
rootwork’s picture

Status: Needs work » Needs review
StatusFileSize
new1 KB
new825 bytes

Oops, sorry about that. Here's a fixed version.

I went ahead and did an interdiff, even though it's so minor, just so I get practice at doing that.

Status: Needs review » Needs work

The last submitted patch, 2003432-5-rename-tracker_user_access.patch, failed testing.

berdir’s picture

This looks like a normal task to me, not a critical bug?

swentel’s picture

Category: bug » task
Priority: Critical » Normal

Yeah, pretty normal indeed.

phl3tch’s picture

I think the critical designation carried over from a similar bug in the layout module, which was something of a showstopper when viewing user pages.

rootwork’s picture

Status: Needs work » Needs review
Issue tags: -Needs issue summary update, -Novice

Status: Needs review » Needs work

The last submitted patch, 2003432-5-rename-tracker_user_access.patch, failed testing.

rootwork’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 2003432-5-rename-tracker_user_access.patch, failed testing.

rootwork’s picture

Status: Needs work » Needs review
Issue tags: +Needs issue summary update, +Novice
alexpott’s picture

Status: Needs review » Closed (duplicate)

This will be fixed by the conversion to the new route system... #1972990: Convert tracker_page() to a Controller