Moodle

review and improve core output renderer and weblib.php

Details

  • Type: Sub-task Sub-task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.0
  • Fix Version/s: 2.0
  • Component/s: General
  • Labels:
    None
  • Affected Branches:
    MOODLE_20_STABLE
  • Fixed Branches:
    MOODLE_20_STABLE

Description

The new $OUTPUT is forcing us to write tons of repeated lines to get what was possible with only one line before, we need to get some compromise here...

Issue Links

Activity

Hide
Petr Škoda (skodak) added a comment -

at the same time we should remove the "moodle_" prefix from places where it is not really needed such as "moodle_url" class

Show
Petr Škoda (skodak) added a comment - at the same time we should remove the "moodle_" prefix from places where it is not really needed such as "moodle_url" class
Hide
Gabriel Dias added a comment -

Just to know, could you give any example where i can see the repeated code. Want to help, that's all

Show
Gabriel Dias added a comment - Just to know, could you give any example where i can see the repeated code. Want to help, that's all
Hide
Petr Škoda (skodak) added a comment -

Thanks for the offer and sorry for not reacting earlier.
No more API changes are planned in 2.0, I suppose we will continue with cleanup in 2.1.

Ciao

Petr Skoda

Show
Petr Škoda (skodak) added a comment - Thanks for the offer and sorry for not reacting earlier. No more API changes are planned in 2.0, I suppose we will continue with cleanup in 2.1. Ciao Petr Skoda

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: