Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot update annotation name (Trac #3238) #3238

Closed
elgg-gitbot opened this issue Feb 16, 2013 · 6 comments
Closed

Cannot update annotation name (Trac #3238) #3238

elgg-gitbot opened this issue Feb 16, 2013 · 6 comments
Labels
Milestone

Comments

@elgg-gitbot
Copy link

Original ticket http://trac.elgg.org/ticket/3238 on 41231095-11-13 by trac user kevinjardine, assigned to cash.

Elgg version: 1.7

The update_annotation function does not let you update the annotation name despite taking it as an argument. There appears to be no way to do this without writing custom SQL.

To fix this, change lines 160-162 in annotations.php from

$result = update_data("UPDATE {$CONFIG->dbprefix}annotations
        set value_id='$value', value_type='$value_type', access_id=$access_id, owner_guid=$owner_guid
        where id=$annotation_id and name_id='$name' and $access");

to

$result = update_data("UPDATE {$CONFIG->dbprefix}annotations
        set value_id='$value', name_id='$name', value_type='$value_type', access_id=$access_id, owner_guid=$owner_guid
        where id=$annotation_id and $access");
@elgg-gitbot
Copy link
Author

cash wrote on 41231116-01-09

This is also true for Elgg 1.7

@elgg-gitbot
Copy link
Author

Milestone changed to Elgg 1.7.8 by cash on 41234250-05-10

@elgg-gitbot
Copy link
Author

cash wrote on 41234253-02-21

(In [svn:8850]) Refs #3238 fix for updating name of metadata for trunk

@elgg-gitbot
Copy link
Author

cash wrote on 41234268-08-10

(In [svn:8851]) Fixes #3238 update_metadata() allows updating the name and integrated the fix for #3246 into 1.7 branch

@elgg-gitbot
Copy link
Author

cash wrote on 41234297-04-03

(In [svn:8852]) Refs #3238 #3246 anootations needed the same fixes as metadata

@elgg-gitbot
Copy link
Author

cash wrote on 41234322-11-22

(In [svn:8853]) Refs #3238 #3246 integrated fixes for annotations into 1.7 branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

1 participant