light clean up on the embed plugin which doesn't appear to work at present

git-svn-id: http://code.elgg.org/elgg/trunk@8125 36083f99-b078-4883-b0ff-0f9b5a30f544

diff --git a/mod/embed/languages/en.php b/mod/embed/languages/en.php
index f8edf38123e96214bbdb6ac7a9dbc150ca312607..f3a49e38bbc5e47638999452c63ed38421009c40 100644 (file)
--- a/mod/embed/languages/en.php
+++ b/mod/embed/languages/en.php
@@ -2,8 +2,8 @@
 /**
  * Embed English language strings
  *
- * @var unknown_type
  */
+
 $english = array(
        'media:insert' => 'Embed content',
        'embed:embed' => 'Embed',

diff --git a/mod/embed/start.php b/mod/embed/start.php
index 8ad1a6178f469ef4a0d9c05bec9f9169e6a52309..f8a8701fa7ccf6cf546488b314201d150c748fe6 100644 (file)
--- a/mod/embed/start.php
+++ b/mod/embed/start.php
@@ -5,14 +5,16 @@
  * @package ElggEmbed
  */
 
+
+elgg_register_event_handler('init', 'system', 'embed_init');
+
 /**
  * Init function
- *
  */
 function embed_init() {
        elgg_extend_view('css/screen', 'embed/css');
        elgg_extend_view('js/elgg', 'embed/js');
-       elgg_extend_view('html_head/extend', 'embed/metatags');
+       elgg_extend_view('footer/analytics', 'embed/lightbox_init');
        elgg_extend_view('input/longtext', 'embed/link', 1);
 
        // Page handler for the modal media embed
@@ -29,12 +31,10 @@ function embed_page_handler($page) {
                $page[0] = 'embed';
        }
 
-       switch($page[0]) {
+       switch ($page[0]) {
                case 'upload':
                        echo elgg_view('embed/upload');
-
                        break;
-
                case 'embed':
                default:
                        // trigger hook to get section tabs
@@ -58,12 +58,9 @@ function embed_page_handler($page) {
                                'upload_sections' => $upload_sections,
                                'internal_name' => $internal_name
                        ));
-
                        break;
        }
 
        // exit because this is in a modal display.
        exit;
 }
-
-elgg_register_event_handler('init', 'system', 'embed_init');
\ No newline at end of file

diff --git a/mod/embed/views/default/embed/addcontentjs.php b/mod/embed/views/default/embed/addcontentjs.php
index c32d90871b894d7451874e95201e4df2a2327d4e..18d2927fdc9540bc0f118a06a6566ff7f27b262f 100644 (file)
--- a/mod/embed/views/default/embed/addcontentjs.php
+++ b/mod/embed/views/default/embed/addcontentjs.php
@@ -1,5 +1,6 @@
 <?php 
 /**
  * Blank for compatibility.
+ * @deprecated 1.8
  */
 ?>
\ No newline at end of file

diff --git a/mod/embed/views/default/embed/lightbox_init.php b/mod/embed/views/default/embed/lightbox_init.php
new file mode 100644 (file)
index 0000000..9f27106
--- /dev/null
+++ b/mod/embed/views/default/embed/lightbox_init.php
@@ -0,0 +1,6 @@
+<?php //@todo JS 1.8: no ?>
+<script type="text/javascript">
+       jQuery(document).ready(function($) {
+               $('a[rel*=facebox]').facebox()
+       });
+</script>

diff --git a/mod/embed/views/default/embed/link.php b/mod/embed/views/default/embed/link.php
index c4fae0d42a60460a1dd817e5df0900cf3800e818..4ed193047c5615b48b8ad68ef861fc5926c8709c 100644 (file)
--- a/mod/embed/views/default/embed/link.php
+++ b/mod/embed/views/default/embed/link.php
@@ -10,5 +10,10 @@ if (elgg_is_active_plugin('ecml')) {
        $active_section = '';
 }
 
+$url = "pg/embed/?{$active_section}internal_name={$vars['internalname']}";
+$url = elgg_normalize_url($url);
+
 ?>
-<a class="elgg-longtext-control small link" href="<?php echo elgg_get_site_url() . 'pg/embed'; ?>?<?php echo $active_section; ?>internal_name=<?php echo $vars['internalname']; ?>" rel="facebox"><?php echo elgg_echo('media:insert'); ?></a>
+<a class="elgg-longtext-control" href="<?php echo $url; ?>" rel="facebox">
+       <?php echo elgg_echo('media:insert'); ?>
+</a>

diff --git a/mod/embed/views/default/embed/metatags.php b/mod/embed/views/default/embed/metatags.php
deleted file mode 100644 (file)
index 5ff6023..0000000
--- a/mod/embed/views/default/embed/metatags.php
+++ /dev/null
@@ -1,6 +0,0 @@
-               <?php //@todo JS 1.8: no ?>
-               <script type="text/javascript">
-               jQuery(document).ready(function($) {
-               $('a[rel*=facebox]').facebox()
-               });
-               </script>