We have moved to Github. Please open tickets there.

Opened 3 years ago

Closed 2 years ago

#1833 closed Defect (fixed)

changes lost to posted blog entry after preview

Reported by: cash Owned by:
Priority: normal Milestone: Elgg 1.7.8
Component: Blog Version: 1.7
Severity: minor Keywords:
Cc: brettp Difficulty:

Description (last modified by cash)

  1. write and publish blog post
  2. edit blog post and change some text
  3. preview post
  4. click return to post and your changes are gone

Change History (9)

comment:1 Changed 3 years ago by brettp

  • Milestone set to Elgg 1.8
  • Resolution set to worksforme
  • Severity set to minor
  • Status changed from new to closed
  • Version set to 1.7

Looks like this was fixed in sometime in 1.7.1/2.

comment:2 Changed 3 years ago by cash

  • Description modified (diff)
  • Resolution worksforme deleted
  • Status changed from closed to reopened

It works in Chrome but not in Firefox (and possibly not in IE)

comment:3 Changed 3 years ago by brettp

  • Milestone changed from Elgg 1.8 to Elgg 1.7.4

Not sure why this got pushed to 1.8, but 1.8 uses a new blog that saves drafts instead of previewing, so this won't be a problem. Moving this down to 1.7.4, which does still have the bug.

comment:4 Changed 3 years ago by brettp

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [svn:7041]) Fixes #1833: Saving a blog draft saves the blogguid to the session to avoid problems when pulling in drafts on existing blog posts.

comment:5 Changed 2 years ago by kevinjardine

  • Resolution fixed deleted
  • Status changed from closed to reopened

The blogguid fix has unfortunately introduced a new bug as described here:

http://community.elgg.org/pg/forum/topic/661881/empty-title-and-contents-when-editing-blog/

To fix this problem, add:

remove_metadata($_SESSIONuser?->guid,'blogguid');

after line 69 of

mod/blog/actions/edit.php

comment:6 Changed 2 years ago by kevinjardine

That should be:

remove_metadata($_SESSION['user']->guid,'blogguid');
}}

to keep Trac happy.

comment:7 Changed 2 years ago by kevinjardine

Third time lucky:

remove_metadata($_SESSION['user']->guid,'blogguid');

comment:8 Changed 2 years ago by cash

  • Milestone changed from Elgg 1.7.4 to Elgg 1.7.8

Thanks for tracking this down, Kevin!

comment:9 Changed 2 years ago by cash

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [svn:8726]) Fixes #1833 clearing the blog guid metadata when saving an edited blog post

Note: See TracTickets for help on using tickets.