Bug 12036

Summary:	Neworderempty.pl fails to add items when itemrecorddisplay is too long (acqCreateItem=ordering)
Product:	Koha	Reporter:	Marcel de Rooy <m.de.rooy>
Component:	Acquisitions	Assignee:	Bugs List <koha-bugs>
Status:	CLOSED WORKSFORME	QA Contact:	Testopia <testopia>
Severity:	normal
Priority:	P5 - low	CC:	jonathan.druart
Version:	3.14
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	---
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:
Circulation function:

Description Marcel de Rooy 2014-04-03 11:45:30 UTC

In 3.14.4 I encounter this problem in Acquisition, neworderempty.pl:

The output of PrepareItemrecordDisplay (called by itemrecorddisplay.pl in additem.js [cloneItemBlock]) seems to be too long.
The line $(clone).append(data) in the ajax success function in cloneItemBlock fails silently.
This makes it impossible to add items to a new empty order when AcqCreateItem=ordering.

Did anyone else see that already or solved it? 
(I created the ACQ framework by just copying from Default.)
On a clean master I could not reproduce this problem, but it seemed that the output of PrepareItemrecordDisplay was around 23000 there instead of 27000 chars.

Comment 1 Marcel de Rooy 2014-04-03 12:44:47 UTC

From IRC:

Joubu	marcelr: It works for meI added some subfields to my ACQ framework. itemrecorddisplay.pl?frameworkcode=ACQ returns 38098 chars and the form is correctly displayed

Comment 2 Jonathan Druart 2015-04-28 08:21:32 UTC

Marcel, do you still have this issue?

Comment 3 Marcel de Rooy 2015-05-04 09:38:53 UTC

(In reply to Jonathan Druart from comment #2)
> Marcel, do you still have this issue?

Normally, I do not have AcqCreateItem set to ordering. So I did not see it recently. But no proof of a solution yet :)

Comment 4 Jonathan Druart 2016-08-16 14:43:52 UTC

Marked as resolved, please reopen if the issue still exists.