Skip to content

Commit

Permalink
Remove basic view compose code
Browse files Browse the repository at this point in the history
  • Loading branch information
@slusarz
slusarz committed Oct 16, 2014
1 parent 78f7ec9 commit d61e66f
Show file tree
Hide file tree
Showing 19 changed files with 1,860 additions and 3,939 deletions.
2 changes: 1 addition & 1 deletion imp/js/ckeditor/imagepoll.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ var IMP_Ckeditor_Imagepoll = {};
ids.push(s[1]);
});

DimpCompose.attachlist.removeAttach(ids);
ImpCompose.attachlist.removeAttach(ids);
};

}(IMP_Ckeditor_Imagepoll));
2 changes: 1 addition & 1 deletion imp/js/ckeditor/imageupload.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ document.observe('dom:loaded', function() {
type: 'text'
});

params = $H({ composeCache: $F(DimpCompose.getCacheElt()) });
params = $H({ composeCache: $F(ImpCompose.getCacheElt()) });
HordeCore.addRequestParams(params);

upload.get('uploadButton').filebrowser = {
Expand Down
2 changes: 1 addition & 1 deletion imp/js/ckeditor/pasteattachment.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ CKEDITOR.plugins.add('pasteattachment', {

function uploadAtc(files)
{
DimpCompose.attachlist.uploadAttach(
ImpCompose.attachlist.uploadAttach(
files,
{ img_data: 1 },
attachCallback
Expand Down
117 changes: 0 additions & 117 deletions imp/js/compose-base.js
View file

This file was deleted.

2 comments on commit d61e66f

@joaomachado
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

compose-base.js was deleted on this commit

--- a/imp/lib/Script/Package/Autocomplete.php
+++ b/imp/lib/Script/Package/Autocomplete.php
@@ -31,7 +31,7 @@ extends Horde_Core_Script_Package_Keynavlist
     {   
         parent::__construct();

-        $this->_files[] = new Horde_Script_File_JsDir('compose-base.js', 'imp');
         $this->_files[] = new Horde_Script_File_JsDir('autocomplete.js', 'imp');
         $this->_files[] = new Horde_Script_File_JsDir('latinize.js', 'imp');
     }

@slusarz
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's correct. compose-base.js was combined with compose-dimp.js to create the "new" compose.js.

Semantically, trying to eliminate all remaining dimp references (since that hasn't meant anything in years).

Please sign in to comment.