Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-29258

Weird breadcrumb navigation text when exporting a file to a portfolio

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.1, 2.5
    • Fix Version/s: 2.3.7, 2.4.4
    • Component/s: Portfolio
    • Labels:
    • Testing Instructions:
      Hide
      1. Enable site-wide portfolios setting
      2. Enable file download portfolio
      3. Go to a course and create a chat
      4. Go to the chat begin a session, write something and close it
      5. Return to the chat instance and view past chat sessions
      6. Press Export all to portfolio
      7. You SHOULD NOT see any PHP warning regarding undefined 'name' offset'
      8. The breadcrumb SHOULD make sense, including the chat activity name just once
      Show
      Enable site-wide portfolios setting Enable file download portfolio Go to a course and create a chat Go to the chat begin a session, write something and close it Return to the chat instance and view past chat sessions Press Export all to portfolio You SHOULD NOT see any PHP warning regarding undefined 'name' offset' The breadcrumb SHOULD make sense, including the chat activity name just once
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-29258_master

      Description

      I have submitted a private file to an assignment and then I clicked the export icon (green arrow)

      Now, it all works fine. except the weird breadcrumb navigation text.

      The filename in English is called "pdf.png".

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              salvetore Michael de Raadt added a comment -

              Thanks for reporting this.

              Feel free to help us work on this issue. If you do add a solution, please add a 'patch' label.

              Show
              salvetore Michael de Raadt added a comment - Thanks for reporting this. Feel free to help us work on this issue. If you do add a solution, please add a 'patch' label.
              Hide
              dmonllao David Monllaó added a comment -

              Adding pull branches, is blocking MDLQA-5392.

              Submitting for peer review, I removed the activity name as there is already a link to the activity in the breadcrumb, not sure if there are settings that can affect this breadcrumb element.

              Show
              dmonllao David Monllaó added a comment - Adding pull branches, is blocking MDLQA-5392 . Submitting for peer review, I removed the activity name as there is already a link to the activity in the breadcrumb, not sure if there are settings that can affect this breadcrumb element.
              Hide
              andyjdavis Andrew Davis added a comment -

              [Y] Syntax
              [NA] Output
              [Y] Whitespace
              [NA] Language
              [NA] Databases
              [Y] Testing
              [NA] Security
              [NA] Documentation
              [Y] Git
              [Y] Sanity check

              Looks good. You are go for integration.

              Show
              andyjdavis Andrew Davis added a comment - [Y] Syntax [NA] Output [Y] Whitespace [NA] Language [NA] Databases [Y] Testing [NA] Security [NA] Documentation [Y] Git [Y] Sanity check Looks good. You are go for integration.
              Hide
              dmonllao David Monllaó added a comment -

              Thanks Andrew

              Show
              dmonllao David Monllaó added a comment - Thanks Andrew
              Hide
              poltawski Dan Poltawski added a comment -

              Thanks David, integrated and tested during integration.

              Show
              poltawski Dan Poltawski added a comment - Thanks David, integrated and tested during integration.
              Hide
              poltawski Dan Poltawski added a comment -

              Thanks! You're changes are now spread to the world through this git and our source control repositories.

              No time to rest though, we've got days to make 2.5 the best yet!

              ciao

              Show
              poltawski Dan Poltawski added a comment - Thanks! You're changes are now spread to the world through this git and our source control repositories. No time to rest though, we've got days to make 2.5 the best yet! ciao

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    13/May/13