Unable to create dashboards on latest revision of trunk.

Hello,

We are on the latest revision(18220) of dhis-trunk. We are facing issues creating dashboard.

We tried debugging the code and had seen that method with signature:

List getByUid( Collection uids ) [Line no: 436]

is throwing an exception while returning a list.

That is, return getSharingCriteria().add( Restrictions.in( “uid”, uids ) ).list()

is throwing an exception when list() method is called on getSharingCriteria().add( Restrictions.in( “uid”, uids ) ) .

Request:

Url: http://localhost:8080/api/dashboards

Method: POST

Payload:

{

“displayName”: “AFGHANISTAN”,

“name”: “AFGHANISTAN”,

“dashboardItems”: [

{

“chart”: {

“id”: “chartId”

}

}

],

“publicAccess”: “--------”,

“access”: {

“read”: true,

“manage”: true,

“update”: true,

“externalize”: true,

“write”: true,

“delete”: true

},

“externalAccess”: false,

“userGroupAccesses”: [],

“id”: “dashboardId”

}

Can you look into the issue?

···

Regards,

Mansi Singhal

This is probably related to the other TW bug reported on this list? If dashboards don’t have UIDs, it will fail.

If its not, can you at least confirm that dashboard table contains the UID column?

···


Morten

On Wed, Feb 11, 2015 at 1:27 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hello,

We are on the latest revision(18220) of dhis-trunk. We are facing issues creating dashboard.

We tried debugging the code and had seen that method with signature:

List getByUid( Collection uids ) [Line no: 436]

is throwing an exception while returning a list.

That is, return getSharingCriteria().add( Restrictions.in( “uid”, uids ) ).list()

is throwing an exception when list() method is called on getSharingCriteria().add( Restrictions.in( “uid”, uids ) ) .

Request:

Url: http://localhost:8080/api/dashboards

Method: POST

Payload:

{

“displayName”: “AFGHANISTAN”,

“name”: “AFGHANISTAN”,

“dashboardItems”: [

{

“chart”: {

“id”: “chartId”

}

}

],

“publicAccess”: “--------”,

“access”: {

“read”: true,

“manage”: true,

“update”: true,

“externalize”: true,

“write”: true,

“delete”: true

},

“externalAccess”: false,

“userGroupAccesses”: [],

“id”: “dashboardId”

}

Can you look into the issue?


Regards,

Mansi Singhal


Mailing list: https://launchpad.net/~dhis2-devs

Post to : dhis2-devs@lists.launchpad.net

Unsubscribe : https://launchpad.net/~dhis2-devs

More help : https://help.launchpad.net/ListHelp

Hey Morten,

Dashboard table contains the UID column. This is something else. Can you please look into the issue.

···

On Wed, Feb 11, 2015 at 1:05 PM, Morten Olav Hansen mortenoh@gmail.com wrote:

This is probably related to the other TW bug reported on this list? If dashboards don’t have UIDs, it will fail.

If its not, can you at least confirm that dashboard table contains the UID column?


Morten

On Wed, Feb 11, 2015 at 1:27 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hello,

We are on the latest revision(18220) of dhis-trunk. We are facing issues creating dashboard.

We tried debugging the code and had seen that method with signature:

List getByUid( Collection uids ) [Line no: 436]

is throwing an exception while returning a list.

That is, return getSharingCriteria().add( Restrictions.in( “uid”, uids ) ).list()

is throwing an exception when list() method is called on getSharingCriteria().add( Restrictions.in( “uid”, uids ) ) .

Request:

Url: http://localhost:8080/api/dashboards

Method: POST

Payload:

{

“displayName”: “AFGHANISTAN”,

“name”: “AFGHANISTAN”,

“dashboardItems”: [

{

“chart”: {

“id”: “chartId”

}

}

],

“publicAccess”: “--------”,

“access”: {

“read”: true,

“manage”: true,

“update”: true,

“externalize”: true,

“write”: true,

“delete”: true

},

“externalAccess”: false,

“userGroupAccesses”: [],

“id”: “dashboardId”

}

Can you look into the issue?


Regards,

Mansi Singhal


Mailing list: https://launchpad.net/~dhis2-devs

Post to : dhis2-devs@lists.launchpad.net

Unsubscribe : https://launchpad.net/~dhis2-devs

More help : https://help.launchpad.net/ListHelp

Regards,

Mansi Singhal

PFA the snapshot of dashboard table.

···

On Wed, Feb 11, 2015 at 1:51 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hey Morten,

Dashboard table contains the UID column. This is something else. Can you please look into the issue.

On Wed, Feb 11, 2015 at 1:05 PM, Morten Olav Hansen mortenoh@gmail.com wrote:

This is probably related to the other TW bug reported on this list? If dashboards don’t have UIDs, it will fail.

If its not, can you at least confirm that dashboard table contains the UID column?

Regards,

Mansi Singhal


Morten

On Wed, Feb 11, 2015 at 1:27 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hello,

We are on the latest revision(18220) of dhis-trunk. We are facing issues creating dashboard.

We tried debugging the code and had seen that method with signature:

List getByUid( Collection uids ) [Line no: 436]

is throwing an exception while returning a list.

That is, return getSharingCriteria().add( Restrictions.in( “uid”, uids ) ).list()

is throwing an exception when list() method is called on getSharingCriteria().add( Restrictions.in( “uid”, uids ) ) .

Request:

Url: http://localhost:8080/api/dashboards

Method: POST

Payload:

{

“displayName”: “AFGHANISTAN”,

“name”: “AFGHANISTAN”,

“dashboardItems”: [

{

“chart”: {

“id”: “chartId”

}

}

],

“publicAccess”: “--------”,

“access”: {

“read”: true,

“manage”: true,

“update”: true,

“externalize”: true,

“write”: true,

“delete”: true

},

“externalAccess”: false,

“userGroupAccesses”: [],

“id”: “dashboardId”

}

Can you look into the issue?


Regards,

Mansi Singhal


Mailing list: https://launchpad.net/~dhis2-devs

Post to : dhis2-devs@lists.launchpad.net

Unsubscribe : https://launchpad.net/~dhis2-devs

More help : https://help.launchpad.net/ListHelp

Regards,

Mansi Singhal

Is it only through the web-api you see the issue? I can create a new one through the UI just fine in the latest revision (on a empty db).

I will test it with the web-api soon.

···


Morten

On Wed, Feb 11, 2015 at 3:23 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

PFA the snapshot of dashboard table.

On Wed, Feb 11, 2015 at 1:51 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hey Morten,

Dashboard table contains the UID column. This is something else. Can you please look into the issue.

Regards,

Mansi Singhal

On Wed, Feb 11, 2015 at 1:05 PM, Morten Olav Hansen mortenoh@gmail.com wrote:

This is probably related to the other TW bug reported on this list? If dashboards don’t have UIDs, it will fail.

If its not, can you at least confirm that dashboard table contains the UID column?

Regards,

Mansi Singhal


Morten

On Wed, Feb 11, 2015 at 1:27 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hello,

We are on the latest revision(18220) of dhis-trunk. We are facing issues creating dashboard.

We tried debugging the code and had seen that method with signature:

List getByUid( Collection uids ) [Line no: 436]

is throwing an exception while returning a list.

That is, return getSharingCriteria().add( Restrictions.in( “uid”, uids ) ).list()

is throwing an exception when list() method is called on getSharingCriteria().add( Restrictions.in( “uid”, uids ) ) .

Request:

Url: http://localhost:8080/api/dashboards

Method: POST

Payload:

{

“displayName”: “AFGHANISTAN”,

“name”: “AFGHANISTAN”,

“dashboardItems”: [

{

“chart”: {

“id”: “chartId”

}

}

],

“publicAccess”: “--------”,

“access”: {

“read”: true,

“manage”: true,

“update”: true,

“externalize”: true,

“write”: true,

“delete”: true

},

“externalAccess”: false,

“userGroupAccesses”: [],

“id”: “dashboardId”

}

Can you look into the issue?


Regards,

Mansi Singhal


Mailing list: https://launchpad.net/~dhis2-devs

Post to : dhis2-devs@lists.launchpad.net

Unsubscribe : https://launchpad.net/~dhis2-devs

More help : https://help.launchpad.net/ListHelp

Yeah Yeah. We didn’t check on the UI. We have seen it only for web-api.

And the issue is coming only if payload contains “dashboardItems”.

···

On Wed, Feb 11, 2015 at 1:55 PM, Morten Olav Hansen mortenoh@gmail.com wrote:

Is it only through the web-api you see the issue? I can create a new one through the UI just fine in the latest revision (on a empty db).

I will test it with the web-api soon.


Morten

On Wed, Feb 11, 2015 at 3:23 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

PFA the snapshot of dashboard table.

On Wed, Feb 11, 2015 at 1:51 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hey Morten,

Dashboard table contains the UID column. This is something else. Can you please look into the issue.

Regards,

Mansi Singhal

On Wed, Feb 11, 2015 at 1:05 PM, Morten Olav Hansen mortenoh@gmail.com wrote:

This is probably related to the other TW bug reported on this list? If dashboards don’t have UIDs, it will fail.

If its not, can you at least confirm that dashboard table contains the UID column?

Regards,

Mansi Singhal


Morten

On Wed, Feb 11, 2015 at 1:27 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hello,

We are on the latest revision(18220) of dhis-trunk. We are facing issues creating dashboard.

We tried debugging the code and had seen that method with signature:

List getByUid( Collection uids ) [Line no: 436]

is throwing an exception while returning a list.

That is, return getSharingCriteria().add( Restrictions.in( “uid”, uids ) ).list()

is throwing an exception when list() method is called on getSharingCriteria().add( Restrictions.in( “uid”, uids ) ) .

Request:

Url: http://localhost:8080/api/dashboards

Method: POST

Payload:

{

“displayName”: “AFGHANISTAN”,

“name”: “AFGHANISTAN”,

“dashboardItems”: [

{

“chart”: {

“id”: “chartId”

}

}

],

“publicAccess”: “--------”,

“access”: {

“read”: true,

“manage”: true,

“update”: true,

“externalize”: true,

“write”: true,

“delete”: true

},

“externalAccess”: false,

“userGroupAccesses”: [],

“id”: “dashboardId”

}

Can you look into the issue?


Regards,

Mansi Singhal


Mailing list: https://launchpad.net/~dhis2-devs

Post to : dhis2-devs@lists.launchpad.net

Unsubscribe : https://launchpad.net/~dhis2-devs

More help : https://help.launchpad.net/ListHelp

Regards,

Mansi Singhal

Ok, I will have a look

···

On Wed, Feb 11, 2015 at 3:33 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Yeah Yeah. We didn’t check on the UI. We have seen it only for web-api.

And the issue is coming only if payload contains “dashboardItems”.


Morten

On Wed, Feb 11, 2015 at 1:55 PM, Morten Olav Hansen mortenoh@gmail.com wrote:

Is it only through the web-api you see the issue? I can create a new one through the UI just fine in the latest revision (on a empty db).

I will test it with the web-api soon.

Regards,

Mansi Singhal


Morten

On Wed, Feb 11, 2015 at 3:23 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

PFA the snapshot of dashboard table.

On Wed, Feb 11, 2015 at 1:51 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hey Morten,

Dashboard table contains the UID column. This is something else. Can you please look into the issue.

Regards,

Mansi Singhal

On Wed, Feb 11, 2015 at 1:05 PM, Morten Olav Hansen mortenoh@gmail.com wrote:

This is probably related to the other TW bug reported on this list? If dashboards don’t have UIDs, it will fail.

If its not, can you at least confirm that dashboard table contains the UID column?

Regards,

Mansi Singhal


Morten

On Wed, Feb 11, 2015 at 1:27 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hello,

We are on the latest revision(18220) of dhis-trunk. We are facing issues creating dashboard.

We tried debugging the code and had seen that method with signature:

List getByUid( Collection uids ) [Line no: 436]

is throwing an exception while returning a list.

That is, return getSharingCriteria().add( Restrictions.in( “uid”, uids ) ).list()

is throwing an exception when list() method is called on getSharingCriteria().add( Restrictions.in( “uid”, uids ) ) .

Request:

Url: http://localhost:8080/api/dashboards

Method: POST

Payload:

{

“displayName”: “AFGHANISTAN”,

“name”: “AFGHANISTAN”,

“dashboardItems”: [

{

“chart”: {

“id”: “chartId”

}

}

],

“publicAccess”: “--------”,

“access”: {

“read”: true,

“manage”: true,

“update”: true,

“externalize”: true,

“write”: true,

“delete”: true

},

“externalAccess”: false,

“userGroupAccesses”: [],

“id”: “dashboardId”

}

Can you look into the issue?


Regards,

Mansi Singhal


Mailing list: https://launchpad.net/~dhis2-devs

Post to : dhis2-devs@lists.launchpad.net

Unsubscribe : https://launchpad.net/~dhis2-devs

More help : https://help.launchpad.net/ListHelp

Hi Mansi,

can you also check whether table "dashboarditem" also contains a uid column?

Lars

Hey Lars,

dashboardItem table has uid column. PFA the snapshot for the same.

···

On Wed, Feb 11, 2015 at 2:33 PM, Lars Helge Øverland larshelge@gmail.com wrote:

Hi Mansi,

can you also check whether table “dashboarditem” also contains a uid column?

Lars

Regards,

Mansi Singhal

Hi ,

Did anyone have a chance to take a look at this issue? . We are still facing the same issue on rev no. 18249.

···

On Wed, Feb 11, 2015 at 2:35 PM, Mansi Singhal msinghal@thoughtworks.com wrote:

Hey Lars,

dashboardItem table has uid column. PFA the snapshot for the same.

On Wed, Feb 11, 2015 at 2:33 PM, Lars Helge Øverland larshelge@gmail.com wrote:

Regards,

Mansi Singhal

Hi Mansi,

can you also check whether table “dashboarditem” also contains a uid column?

Lars

Sandesh DoolipetaApplication Developer
Email
sandeshd@thoughtworks.com

Telephone
9686062727

ThoughtWorks

Hi Mansi,

is the request you included in the email the same as you used when
attempting to POST? If you look under dashboardItems > chart > id you have
"chartId", which is not a valid UID - this should be a reference to an
existing chart UID. Try changing that and see if it helps.

{
    "displayName": "AFGHANISTAN",
    "name": "AFGHANISTAN",
    "dashboardItems": [
        {
            "chart": {
                "id": "chartId"
            }
        }
    ],

regards,

Lars

Hi Lars ,

The chart ID in the payload sent to you was changed. When we tried posting in the app , we used a valid chart ID.

···

On Sun, Feb 15, 2015 at 4:53 PM, Lars Helge Øverland larshelge@gmail.com wrote:

Hi Mansi,

is the request you included in the email the same as you used when attempting to POST? If you look under dashboardItems > chart > id you have “chartId”, which is not a valid UID - this should be a reference to an existing chart UID. Try changing that and see if it helps.

{

“displayName”: “AFGHANISTAN”,

“name”: “AFGHANISTAN”,

“dashboardItems”: [

{

“chart”: {

“id”: “chartId”

}

}

],

regards,

Lars

Sandesh DoolipetaApplication Developer
Email
sandeshd@thoughtworks.com

Telephone
9686062727

ThoughtWorks

Hi Sandesh

Was this working before? because this is not really how the API works… normally you would create the dashboard, then add the items…

We do support doing both if you do something like this (untested, but should work)

{

“dashboardItems”: [

{ “id”: “di-client-generated-uid”, “chart”: “chart-id” }

],

“dashboards”: [

{ “id”: “dashboard-id”, “name”: “dashboard-name”, “dashboardItems”: [{“id”: “di-client-generated-id”}] }

]

}

Then send this of to /api/metadata endpoint

···

On Sun, Feb 15, 2015 at 8:58 PM, Sandesh Sukumar Doolipeta sandeshd@thoughtworks.com wrote:

Hi Lars ,

The chart ID in the payload sent to you was changed. When we tried posting in the app , we used a valid chart ID.


Morten

On Sun, Feb 15, 2015 at 4:53 PM, Lars Helge Øverland larshelge@gmail.com wrote:


Sandesh DoolipetaApplication Developer
Email
sandeshd@thoughtworks.com

Telephone
9686062727

ThoughtWorks

Hi Mansi,

is the request you included in the email the same as you used when attempting to POST? If you look under dashboardItems > chart > id you have “chartId”, which is not a valid UID - this should be a reference to an existing chart UID. Try changing that and see if it helps.

{

“displayName”: “AFGHANISTAN”,

“name”: “AFGHANISTAN”,

“dashboardItems”: [

{

“chart”: {

“id”: “chartId”

}

}

],

regards,

Lars