1. This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

Fixed Lowercase 'Open' in Moderation Queue

Discussion in 'Resolved Bug Reports' started by Jeremy P, Feb 20, 2011.

  1. Jeremy P

    Jeremy P Well-Known Member

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

    xenforo.jpg
     
  2. Brogan

    Brogan XenForo Moderator Staff Member

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

    Are you running RC2?

    Have you perhaps edited the phrase?
     
  3. Jeremy P

    Jeremy P Well-Known Member

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

    Brogan XenForo Moderator Staff Member

    It's a phrase: open_report

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

    The template is report_view
     
  5. Jeremy P

    Jeremy P Well-Known Member

    The phrase open_report exists and is capitalized. Hmm.

    prepareReport() in XenForo_ReportHandler_Abstract..
    PHP:
    switch ($report['report_state'])
    {
        case 
    'open'$report['reportState'] = new XenForo_Phrase('open'); break;
        
    /* ... */
    }
    I don't have a phrase with the id 'open'.
     
  6. Brogan

    Brogan XenForo Moderator Staff Member

    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:
    PHP:
            switch ($report['report_state'])
            {
                case 
    'open'$report['reportState'] = new XenForo_Phrase('open_report'); break;
    I'll mark this as already fixed :)
     
  7. Jeremy P

    Jeremy P Well-Known Member

    Awesome, thanks for the information :)
     
  8. soloarquitectura

    soloarquitectura Active Member

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

    Mike XenForo Developer Staff Member

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

Share This Page