Project

General

Profile

Feature #1370

Check files in main/metadata

Added by Julio Montoya over 9 years ago. Updated over 4 years ago.

Status:
Feature implemented
Priority:
Low
Assignee:
Category:
System
Target version:
Start date:
27/05/2010
Due date:
% Done:

100%

Estimated time:
1.00 h
Spent time:
Complexity:
Normal
SCRUM pts - complexity:
5

Description

Check if is possible to remove or improve those files.


Related issues

Related to Chamilo LMS - Bug #4139: Error Table 'chamilo19.c_metadata' doesn't exist when editing link's metadataBug resolved13/12/2011

Actions

Associated revisions

Revision 02144c80 (diff)
Added by Julio Montoya over 5 years ago

Removing metadata see #1370

Revision b92cc2d1 (diff)
Added by Julio Montoya over 5 years ago

Removing metadata calls see #1370

Revision 0c0a2bf1 (diff)
Added by Julio Montoya over 4 years ago

Remove metadata code see #1370

Revision b3e76394 (diff)
Added by Yannick Warnier over 4 years ago

Remove metadata edition page link - refs #1370

History

#1

Updated by Julio Montoya over 9 years ago

  • Target version set to 1.8.7.1
#2

Updated by Yannick Warnier about 9 years ago

  • Target version changed from 1.8.7.1 to 1.8.8 stable
#3

Updated by Yannick Warnier over 8 years ago

  • Priority changed from Normal to Low
  • Target version changed from 1.8.8 stable to 1.8.8.4
  • Estimated time set to 1.00 h
  • SCRUM pts - complexity changed from ? to 5

I think this code is still used in the documents tool (when editing a document, see below, the link "Edit metadata"). If still used, it should be left alone. If not, we can probably remove it, yes.

#4

Updated by Yannick Warnier about 8 years ago

  • Target version changed from 1.8.8.4 to 1.9 Stable
#5

Updated by Julio Montoya over 7 years ago

any idea why we maintain that code ...

#6

Updated by Yannick Warnier over 7 years ago

  • Target version changed from 1.9 Stable to 1.9.2
#7

Updated by Yannick Warnier about 7 years ago

  • Target version changed from 1.9.2 to 2.0
#8

Updated by Julio Montoya over 5 years ago

  • Status changed from New to Needs testing
  • % Done changed from 0 to 100
#9

Updated by Julio Montoya over 5 years ago

  • % Done changed from 100 to 80
#10

Updated by Julio Montoya over 4 years ago

  • Status changed from Needs testing to Needs more info
  • Target version changed from 2.0 to 1.10.0

The metadata folder has very old code. And it seems that is not used anymore.
It should be removed.

#11

Updated by Julio Montoya over 4 years ago

  • Status changed from Needs more info to Assigned
  • Assignee set to Julio Montoya

I'm removing this code (in my doctrine branch).

#12

Updated by Yannick Warnier over 4 years ago

It still works. There is value in that code (or at least in the data it stores).
Can we agree to remove the code but leave the database tables there for now? (so that if someone upgrades, they don't loose their data and can develop something new on top of that)

#13

Updated by Julio Montoya over 4 years ago

Yannick Warnier wrote:

It still works. There is value in that code (or at least in the data it stores).
Can we agree to remove the code but leave the database tables there for now? (so that if someone upgrades, they don't loose their data and can develop something new on top of that)

Ok, I won't drop the table.

#14

Updated by Yannick Warnier over 4 years ago

  • Category set to System

Make sure you don't forget to remove the links from the document edition screen and the links edition screen (I think these are the only places where this tool is accessible still).

I'm putting a "System" category because it's kind of a structural change.

#15

Updated by Julio Montoya over 4 years ago

Yannick Warnier wrote:

Make sure you don't forget to remove the links from the document edition screen and the links edition screen (I think these are the only places where this tool is accessible still).

I'm putting a "System" category because it's kind of a structural change.

Already done in my doctrine branch. I will do a merge

#16

Updated by Julio Montoya over 4 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Julio Montoya)
#17

Updated by Yannick Warnier over 4 years ago

There was still a metadata edition link in the link creation form. Please be more careful when removing things. I mentioned it in comment 14 as well...

Fixed in https://github.com/chamilo/chamilo-lms/commit/b3e76394df905656cfa62d843224217c5afc60fa

#18

Updated by Yannick Warnier over 4 years ago

  • Status changed from Needs testing to Feature implemented
  • Assignee set to Julio Montoya
  • % Done changed from 80 to 100

Metadata removed!

Also available in: Atom PDF