Closed (fixed)
Project:
Project issue tracking
Version:
7.x-2.x-dev
Component:
Issues
Priority:
Major
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
26 Apr 2013 at 20:29 UTC
Updated:
4 Jan 2014 at 03:28 UTC
Jump to comment: Most recent
Comments
Comment #1
dwwConfirmed locally. Only happens on the issue node type, not for example, projects.
I suspected nodechanges was to blame, but I removed that field from the comment bundle and it was still broken.
Don't have time to further investigate now, leaving unassigned.
Comment #2
dwwSpent some quality time on the flight back to the US to get to the bottom of this. Basically, I started from scratch with a second issue node type, and kept configuring it more and more like the default project_issue we ship until I found what was breaking preview. Turns out that once you add field_project, preview dies. I then tried to figure out why. To make a long story short, the problem is here:
Node previews in D7 are still implemented using
$form['#prefix'], so this code is clobbering the preview. I pushed a fix that preserves the existing #prefix (and #suffix, just in case), and now the 'Preview' and 'View changes' buttons are working fine. It even works to change the project once you've previewed and everything still behaves as expected. ;)http://drupalcode.org/project/project_issue.git/commit/48bbe0d
Now merged into bzr. Should be live on git7 soon.
Comment #3
tvn commentedConfirmed, works on git7. Though preview shows huge 'updated this issue' button.
Comment #4
dwwThe huge button is because of #1977594: Issue page: sidebar. I don't think I merged drupalorg into bzr, so the latest update there to configure these might not have happened or something. Anyway, we can clean that up either at #1977594 or #1970046: Properly configure issue fields on rebuild.