Moodle

Missing strings in XMLDB editor

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.3
  • Fix Version/s: 2.1.7, 2.2.4
  • Component/s: Administration
  • Labels:
    None
  • Testing Instructions:
    Hide
    1. Log in as Admin
    2. Open XMLDB editor (Settings -> Site administration -> Development -> XMLDB editor)
    3. Edit any database (By clicking on load and then edit link)
    4. Click "Edit" on table list (Shown under tables)
    5. Edit primary key (Click "Edit" on any key visible in Keys table)
    6. Click "Change" and you should not see any error.

    Note: Make sure to test this on 2.1 as well.

    Show
    Log in as Admin Open XMLDB editor (Settings -> Site administration -> Development -> XMLDB editor) Edit any database (By clicking on load and then edit link) Click "Edit" on table list (Shown under tables) Edit primary key (Click "Edit" on any key visible in Keys table) Click "Change" and you should not see any error. Note: Make sure to test this on 2.1 as well.
  • Affected Branches:
    MOODLE_23_STABLE
  • Fixed Branches:
    MOODLE_21_STABLE, MOODLE_22_STABLE
  • Pull from Repository:
  • Pull Master Branch:
    MDL-33090-missing-xmldb-strings
  • Pull Master Diff URL:

Description

Seen these when adding a new index to a table:

Invalid get_string() identifier: 'keynameempty' or component 'tool_xmldb'. Perhaps you are missing $string['keynameempty'] = ''; in /home/nicols/git/software/moodle/admin/tool/xmldb/lang/en/tool_xmldb.php?
line 6518 of /lib/moodlelib.php: call to debugging()
line 7142 of /lib/moodlelib.php: call to core_string_manager->get_string()
line 132 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to get_string()
line 61 of /admin/tool/xmldb/actions/edit_key_save/edit_key_save.class.php: call to XMLDBAction->loadStrings()
line 61 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to edit_key_save->init()
line 68 of /admin/tool/xmldb/index.php: call to XMLDBAction->__construct()
Invalid get_string() identifier: 'incorrectkeyname' or component 'tool_xmldb'. Perhaps you are missing $string['incorrectkeyname'] = ''; in /home/nicols/git/software/moodle/admin/tool/xmldb/lang/en/tool_xmldb.php?
line 6518 of /lib/moodlelib.php: call to debugging()
line 7142 of /lib/moodlelib.php: call to core_string_manager->get_string()
line 132 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to get_string()
line 61 of /admin/tool/xmldb/actions/edit_key_save/edit_key_save.class.php: call to XMLDBAction->loadStrings()
line 61 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to edit_key_save->init()
line 68 of /admin/tool/xmldb/index.php: call to XMLDBAction->__construct()
Invalid get_string() identifier: 'nofieldsspecified' or component 'tool_xmldb'. Perhaps you are missing $string['nofieldsspecified'] = ''; in /home/nicols/git/software/moodle/admin/tool/xmldb/lang/en/tool_xmldb.php?
line 6518 of /lib/moodlelib.php: call to debugging()
line 7142 of /lib/moodlelib.php: call to core_string_manager->get_string()
line 132 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to get_string()
line 61 of /admin/tool/xmldb/actions/edit_key_save/edit_key_save.class.php: call to XMLDBAction->loadStrings()
line 61 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to edit_key_save->init()
line 68 of /admin/tool/xmldb/index.php: call to XMLDBAction->__construct()
Invalid get_string() identifier: 'duplicatefieldsused' or component 'tool_xmldb'. Perhaps you are missing $string['duplicatefieldsused'] = ''; in /home/nicols/git/software/moodle/admin/tool/xmldb/lang/en/tool_xmldb.php?
line 6518 of /lib/moodlelib.php: call to debugging()
line 7142 of /lib/moodlelib.php: call to core_string_manager->get_string()
line 132 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to get_string()
line 61 of /admin/tool/xmldb/actions/edit_key_save/edit_key_save.class.php: call to XMLDBAction->loadStrings()
line 61 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to edit_key_save->init()
line 68 of /admin/tool/xmldb/index.php: call to XMLDBAction->__construct()
Invalid get_string() identifier: 'fieldsusedinindex' or component 'tool_xmldb'. Perhaps you are missing $string['fieldsusedinindex'] = ''; in /home/nicols/git/software/moodle/admin/tool/xmldb/lang/en/tool_xmldb.php?
line 6518 of /lib/moodlelib.php: call to debugging()
line 7142 of /lib/moodlelib.php: call to core_string_manager->get_string()
line 132 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to get_string()
line 61 of /admin/tool/xmldb/actions/edit_key_save/edit_key_save.class.php: call to XMLDBAction->loadStrings()
line 61 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to edit_key_save->init()
line 68 of /admin/tool/xmldb/index.php: call to XMLDBAction->__construct()
Invalid get_string() identifier: 'noreftablespecified' or component 'tool_xmldb'. Perhaps you are missing $string['noreftablespecified'] = ''; in /home/nicols/git/software/moodle/admin/tool/xmldb/lang/en/tool_xmldb.php?
line 6518 of /lib/moodlelib.php: call to debugging()
line 7142 of /lib/moodlelib.php: call to core_string_manager->get_string()
line 132 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to get_string()
line 61 of /admin/tool/xmldb/actions/edit_key_save/edit_key_save.class.php: call to XMLDBAction->loadStrings()
line 61 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to edit_key_save->init()
line 68 of /admin/tool/xmldb/index.php: call to XMLDBAction->__construct()
Invalid get_string() identifier: 'wrongnumberofreffields' or component 'tool_xmldb'. Perhaps you are missing $string['wrongnumberofreffields'] = ''; in /home/nicols/git/software/moodle/admin/tool/xmldb/lang/en/tool_xmldb.php?
line 6518 of /lib/moodlelib.php: call to debugging()
line 7142 of /lib/moodlelib.php: call to core_string_manager->get_string()
line 132 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to get_string()
line 61 of /admin/tool/xmldb/actions/edit_key_save/edit_key_save.class.php: call to XMLDBAction->loadStrings()
line 61 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to edit_key_save->init()
line 68 of /admin/tool/xmldb/index.php: call to XMLDBAction->__construct()
Invalid get_string() identifier: 'noreffieldsspecified' or component 'tool_xmldb'. Perhaps you are missing $string['noreffieldsspecified'] = ''; in /home/nicols/git/software/moodle/admin/tool/xmldb/lang/en/tool_xmldb.php?
line 6518 of /lib/moodlelib.php: call to debugging()
line 7142 of /lib/moodlelib.php: call to core_string_manager->get_string()
line 132 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to get_string()
line 61 of /admin/tool/xmldb/actions/edit_key_save/edit_key_save.class.php: call to XMLDBAction->loadStrings()
line 61 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to edit_key_save->init()
line 68 of /admin/tool/xmldb/index.php: call to XMLDBAction->__construct()
Invalid get_string() identifier: 'primarykeyonlyallownotnullfields' or component 'tool_xmldb'. Perhaps you are missing $string['primarykeyonlyallownotnullfields'] = ''; in /home/nicols/git/software/moodle/admin/tool/xmldb/lang/en/tool_xmldb.php?
line 6518 of /lib/moodlelib.php: call to debugging()
line 7142 of /lib/moodlelib.php: call to core_string_manager->get_string()
line 132 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to get_string()
line 61 of /admin/tool/xmldb/actions/edit_key_save/edit_key_save.class.php: call to XMLDBAction->loadStrings()
line 61 of /admin/tool/xmldb/actions/XMLDBAction.class.php: call to edit_key_save->init()
line 68 of /admin/tool/xmldb/index.php: call to XMLDBAction->__construct()

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Rex Lorenzo added a comment -

Also getting this in Moodle 2.2.2

Show
Rex Lorenzo added a comment - Also getting this in Moodle 2.2.2
Hide
Permalink
Rex Lorenzo added a comment -

I cannot seem to edit the ticket to submit my patch, but here it is:

Diff with 2.3: https://github.com/rlorenzo/moodle/compare/rlorenzo:master...rlorenzo:MDL-33090-missing-xmldb-strings

Link to patch: https://github.com/rlorenzo/moodle/commit/ae11631886b248c8727f4bbaa8e45319a11b617a

Diff with 2.2: https://github.com/rlorenzo/moodle/compare/rlorenzo:MOODLE_22_STABLE...rlorenzo:MDL-33090_22-missing-xmldb-strings

Link to patch: https://github.com/rlorenzo/moodle/commit/c31149fa6714176cf1312094f502b50b48dab2f7

Show
Rex Lorenzo added a comment - I cannot seem to edit the ticket to submit my patch, but here it is: Diff with 2.3: https://github.com/rlorenzo/moodle/compare/rlorenzo:master...rlorenzo:MDL-33090-missing-xmldb-strings Link to patch: https://github.com/rlorenzo/moodle/commit/ae11631886b248c8727f4bbaa8e45319a11b617a Diff with 2.2: https://github.com/rlorenzo/moodle/compare/rlorenzo:MOODLE_22_STABLE...rlorenzo:MDL-33090_22-missing-xmldb-strings Link to patch: https://github.com/rlorenzo/moodle/commit/c31149fa6714176cf1312094f502b50b48dab2f7
Hide
Permalink
Rajesh Taneja added a comment -

Thanks for the Patch Rex,
I have added links and pushing it for peer-review.

Show
Rajesh Taneja added a comment - Thanks for the Patch Rex, I have added links and pushing it for peer-review.
Hide
Permalink
Andrew Davis added a comment -

Seems simple enough. Does this want to go into 2.1 as well?

Show
Andrew Davis added a comment - Seems simple enough. Does this want to go into 2.1 as well?
Hide
Permalink
Rajesh Taneja added a comment -

Thanks Andrew,

Yes, it should have gone on 2.1 as well. I have created 2.1 branch and pushing it for integration review.

Show
Rajesh Taneja added a comment - Thanks Andrew, Yes, it should have gone on 2.1 as well. I have created 2.1 branch and pushing it for integration review.
Hide
Permalink
Dan Poltawski added a comment -

The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

TIA and ciao

Show
Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
Hide
Permalink
Rajesh Taneja added a comment -

I have re-based 21 branch, Rex can you please re-base master and 22 branch.
Thanks.

Show
Rajesh Taneja added a comment - I have re-based 21 branch, Rex can you please re-base master and 22 branch. Thanks.
Hide
Permalink
Sam Hemelryk added a comment -

Thanks guys this has been integrated now

Show
Sam Hemelryk added a comment - Thanks guys this has been integrated now
Hide
Permalink
Sam Hemelryk added a comment -

Tested during integration review and passed

Show
Sam Hemelryk added a comment - Tested during integration review and passed
Hide
Permalink
Eloy Lafuente (stronk7) added a comment -

We could celebrate it today... but better if we perform a bigger party after releasing Moodle 2.3.

Print this message and come to Perth that day, it's valid for one beer, wine, coke or... water, as you wish.

Many thanks for your collaboration!

Ciao

Show
Eloy Lafuente (stronk7) added a comment - We could celebrate it today... but better if we perform a bigger party after releasing Moodle 2.3. Print this message and come to Perth that day, it's valid for one beer, wine, coke or... water, as you wish. Many thanks for your collaboration! Ciao

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: