Skip to content
This repository has been archived by the owner on Feb 10, 2024. It is now read-only.

Fixes empty control exception when a document type is not saved after… #17

Merged
merged 1 commit into from Oct 22, 2015

Conversation

danlister
Copy link

… adding a cell

Steps to reproduce exception:

  • Choose a layout
  • Add an element
  • Select any DTGE element
  • Choose the Cancel button

A layout should then contain an empty cell which holds no data due to the document type instance not being saved. When rendering the grid on a page template, the following change is required as the docTypeAlias parameter is empty.

I'm not sure whether the issue is related to the grid or the DTGE but the change proposed appears to fix the issue.

… adding a cell

Steps to reproduce exception:

- Choose a layout
- Add an element
- Select any DTGE element
- Choose the Cancel button

A layout should then contain an empty cell which holds no data due to the document type instance not being saved. When rendering the grid on a page template, the following change is required as the `docTypeAlias` parameter is empty.

I'm not sure whether the issue is related to the grid or the DTGE but the change proposed appears to fix the issue.
@mattbrailsford
Copy link
Collaborator

Good find Dan, thanks for reporting this.

I need to make some changes to DTGE which will likely happen after CG now, but when I do those updates, I'll try this out aswell.

Thanks again

Matt

@danlister
Copy link
Author

Thanks Matt 👍

leekelleher added a commit that referenced this pull request Oct 22, 2015
Fixes empty control exception when a document type is not saved after…
@leekelleher leekelleher merged commit 5aacebb into skttl:develop Oct 22, 2015
@leekelleher
Copy link
Collaborator

Finally merging the PR in, thanks @danlister - sorry it's taken so long!
We'd neglected DTGE, focus has been more on Ditto recently.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
3 participants