Best practice forum (Archived)

Minor bug: "Appraisal Status" column in Appraisal Status report source

I added the column "Appraisal Status" to a custom version of the Appraisal Status report but the value returned is numeric (1,2 3 etc) rather than 'the' status expressed in human readable form. Was that the intent?

Totara 2.5.13

Permalink | Reply

Hi Maurice,

At the top of /totara/appraisal/lib.php there are constants defined

46 /*
47 Appraisal status
48 */
49 const STATUS_DRAFT = 0;
50 const STATUS_ACTIVE = 1;
51 const STATUS_CLOSED = 2;
52 const STATUS_COMPLETED = 3;

If you add a 4th constant here it should be fine.

regards,

George.

Permalink | Show parent | Reply

Hi Maurice.

There was a bit of confusion about what you were reporting, so George's answer doesn't apply.

I checked it out and the column does indeed show a code, while it really should show a human-readable string. I'll put in a bug report to get someone to change it. The codes listed in George's report indicate what the numbers mean, which might be useful to you until the patch arrives (which will hopefully be soon).

Nathan

Permalink | Show parent | Reply

A fix for this issue was included in today's (10th June 2014) release of Totara 2.5.14

Permalink | Show parent | Reply

A fix for this issue was included in today's (17th June 2014) release of Totara 2.6.3

Permalink | Show parent | Reply