Fixed Lowercase 'Open' in Moderation Queue

[Jeremy P]

Haven't looked into it but I noticed that the word 'Open' for reports in the queue is not capitalized..

 

[Paul B]

I've just checked on here and on my site and it is capitalised.

Are you running RC2?

Have you perhaps edited the phrase?

 

[Jeremy P]

Running RC2, phrase isn't edited.. hmm. What template contains the queue list items?

 

[Paul B]

It's a phrase: open_report

Unless you've edited the template and hard-coded it?

The template is report_view

 

[Jeremy P]

The phrase open_report exists and is capitalized. Hmm.

prepareReport() in XenForo_ReportHandler_Abstract..

switch ($report['report_state'])
{
	case 'open': $report['reportState'] = new XenForo_Phrase('open'); break;
    /* ... */
}

I don't have a phrase with the id 'open'.

 

[Paul B]

Ah, scratch my earlier, it's probably been fixed in the next release and has more than likely been rolled out here.

So I expect it was a bug and has now been fixed.

Let me check the latest equivalent of the code you posted above.

Yep, here's the new code:

        switch ($report['report_state'])
        {
            case 'open': $report['reportState'] = new XenForo_Phrase('open_report'); break;

I'll mark this as already fixed

 

[Jeremy P]

Awesome, thanks for the information

 

[soloarquitectura]

"open" remains untranslated in RC3, I don't know if this bug has already been reported in another post.

 

[Mike]

The issue is definitely fixed here - that's what Brogan was saying, indicating that it was likely fixed after the release.