Can't uses B2B Customer Entity as target for Opportunity Account Field

This topic contains 3 replies, has 2 voices, and was last updated by Martin 7 years ago.

Starting from March 1, 2020 the forum has been switched to the read-only mode. Please head to StackOverflow for support.

Creator
Topic

April 3, 2017 at 8:42 am #27631

Martin

Participant

Hey,

we’ve discovered a possible bug within the creation of a new Opportunity.

How to reproduce:
1) Create new Opportunity from scratch.
2) On Account click hamburger bar, choose Business Customer as Entity Type from top left menu
3) Select Any of the B2B customers
4) Dialog closes

Also, when on a B2B Customers page, by clicking the ‘new opportunity’ button in the more actions menu, same thing happens but freezes immediately on the edit page.

what happens:
Field stays empty (no name displayed). anyway, after saving, the b2b customer is successfully set in the view page.

But: When you re-edit the opportunity, the edit page stays in loading state and can’t be edited.

Developer tools show the following error (js related):

Uncaught TypeError: Cannot read property ‘type’ of null
at eval (eval at b.template (oro.min.js?version=7f1d6e33:1486), <anonymous>:8:10)
at a (oro.min.js?version=7f1d6e33:1486)
at Object.formatSelection (oro.min.js?version=7f1d6e33:1520)
at n.e.updateSelection (oro.min.js?version=7f1d6e33:1505)
at oro.min.js?version=7f1d6e33:1505
at n (oro.min.js?version=7f1d6e33:1520)
at Object.success (oro.min.js?version=7f1d6e33:1520)
at c (oro.min.js?version=7f1d6e33:1484)
at Object.fireWith [as resolveWith] (oro.min.js?version=7f1d6e33:1484)
at n (oro.min.js?version=7f1d6e33:1485)

which relates to the follwing part of code (this was cut down, to identify the portion of the script)

i.isInViewPort() && i.setCursorToEnd().focus()
}
}, isInViewPort: function() {
var t = e(this),
i = t.offset().top,
n = i + t.height(),
o = e(window).scrollTop(),
s = o + e(window).height();
return i >= o && n <= s
}, getBorders: function(e) {
function t(e) {
switch (e) {
    case "thin":
        return 1;
    case "medium":
        return 2;
    case "thick":
        return 5;
    default:
        return Math.round(parseFloat(e))
}
}
var i = window.getComputedStyle(e || this[0], null);
return {
top: t(i.getPropertyValue("borderTopWidth") || i.borderTopWidth),
bottom: t(i.getPropertyValue("borderBottomWidth") || i.borderBottomWidth),
left: t(i.getPropertyValue("borderLeftWidth") || i.borderLeftWidth),
right: t(i.getPropertyValue("borderRightWidth") || i.borderRightWidth)
}
}, insertAtCursor: function(e) {
return this.each(function() {
var t, i, n = this,
    o = n.value;
if ("selectionStart" in n) try {
    t = n.selectionStart, i = n.selectionEnd, n.value = o.substr(0, t) + e + o.substr(i), n.selectionEnd = n.selectionStart = t + e.length
} catch (t) {
    n.value += e
} else n.value += e
})
}, bindFirst: function(t, i, n, o) {
var s = t.indexOf("."),
r = s > 0 ? t.substring(s) : "";
return t = s > 0 ? t.substring(0, s) : t, o = void 0 !== o ? o : void 0 !== n ? n : i, n = "function" != typeof n ? n : void 0, i = "function" != typeof i ? i : void 0, this.each(function() {
var s = e(this),
    a = this["on" + t];
a && (s.bind(t, function(e) {
    return a(e.originalEvent)
}), this["on" + t] = null), s.on(t + r, i, n, o);
var l = s.data("events") || e._data(s[0], "events"),
    c = l[t],
    d = c.pop();
c.unshift(d)
})
}, addClassTemporarily: function(t, i) {
return i = i || 0, this.each(function() {
var n = e(this);
n.addClass(t), setTimeout(function() {
    n.removeClass(t)
}, i)
})
}, formFieldValues: function(t) {
var i = this.find(":input").get();
return 0 === arguments.length ? (t = {}, e.each(i, function() {
this.name && !this.disabled && (this.checked || /select|textarea/i.test(this.nodeName) || /text|hidden|password/i.test(this.type)) && (void 0 === t[this.name] && (t[this.name] = []), t[this.name].push(e(this).val()))
}), t) : (e.each(i, function() {
if (this.name && t[this.name]) {
    var i = t[this.name],
        n = e(this);
    if ("[object Array]" !== Object.prototype.toString.call(i) && (i = [i]), "checkbox" === this.type || "radio" === this.type) {
        for (var o = n.val(), s = !1, r = 0; r < i.length; r++)
            if (i[r] === o) {
                s = !0;
                break
            }
        n.attr("checked", s)
    } else n.val(i[0])
}
}), this)
}
}), e
}),
function() {
    function e(e) {
        function t(t, i, n, o, s, r) {
            for (; s >= 0 && s < r; s += e) {
                var a = o ? o[s] : s;
                n = i(n, t[a], a, t)
            }
            return n
        }
        return function(i, n, o, s) {
            n = y(n, s, 4);
            var r = !A(i) && b.keys(i),
                a = (r || i).length,
                l = e > 0 ? 0 : a - 1;
            return arguments.length < 3 && (o = i[r ? r[l] : l], l += e), t(i, n, o, r, l, a)
        }
    }

    function t(e) {
        return function(t, i, n) {
            i = _(i, n);
            for (var o = j(t), s = e > 0 ? 0 : o - 1; s >= 0 && s < o; s += e)
                if (i(t[s], s, t)) return s;
            return -1
        }
    }

    function i(e, t, i) {
        return function(n, o, s) {
            var r = 0,
                a = j(n);
            if ("number" == typeof s) e > 0 ? r = s >= 0 ? s : Math.max(s + a, r) : a = s >= 0 ? Math.min(s + 1, a) : s + a + 1;
            else if (i && s && a) return s = i(n, o), n[s] === o ? s : -1;
            if (o !== o) return s = t(d.call(n, r, a), b.isNaN), s >= 0 ? s + r : -1;
            for (s = e > 0 ? r : a - 1; s >= 0 && s < a; s += e)
                if (n[s] === o) return s;
            return -1
        }
    }

    function n(e, t) {
        var i = O.length,
            n = e.constructor,
            o = b.isFunction(n) && n.prototype || a,
            s = "constructor";
        for (b.has(e, s) && !b.contains(t, s) && t.push(s); i--;) s = O[i], s in e && e[s] !== o[s] && !b.contains(t, s) && t.push(s)
    }
    var o = this,
        s = o._,
        r = Array.prototype,
        a = Object.prototype,
        l = Function.prototype,
        c = r.push,
        d = r.slice,
        u = a.toString,
        h = a.hasOwnProperty,
        p = Array.isArray,
        f = Object.keys,
        m = l.bind,
        g = Object.create,
        v = function() {},
        b = function(e) {
            return e instanceof b ? e : this instanceof b ? void(this._wrapped = e) : new b(e)
        };
    "undefined" != typeof exports ? ("undefined" != typeof module && module.exports && (exports = module.exports = b), exports._ = b) : o._ = b, b.VERSION = "1.8.3";
    var y = function(e, t, i) {
            if (void 0 === t) return e;
            switch (null == i ? 3 : i) {
                case 1:
                    return function(i) {
                        return e.call(t, i)
                    };
                case 2:
                    return function(i, n) {
                        return e.call(t, i, n)
                    };
                case 3:
                    return function(i, n, o) {
                        return e.call(t, i, n, o)
                    };
                case 4:
                    return function(i, n, o, s) {
                        return e.call(t, i, n, o, s)
                    }
            }
            return function() {
                return e.apply(t, arguments)
            }
        },
        _ = function(e, t, i) {
            return null == e ? b.identity : b.isFunction(e) ? y(e, t, i) : b.isObject(e) ? b.matcher(e) : b.property(e)
        };
    b.iteratee = function(e, t) {
        return _(e, t, 1 / 0)
    };

100

101

102

103

104

105

106

107

108

109

110

111

112

113

114

115

116

117

118

119

120

121

122

123

124

125

126

127

128

129

130

131

132

133

134

135

136

137

138

139

140

141

142

143

144

145

146

147

148

149

150

151

152

153

154

155

156

157

158

159

160

161

162

163

164

165

166

167

168

169

170

171

172

173

174

175

176

i.isInViewPort() && i.setCursorToEnd().focus()

}

}, isInViewPort: function() {

var t = e(this),

i = t.offset().top,

n = i + t.height(),

o = e(window).scrollTop(),

s = o + e(window).height();

return i >= o && n <= s

}, getBorders: function(e) {

function t(e) {

switch (e) {

case "thin":

return 1;

case "medium":

return 2;

case "thick":

return 5;

default:

return Math.round(parseFloat(e))

}

var i = window.getComputedStyle(e || this[0], null);

return {

top: t(i.getPropertyValue("borderTopWidth") || i.borderTopWidth),

bottom: t(i.getPropertyValue("borderBottomWidth") || i.borderBottomWidth),

left: t(i.getPropertyValue("borderLeftWidth") || i.borderLeftWidth),

right: t(i.getPropertyValue("borderRightWidth") || i.borderRightWidth)

}

}, insertAtCursor: function(e) {

return this.each(function() {

var t, i, n = this,

o = n.value;

if ("selectionStart" in n) try {

t = n.selectionStart, i = n.selectionEnd, n.value = o.substr(0, t) + e + o.substr(i), n.selectionEnd = n.selectionStart = t + e.length

} catch (t) {

n.value += e

} else n.value += e

})

}, bindFirst: function(t, i, n, o) {

var s = t.indexOf("."),

r = s > 0 ? t.substring(s) : "";

return t = s > 0 ? t.substring(0, s) : t, o = void 0 !== o ? o : void 0 !== n ? n : i, n = "function" != typeof n ? n : void 0, i = "function" != typeof i ? i : void 0, this.each(function() {

var s = e(this),

a = this["on" + t];

a && (s.bind(t, function(e) {

return a(e.originalEvent)

}), this["on" + t] = null), s.on(t + r, i, n, o);

var l = s.data("events") || e._data(s[0], "events"),

c = l[t],

d = c.pop();

c.unshift(d)

})

}, addClassTemporarily: function(t, i) {

return i = i || 0, this.each(function() {

var n = e(this);

n.addClass(t), setTimeout(function() {

n.removeClass(t)

}, i)

})

}, formFieldValues: function(t) {

var i = this.find(":input").get();

return 0 === arguments.length ? (t = {}, e.each(i, function() {

this.name && !this.disabled && (this.checked || /select|textarea/i.test(this.nodeName) || /text|hidden|password/i.test(this.type)) && (void 0 === t[this.name] && (t[this.name] = []), t[this.name].push(e(this).val()))

}), t) : (e.each(i, function() {

if (this.name && t[this.name]) {

var i = t[this.name],

n = e(this);

if ("[object Array]" !== Object.prototype.toString.call(i) && (i = [i]), "checkbox" === this.type || "radio" === this.type) {

for (var o = n.val(), s = !1, r = 0; r < i.length; r++)

if (i[r] === o) {

s = !0;

break

}

n.attr("checked", s)

} else n.val(i[0])

}

}), this)

}

}), e

}),

function() {

function e(e) {

function t(t, i, n, o, s, r) {

for (; s >= 0 && s < r; s += e) {

var a = o ? o[s] : s;

n = i(n, t[a], a, t)

}

return n

}

return function(i, n, o, s) {

n = y(n, s, 4);

var r = !A(i) && b.keys(i),

a = (r || i).length,

l = e > 0 ? 0 : a - 1;

return arguments.length < 3 && (o = i[r ? r[l] : l], l += e), t(i, n, o, r, l, a)

}

function t(e) {

return function(t, i, n) {

i = _(i, n);

for (var o = j(t), s = e > 0 ? 0 : o - 1; s >= 0 && s < o; s += e)

if (i(t[s], s, t)) return s;

return -1

}

function i(e, t, i) {

return function(n, o, s) {

var r = 0,

a = j(n);

if ("number" == typeof s) e > 0 ? r = s >= 0 ? s : Math.max(s + a, r) : a = s >= 0 ? Math.min(s + 1, a) : s + a + 1;

else if (i && s && a) return s = i(n, o), n[s] === o ? s : -1;

if (o !== o) return s = t(d.call(n, r, a), b.isNaN), s >= 0 ? s + r : -1;

for (s = e > 0 ? r : a - 1; s >= 0 && s < a; s += e)

if (n[s] === o) return s;

return -1

}

function n(e, t) {

var i = O.length,

n = e.constructor,

o = b.isFunction(n) && n.prototype || a,

s = "constructor";

for (b.has(e, s) && !b.contains(t, s) && t.push(s); i--;) s = O[i], s in e && e[s] !== o[s] && !b.contains(t, s) && t.push(s)

}

var o = this,

s = o._,

r = Array.prototype,

a = Object.prototype,

l = Function.prototype,

c = r.push,

d = r.slice,

u = a.toString,

h = a.hasOwnProperty,

p = Array.isArray,

f = Object.keys,

m = l.bind,

g = Object.create,

v = function() {},

b = function(e) {

return e instanceof b ? e : this instanceof b ? void(this._wrapped = e) : new b(e)

};

"undefined" != typeof exports ? ("undefined" != typeof module && module.exports && (exports = module.exports = b), exports._ = b) : o._ = b, b.VERSION = "1.8.3";

var y = function(e, t, i) {

if (void 0 === t) return e;

switch (null == i ? 3 : i) {

case 1:

return function(i) {

return e.call(t, i)

};

case 2:

return function(i, n) {

return e.call(t, i, n)

};

case 3:

return function(i, n, o) {

return e.call(t, i, n, o)

};

case 4:

return function(i, n, o, s) {

return e.call(t, i, n, o, s)

}

return function() {

return e.apply(t, arguments)

}

_ = function(e, t, i) {

return null == e ? b.identity : b.isFunction(e) ? y(e, t, i) : b.isObject(e) ? b.matcher(e) : b.property(e)

};

b.iteratee = function(e, t) {

return _(e, t, 1 / 0)

};

Neither Dev nor production log throw any errors, so i think it relates to a gui bug?

Is this Issue known or is there a workaround already?

Kindly Regards
Martin

Creator
Topic

Viewing 3 replies - 1 through 3 (of 3 total)

Author
Replies
April 4, 2017 at 1:50 am #27632

Martin
Participant
Okay, quite easy fix. The Problem happens, when no Icon is choosen for the B2B Customer Entity within the Channel definition. When this is null, the Select Box thwows the error above.
Adding a Icon resolves this issues, and the entity can be used.
Maybe this should be fixed though because it’s not easy to track down.
Kindly Regards
Martin
April 4, 2017 at 11:59 am #27633

msulima
Moderator
Hi, @Martin.
Thank you for reporting the issue! I was able to reproduce the issue. We will fix it in next release.
We greatly appreciate all contributions to Oro projects.
Check our free technical guide ”How to Build a Reliable and Secure Infrastructure for OroCommerce”.
April 5, 2017 at 3:10 am #27634

Martin
Participant
Hey,
i’ll then mark this topic as closed.
Kindly regards
Martin
Author
Replies

Viewing 3 replies - 1 through 3 (of 3 total)

The forum ‘OroCRM – Installation/Technical Issues or Problems’ is closed to new topics and replies.

OroCRM Forums