Issue from new Install, not from upgrade.

Condition:
- All Domain modules are enabled.
- A content type of 'page' has been created in 7.x, similar to 'page' in 6.x

Problem: After adding a new page (node), with the following settings selected:
- send to all affiliate
- published to the only affiliate so far created
- source domain is the only affiliate

The following error is generated on the next/confirmation screen (where 'n' is the node #):

Notice: Undefined index: 'n' in domain_get_node_domains() (line 1654 of /var/www/html/sites/all/modules/domain/domain.module).

In my domain.module file, line 1654 is:
return $lookup[$nodes];

CommentFileSizeAuthor
#10 1080840.gineta-index.patch937 bytesagentrickard
#9 1080840.gineta-index.patch809 bytesagentrickard
#2 1080840.ununpentium.index_.patch354 bytesagentrickard

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Active » Postponed (maintainer needs more info)

What "next/confirmation" screen?

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new 1080840.ununpentium.index_.patch354 bytes

I replicated this by simply saving the node. The attached should fix the error.

This fixes the notice, but I noticed a nasty problem with the overlay. #1081468: Redirects on save fail with Overlay.

ununpentium’s picture

ununpentium commented

Confirmed error fix. Thanks.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Needs review » Reviewed & tested by the community

Awesome.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Reviewed & tested by the community » Fixed

Fixed.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

This problem does not exist in 6.x.

gineta’s picture

gineta commented
Status: Fixed » Needs review

Sorry this path not work for me. Any other ideas?

Notice: Undefined index: 4 in domain_node_load() (line 1461 of /mnt/var/www/vhosts/gigalearn/httpdocs/sites/all/modules/domain/domain.module).
Notice: Undefined index: 4 in domain_node_load() (line 1462 of /mnt/var/www/vhosts/gigalearn/httpdocs/sites/all/modules/domain/domain.module).

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Needs review » Postponed (maintainer needs more info)

What are you doing when this error occurs? Please explain.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new 1080840.gineta-index.patch809 bytes

That second error is different, and theoretically should not occur, but we can account for it here.

Please review so we can get a bugfix release out tomorrow.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Needs review » Fixed
StatusFileSize
new 1080840.gineta-index.patch937 bytes

Slight revision. Tests fine. Fixed.

Status: Fixed » Closed (fixed)

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