Approve
Delete
@@ -44,54 +44,23 @@
-
+
+
\ No newline at end of file
diff --git a/edit-transcripts.js b/edit-transcripts.js
new file mode 100644
index 0000000..625317a
--- /dev/null
+++ b/edit-transcripts.js
@@ -0,0 +1,91 @@
+function switch_transcript() {
+ $$('.edit-transcript').each(function(t) {
+ (t.id == "wdts-transcripts-" + $F('wdts-language')) ? t.show() : t.hide();
+ });
+}
+switch_transcript();
+Event.observe(language_selector, 'change', switch_transcript);
+
+$$('.approve-transcript').each(function(b) {
+ Event.observe(b, 'click', function(e) {
+ Event.stop(e);
+ var lang = b.parentNode.parentNode.select("input[name*=[language]]").shift();
+ var post_id = b.parentNode.parentNode.parentNode.select("input[name*=[post_id]]").shift();
+ var key = b.parentNode.parentNode.select("input[name*=[key]]").shift();
+ if (lang && post_id && key) {
+ lang = lang.value;
+ post_id = post_id.value;
+ key = key.value;
+ var editor = $('wdts-transcripts-' + lang);
+
+ var raw_transcript = b.parentNode.parentNode.select(".queued-transcription-raw").shift();
+ if (raw_transcript && editor) {
+ var ok = true;
+ if (editor.value.match(/[^ ]/)) {
+ ok = confirm(messages.overwrite);
+ }
+ if (ok) {
+ editor.value = raw_transcript.innerHTML;
+ var p = b.parentNode.parentNode;
+
+ new Ajax.Request(
+ ajax_url, {
+ 'method': 'post',
+ 'parameters': {
+ 'wdts[_nonce]': nonce,
+ 'wdts[module]': 'approve-transcript',
+ 'wdts[key]': key,
+ 'wdts[post_id]': post_id
+ },
+ 'onSuccess': function() {
+ p.update(messages.approved);
+ new Effect.Highlight(p);
+ var i,il;
+
+ for (i = 0, il = language_selector.options.length; i < il; ++i) {
+ if (language_selector.options[i].value == lang) {
+ language_selector.selectedIndex = i;
+ switch_transcript();
+ break;
+ }
+ }
+ }
+ }
+ );
+ }
+ }
+ }
+ });
+});
+
+$$('.delete-transcript').each(function(b) {
+ Event.observe(b, 'click', function(e) {
+ Event.stop(e);
+
+ if (confirm(messages.delete)) {
+ var post_id = b.parentNode.parentNode.parentNode.select("input[name*=[post_id]]").shift();
+ var key = b.parentNode.parentNode.select("input[name*=[key]]").shift();
+ if (post_id && key) {
+ post_id = post_id.value;
+ key = key.value;
+ var p = b.parentNode.parentNode;
+
+ new Ajax.Request(
+ ajax_url, {
+ 'method': 'post',
+ 'parameters': {
+ 'wdts[_nonce]': nonce,
+ 'wdts[module]': 'delete-transcript',
+ 'wdts[key]': key,
+ 'wdts[post_id]': post_id
+ },
+ 'onSuccess': function() {
+ p.update(messages.deleted);
+ new Effect.Highlight(p);
+ }
+ }
+ );
+ }
+ }
+ });
+});
\ No newline at end of file
diff --git a/test/WDTSTranscriptTest.php b/test/WDTSTranscriptTest.php
index 22c9dfb..fc4618e 100644
--- a/test/WDTSTranscriptTest.php
+++ b/test/WDTSTranscriptTest.php
@@ -17,6 +17,8 @@ class WDTSTranscriptTest extends PHPUnit_Framework_TestCase {
}
function testSaveTranscript() {
+ $this->w->allow_multiple = false;
+
$this->w->save_transcript(array(
'language' => 'en',
'transcript' => 'this is a transcript'
@@ -27,7 +29,8 @@ class WDTSTranscriptTest extends PHPUnit_Framework_TestCase {
array(
'language' => 'en',
'transcript' => 'this is a transcript',
- 'user_id' => 1
+ 'user_id' => 1,
+ 'key' => 0
)
),
get_post_meta(1, $this->w->key, true)
@@ -43,7 +46,8 @@ class WDTSTranscriptTest extends PHPUnit_Framework_TestCase {
array(
'language' => 'en',
'transcript' => 'this is another transcript',
- 'user_id' => 1
+ 'user_id' => 1,
+ 'key' => 0
)
),
get_post_meta(1, $this->w->key, true)
@@ -59,39 +63,101 @@ class WDTSTranscriptTest extends PHPUnit_Framework_TestCase {
array(
'language' => 'en',
'transcript' => 'this is another transcript',
- 'user_id' => 1
+ 'user_id' => 1,
+ 'key' => 0
),
array(
'language' => 'fr',
'transcript' => "il s'agit d'une nouvelle transcription",
- 'user_id' => 1
+ 'user_id' => 1,
+ 'key' => 1
),
),
get_post_meta(1, $this->w->key, true)
);
- }
+
+ $this->w->allow_multiple = true;
+
+ $this->w->save_transcript(array(
+ 'language' => 'en',
+ 'transcript' => 'this is yet another transcript'
+ ));
+
+ $this->assertEquals(
+ array(
+ array(
+ 'language' => 'en',
+ 'transcript' => 'this is another transcript',
+ 'user_id' => 1,
+ 'key' => 0
+ ),
+ array(
+ 'language' => 'fr',
+ 'transcript' => "il s'agit d'une nouvelle transcription",
+ 'user_id' => 1,
+ 'key' => 1
+ ),
+ array(
+ 'language' => 'en',
+ 'transcript' => 'this is yet another transcript',
+ 'user_id' => 1,
+ 'key' => 2
+ ),
+ ),
+ get_post_meta(1, $this->w->key, true)
+ );
+ }
function testDeleteTranscript() {
- update_post_meta(1, $this->w->key, array(
- array(
- 'language' => 'en',
- 'transcript' => 'this is another transcript',
- 'user_id' => 1
- ),
- array(
- 'language' => 'fr',
- 'transcript' => "il s'agit d'une nouvelle transcription",
- 'user_id' => 1
- ),
+ $this->w->save_transcript(array(
+ 'language' => 'en',
+ 'transcript' => 'this is another transcript'
));
+ $this->w->save_transcript(array(
+ 'language' => 'fr',
+ 'transcript' => "il s'agit d'une nouvelle transcription",
+ ));
+
$this->w->delete_transcript('en');
$this->assertEquals(array(
array(
'language' => 'fr',
'transcript' => "il s'agit d'une nouvelle transcription",
- 'user_id' => 1
+ 'user_id' => 1,
+ 'key' => 1
+ ),
+ ), get_post_meta(1, $this->w->key, true));
+ }
+
+ function testDeleteTranscriptByKey() {
+ $this->w->save_transcript(array(
+ 'language' => 'en',
+ 'transcript' => 'this is another transcript'
+ ));
+
+ $this->w->save_transcript(array(
+ 'language' => 'fr',
+ 'transcript' => "il s'agit d'une nouvelle transcription",
+ ));
+
+ $this->assertEquals(
+ $this->w->delete_transcript_by_key(0),
+ array(
+ 'language' => 'en',
+ 'transcript' => "this is another transcript",
+ 'user_id' => 1,
+ 'key' => 0
+ )
+ );
+
+ $this->assertEquals(array(
+ array(
+ 'language' => 'fr',
+ 'transcript' => "il s'agit d'une nouvelle transcription",
+ 'user_id' => 1,
+ 'key' => 1
),
), get_post_meta(1, $this->w->key, true));
}
diff --git a/toggle-transcript.js b/toggle-transcript.js
new file mode 100644
index 0000000..916a038
--- /dev/null
+++ b/toggle-transcript.js
@@ -0,0 +1,13 @@
+$$('.transcript-bundle').each(function(d) {
+ var select = d.select("select");
+ if (select.length == 1) {
+ select = select[0];
+ var toggle_transcripts = function() {
+ d.select(".transcript-holder").each(function(div) {
+ div.hasClassName($F(select)) ? div.show() : div.hide();
+ });
+ };
+ Event.observe(select, 'change', toggle_transcripts);
+ Event.observe(window, 'load', toggle_transcripts)
+ }
+});
diff --git a/what-did-they-say.php b/what-did-they-say.php
index 7179c77..7af5100 100644
--- a/what-did-they-say.php
+++ b/what-did-they-say.php
@@ -26,11 +26,9 @@ Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
foreach (glob(dirname(__FILE__) . '/classes/*.php') as $file) { require_once($file); }
-$what_did_they_say_admin =& new WhatDidTheySayAdmin(&$what_did_they_say);
+$what_did_they_say_admin = new WhatDidTheySayAdmin(&$what_did_they_say);
add_action('init', array(&$what_did_they_say_admin, 'init'));
-register_activation_hook(__FILE__, array(&$what_did_they_say, 'install'));
-register_activation_hook(__FILE__, array(&$what_did_they_say_admin, 'install'));
// template tags
// please, if you use any of these, wrap them in function_exists() so your site doesn't
@@ -72,10 +70,10 @@ function the_media_transcript($language = null) {
* @return string The name of the requested language.
*/
function get_the_language_name($language = null) {
- global $what_did_they_say;
-
- if (is_null($language)) { $language = $what_did_they_say->get_default_language(); }
- return $what_did_they_say->get_language_name($language);
+ $language_options = new WDTSLanguageOptions();
+
+ if (is_null($language)) { $language = $language_options->get_default_language(); }
+ return $language_options->get_language_name($language);
}
/**
@@ -106,19 +104,21 @@ function transcripts_display($dropdown_message = null, $single_language_message
$post_transcripts = $approved_transcripts->get_transcripts();
if (!empty($post_transcripts)) {
- foreach ($post_transcripts as $code => $transcript) {
- if (substr($code, 0, 1) != "_") {
- $transcript = trim($transcript);
- if (!empty($transcript)) {
- $transcripts[$code] = $transcript;
- }
+ foreach ($post_transcripts as $transcript) {
+ extract($transcript, EXTR_PREFIX_ALL, "transcript");
+ $transcript_transcript = trim($transcript_transcript);
+ if (!empty($transcript_transcript)) {
+ $transcripts[$transcript_language] = $transcript_transcript;
}
}
+ $language_options = new WDTSLanguageOptions();
+
if (count($transcripts) > 0) {
- $default_language = $what_did_they_say->get_default_language();
+ $default_language = $language_options->get_default_language();
$output[] = '
';
+
if (count($transcripts) == 1) {
list($code, $transcript) = each($transcripts);
$output[] = apply_filters('the_language_name', get_the_language_name($code));
@@ -152,7 +152,7 @@ function transcripts_display($dropdown_message = null, $single_language_message
* If you're allowing users to submit transcripts to the post transcript queue, use this tag in your Loop.
*/
function the_media_transcript_queue_editor() {
- global $post, $what_did_they_say;
+ global $post;
if (current_user_can('submit_transcriptions')) {
$queued_transcripts_for_user = false;
@@ -164,40 +164,50 @@ function the_media_transcript_queue_editor() {
$queued_transcripts_for_user = $queued_transcripts->get_transcripts_for_user($user->ID);
}
- $transcript_options = new WDTSTranscriptOptions($post->ID);
+ $language_options = new WDTSLanguageOptions();
- if (is_array($queued_transcripts_for_user)) {
- if (count($queued_transcripts_for_user) > 0) { ?>
-
-
-
-
get_language_name($transcript->language) ?>
-
- transcript ?>
-
-
-
-
-
- are_new_transcripts_allowed()) { ?>
+ $transcript_options = new WDTSTranscriptOptions($post->ID);
+
+ $options = get_option('what-did-they-say-options');
+
+ foreach (array('Approved', 'Queued') as $name) {
+ $var_name = strtolower($name);
+ $class_name = "WDTS${name}Transcript";
+ ${"${var_name}_transcript_manager"} = new $class_name($post->ID);
+ ${"${var_name}_transcripts"} = ${"${var_name}_transcript_manager"}->get_transcripts();
+ }
+
+ $nonce = wp_create_nonce('what-did-they-say');
+
+ ?>
+
+
+
+
+ are_new_transcripts_allowed()) { ?>
+
+