refactor to make it backwards compatible with old history

admin/client/App/screens/Revision/actions.js

@@ -64,9 +64,11 @@ export const selectRevision = revision => {
 export const applyChanges = router => {
 	return (dispatch, getState) => {
 		const state = getState();
+		const { selectedRevision } = state.revisions;
 		const { currentList } = state.lists;
 		const { id } = state.item;
-		const { data, _id: rollbackId } = state.revisions.selectedRevision;
+		const data = selectedRevision.data || selectedRevision.d;
+		const { _id: rollbackId } = selectedRevision;
 		const { currentItem } = state.revisions;
 		const redirectUrl = `${Keystone.adminPath}/${currentList.path}/${id}`;
 		const file = {

admin/client/App/screens/Revision/components/RevisionItem.js

@@ -76,11 +76,12 @@ const RevisionItem = ({
 		<div style={style}>
 			{revisions.map(revision => {
 				const active = selectedRevision._id === revision._id;
-				const { first, last } = revision.user.name;
+				const user = revision.user || revision.u;
+				const { first, last } = user;
 				return (
 					<div key={revision._id}>
 						<RevisionListItem active={active} noedit onClick={() => selectRevision(revision)}>
-							{moment(revision.time).format('YYYY-MM-DD hh:mm:ssa')} by {`${first} ${last}`}
+							{moment(revision.time || revision.t).format('YYYY-MM-DD hh:mm:ssa')} by {`${first} ${last}`}
 						</RevisionListItem>
 						{active
 							? <div className="RevisionsItem__table--container">
@@ -90,7 +91,7 @@ const RevisionItem = ({
 										<th>Current</th>
 										<th>Rollback</th>
 									</tr>
-									{renderDifferences(revision.data)}
+									{renderDifferences(revision.data || revision.d)}
 								</table>
 								<div>
 									<Container>

admin/client/App/screens/Revision/reducer.js

@@ -16,7 +16,8 @@ export default (state = {
 		case LOAD_REVISIONS:
 			return { ...state, error: null, ready: false };
 		case DATA_LOADING_SUCCESS:
-			const currentItem = action.payload.pop().data;
+			const popped = action.payload.pop();
+			const currentItem = popped.data || popped.d;
 			return { ...state, revisions: action.payload, error: null, ready: true, currentItem };
 		case DATA_LOADING_ERROR:
 			if (action.payload) {

admin/server/api/revision/get.js

@@ -9,6 +9,7 @@ module.exports = (req, res) => {
 
 	revisions.find({ id })
 		.populate('user', 'name')
+    .populate('u', 'name')
 		.then(items => res.json(items))
 		.catch(err => res.json(err));
 };

lib/schemaPlugins/history.js

@@ -59,7 +59,7 @@ module.exports = function history () {
 	});
 
 
-	list.schema.pre('save', function (next) { 
+	list.schema.pre('save', function (next) {
 		if (this.isModified()) {
 			this.__rev = (typeof this.__rev === 'number') ? this.__rev + 1 : 1;