summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--includes/common.inc19
-rw-r--r--includes/theme.inc13
-rw-r--r--misc/drupal.css2
-rw-r--r--modules/blogapi.module42
-rw-r--r--modules/blogapi/blogapi.module42
5 files changed, 66 insertions, 52 deletions
diff --git a/includes/common.inc b/includes/common.inc
index d15c9f7a5..5c4835b19 100644
--- a/includes/common.inc
+++ b/includes/common.inc
@@ -813,34 +813,35 @@ function form($form, $method = "post", $action = 0, $options = 0) {
}
function form_item($title, $value, $description = 0) {
- return "<div class=\"form-item\">". ($title ? "<div class=\"title\">$title:</div>" : "") . $value . ($description ? "<div class=\"description\">$description</div>" : "") ."</div>\n";
+ return theme("form_element", $title, $value, $description);
}
+
function form_group($legend, $group, $description = 0) {
return "<fieldset>" . ($legend ? "<legend>$legend</legend>" : "") . $group . ($description ? "<div class=\"description\">$description</div>" : "") . "</fieldset>\n";
}
function form_radio($title, $name, $value = 1, $checked = 0, $description = 0, $attributes = 0) {
- return form_item(0, "<input type=\"radio\" class=\"form-radio\" name=\"edit[$name]\" value=\"". $value ."\"". ($checked ? " checked=\"checked\"" : "") . drupal_attributes($attributes) ." /> $title", $description);
+ return theme("form_element", 0, "<input type=\"radio\" class=\"form-radio\" name=\"edit[$name]\" value=\"". $value ."\"". ($checked ? " checked=\"checked\"" : "") . drupal_attributes($attributes) ." /> $title", $description);
}
function form_checkbox($title, $name, $value = 1, $checked = 0, $description = 0, $attributes = 0) {
- return form_hidden($name, 0) . form_item(0, "<input type=\"checkbox\" class=\"form-checkbox\" name=\"edit[$name]\" value=\"". $value ."\"". ($checked ? " checked=\"checked\"" : "") . drupal_attributes($attributes) ." /> $title", $description);
+ return form_hidden($name, 0) . theme("form_element", 0, "<input type=\"checkbox\" class=\"form-checkbox\" name=\"edit[$name]\" value=\"". $value ."\"". ($checked ? " checked=\"checked\"" : "") . drupal_attributes($attributes) ." /> $title", $description);
}
function form_textfield($title, $name, $value, $size, $maxlength, $description = 0, $attributes = 0) {
$size = $size ? " size=\"$size\"" : "";
- return form_item($title, "<input type=\"text\" maxlength=\"$maxlength\" class=\"form-text\" name=\"edit[$name]\"$size value=\"". check_form($value) ."\"". drupal_attributes($attributes) ." />", $description);
+ return theme("form_element", $title, "<input type=\"text\" maxlength=\"$maxlength\" class=\"form-text\" name=\"edit[$name]\"$size value=\"". check_form($value) ."\"". drupal_attributes($attributes) ." />", $description);
}
function form_password($title, $name, $value, $size, $maxlength, $description = 0, $attributes = 0) {
$size = $size ? " size=\"$size\"" : "";
- return form_item($title, "<input type=\"password\" class=\"form-password\" maxlength=\"$maxlength\" name=\"edit[$name]\"$size value=\"". check_form($value) ."\"". drupal_attributes($attributes) ." />", $description);
+ return theme("form_element", $title, "<input type=\"password\" class=\"form-password\" maxlength=\"$maxlength\" name=\"edit[$name]\"$size value=\"". check_form($value) ."\"". drupal_attributes($attributes) ." />", $description);
}
function form_textarea($title, $name, $value, $cols, $rows, $description = 0, $attributes = 0) {
$cols = $cols ? " cols=\"$cols\"" : "";
module_invoke_all("textarea", $name); // eg. optionally plug in a WYSIWYG editor
- return form_item($title, "<textarea wrap=\"virtual\"$cols rows=\"$rows\" name=\"edit[$name]\" id=\"edit[$name]\"". drupal_attributes($attributes) .">". check_form($value) ."</textarea>", $description);
+ return theme("form_element", $title, "<textarea wrap=\"virtual\"$cols rows=\"$rows\" name=\"edit[$name]\" id=\"edit[$name]\"". drupal_attributes($attributes) .">". check_form($value) ."</textarea>", $description);
}
function form_select($title, $name, $value, $options, $description = 0, $extra = 0, $multiple = 0) {
@@ -848,7 +849,7 @@ function form_select($title, $name, $value, $options, $description = 0, $extra =
foreach ($options as $key=>$choice) {
$select .= "<option value=\"$key\"". (is_array($value) ? (in_array($key, $value) ? " selected=\"selected\"" : "") : ($value == $key ? " selected=\"selected\"" : "")) .">". check_form($choice) ."</option>";
}
- return form_item($title, "<select name=\"edit[$name]". ($multiple ? "[]" : "") ."\"". ($multiple ? " multiple " : "") . ($extra ? " $extra" : "") .">$select</select>", $description);
+ return theme("form_element", $title, "<select name=\"edit[$name]". ($multiple ? "[]" : "") ."\"". ($multiple ? " multiple " : "") . ($extra ? " $extra" : "") .">$select</select>", $description);
}
}
@@ -857,12 +858,12 @@ function form_radios($title, $name, $value, $options, $description = 0) {
foreach ($options as $key=>$choice) {
$output .= form_radio($choice, $name, $key, ($key == $value));
}
- return form_item($title, $output, $description);
+ return theme("form_element", $title, $output, $description);
}
}
function form_file($title, $name, $size, $description = 0) {
- return form_item($title, "<input type=\"file\" class=\"form-file\" name=\"edit[$name]\" size=\"$size\" />\n", $description);
+ return theme("form_element", $title, "<input type=\"file\" class=\"form-file\" name=\"edit[$name]\" size=\"$size\" />\n", $description);
}
function form_hidden($name, $value) {
diff --git a/includes/theme.inc b/includes/theme.inc
index ae284d34d..6dfff30bd 100644
--- a/includes/theme.inc
+++ b/includes/theme.inc
@@ -145,6 +145,19 @@ function _theme_table_cell($cell, $header = 0) {
return $output;
}
+/**
+ Returns themed table.
+
+ @param $title the form element's title
+ @param $value the form element's data
+ @param $description the form element's description or explanation
+
+ @return a string contraining the \a node output.
+**/
+
+function theme_form_element($title, $value, $description = 0) {
+ return "<div class=\"form-item\">". ($title ? "<label>$title:</label>" : "") . $value . ($description ? "<div class=\"description\">$description</div>" : "") ."</div>\n";
+}
/**
Returns themed table.
diff --git a/misc/drupal.css b/misc/drupal.css
index 8fb6b1615..06f2b8584 100644
--- a/misc/drupal.css
+++ b/misc/drupal.css
@@ -113,7 +113,7 @@ fieldset {
.form-item .description {
font-size: 0.85em;
}
-.form-item .title {
+.form-item label {
font-weight: bold;
margin-top: 1.1em;
margin-bottom: 1px;
diff --git a/modules/blogapi.module b/modules/blogapi.module
index 74a28c8aa..d3aead077 100644
--- a/modules/blogapi.module
+++ b/modules/blogapi.module
@@ -17,10 +17,10 @@ function blogapi_help($section) {
function blogapi_xmlrpc() {
$methods = array('blogger.getUsersBlogs' => array('function' => 'blogapi_get_users_blogs'),
- 'blogger.newPost' => array('function' => 'blogapi_new_post'),
- 'blogger.editPost' => array('function' => 'blogapi_edit_post'),
+ 'blogger.newPost' => array('function' => 'blogapi_new_post'),
+ 'blogger.editPost' => array('function' => 'blogapi_edit_post'),
'blogger.deletePost' => array('function' => 'blogapi_delete_post'),
- 'blogger.getRecentPosts' => array('function' => 'blogapi_get_recent_posts'),
+ 'blogger.getRecentPosts' => array('function' => 'blogapi_get_recent_posts'),
'metaWeblog.newPost' => array('function' => 'blogapi_new_post'),
'metaWeblog.editPost' => array('function' => 'blogapi_edit_post'),
'metaWeblog.getPost' => array('function' => 'blogapi_get_post'),
@@ -31,7 +31,7 @@ function blogapi_xmlrpc() {
'mt.getPostCategories' => array('function' => 'blogapi_get_post_categories'),
'mt.setPostCategories' => array('function' => 'blogapi_set_post_categories')
);
-
+
return $methods;
}
@@ -62,7 +62,7 @@ function blogapi_get_user_info($req_params) {
'firstname' => new xmlrpcval(substr($user->name, 0, strrpos($user->name, " ")), 'string'),
'nickname' => new xmlrpcval($user->name, 'string'),
'email' => new xmlrpcval($user->mail, 'string'),
- 'url' => new xmlrpcval(url('blog/view/' . $user->uid), 'string')),
+ 'url' => new xmlrpcval(url('blog/view/' . $user->uid), 'string')),
'struct');
return new xmlrpcresp($struct);
}
@@ -73,11 +73,11 @@ function blogapi_get_user_info($req_params) {
function blogapi_new_post($req_params) {
global $user;
-
+
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$promote = variable_get("node_promote_blog", 0);
@@ -94,7 +94,7 @@ function blogapi_new_post($req_params) {
$title = blogapi_blogger_title($params[3]);
$body = $params[3];
}
-
+
$node = node_validate(array('type' => 'blog',
'uid' => $user->uid,
'name' => $user->name,
@@ -106,7 +106,7 @@ function blogapi_new_post($req_params) {
'moderate' => $moderate,
'revision' => $revision
), $error);
-
+
if (count($error) > 0) {
return blogapi_error($error);
}
@@ -130,7 +130,7 @@ function blogapi_edit_post($req_params) {
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$node = node_load(array('nid' => $params[0]));
@@ -151,12 +151,12 @@ function blogapi_edit_post($req_params) {
$title = blogapi_blogger_title($params[3]);
$body = $params[3];
}
-
+
$node->title = $title;
$node->body = $body;
$node->status = $params[4];
$node = node_validate($node, $error);
-
+
if (count($error) > 0) {
return blogapi_error($error);
}
@@ -176,10 +176,10 @@ function blogapi_get_post($req_params) {
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
- $node = node_load(array('nid' => $params[0]));
+ $node = node_load(array('nid' => $params[0]));
$blog = new xmlrpcval(array('userid' => new xmlrpcval($node->name, 'string'),
'dateCreated' => new xmlrpcval(iso8601_encode($node->created), "dateTime.iso8601"),
'title' => new xmlrpcval($node->title, 'string'),
@@ -194,10 +194,10 @@ function blogapi_delete_post($req_params) {
global $user;
$params = blogapi_convert($req_params);
-
+
$user = blogapi_validate_user($params[2], $params[3]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$ret = node_delete(array('nid' => $params[1], 'confirm' => 1));
@@ -234,7 +234,7 @@ function blogapi_get_post_categories($req_params) {
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$terms = taxonomy_node_get_terms($params[0]);
@@ -256,7 +256,7 @@ function blogapi_set_post_categories($req_params) {
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$nid = $params[0];
@@ -272,7 +272,7 @@ function blogapi_get_recent_posts($req_params) {
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$res = db_query_range("SELECT n.nid, n.title, n.body, n.created, u.name FROM {node} n, {users} u WHERE n.uid=u.uid AND n.type = 'blog' AND n.uid = %d ORDER BY n.created DESC", $user->uid, 0, $params[3]);
@@ -303,13 +303,13 @@ function blogapi_convert($params) {
function blogapi_error($message) {
global $xmlrpcusererr;
-
+
return new xmlrpcresp(0, $xmlrpcusererr + 1, $message);
}
function blogapi_validate_user($username, $password) {
global $user;
-
+
$user = user_load(array('name' => $username, 'pass' => $password, 'status' => 1));
if (!user_access('access blog API')) {
diff --git a/modules/blogapi/blogapi.module b/modules/blogapi/blogapi.module
index 74a28c8aa..d3aead077 100644
--- a/modules/blogapi/blogapi.module
+++ b/modules/blogapi/blogapi.module
@@ -17,10 +17,10 @@ function blogapi_help($section) {
function blogapi_xmlrpc() {
$methods = array('blogger.getUsersBlogs' => array('function' => 'blogapi_get_users_blogs'),
- 'blogger.newPost' => array('function' => 'blogapi_new_post'),
- 'blogger.editPost' => array('function' => 'blogapi_edit_post'),
+ 'blogger.newPost' => array('function' => 'blogapi_new_post'),
+ 'blogger.editPost' => array('function' => 'blogapi_edit_post'),
'blogger.deletePost' => array('function' => 'blogapi_delete_post'),
- 'blogger.getRecentPosts' => array('function' => 'blogapi_get_recent_posts'),
+ 'blogger.getRecentPosts' => array('function' => 'blogapi_get_recent_posts'),
'metaWeblog.newPost' => array('function' => 'blogapi_new_post'),
'metaWeblog.editPost' => array('function' => 'blogapi_edit_post'),
'metaWeblog.getPost' => array('function' => 'blogapi_get_post'),
@@ -31,7 +31,7 @@ function blogapi_xmlrpc() {
'mt.getPostCategories' => array('function' => 'blogapi_get_post_categories'),
'mt.setPostCategories' => array('function' => 'blogapi_set_post_categories')
);
-
+
return $methods;
}
@@ -62,7 +62,7 @@ function blogapi_get_user_info($req_params) {
'firstname' => new xmlrpcval(substr($user->name, 0, strrpos($user->name, " ")), 'string'),
'nickname' => new xmlrpcval($user->name, 'string'),
'email' => new xmlrpcval($user->mail, 'string'),
- 'url' => new xmlrpcval(url('blog/view/' . $user->uid), 'string')),
+ 'url' => new xmlrpcval(url('blog/view/' . $user->uid), 'string')),
'struct');
return new xmlrpcresp($struct);
}
@@ -73,11 +73,11 @@ function blogapi_get_user_info($req_params) {
function blogapi_new_post($req_params) {
global $user;
-
+
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$promote = variable_get("node_promote_blog", 0);
@@ -94,7 +94,7 @@ function blogapi_new_post($req_params) {
$title = blogapi_blogger_title($params[3]);
$body = $params[3];
}
-
+
$node = node_validate(array('type' => 'blog',
'uid' => $user->uid,
'name' => $user->name,
@@ -106,7 +106,7 @@ function blogapi_new_post($req_params) {
'moderate' => $moderate,
'revision' => $revision
), $error);
-
+
if (count($error) > 0) {
return blogapi_error($error);
}
@@ -130,7 +130,7 @@ function blogapi_edit_post($req_params) {
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$node = node_load(array('nid' => $params[0]));
@@ -151,12 +151,12 @@ function blogapi_edit_post($req_params) {
$title = blogapi_blogger_title($params[3]);
$body = $params[3];
}
-
+
$node->title = $title;
$node->body = $body;
$node->status = $params[4];
$node = node_validate($node, $error);
-
+
if (count($error) > 0) {
return blogapi_error($error);
}
@@ -176,10 +176,10 @@ function blogapi_get_post($req_params) {
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
- $node = node_load(array('nid' => $params[0]));
+ $node = node_load(array('nid' => $params[0]));
$blog = new xmlrpcval(array('userid' => new xmlrpcval($node->name, 'string'),
'dateCreated' => new xmlrpcval(iso8601_encode($node->created), "dateTime.iso8601"),
'title' => new xmlrpcval($node->title, 'string'),
@@ -194,10 +194,10 @@ function blogapi_delete_post($req_params) {
global $user;
$params = blogapi_convert($req_params);
-
+
$user = blogapi_validate_user($params[2], $params[3]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$ret = node_delete(array('nid' => $params[1], 'confirm' => 1));
@@ -234,7 +234,7 @@ function blogapi_get_post_categories($req_params) {
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$terms = taxonomy_node_get_terms($params[0]);
@@ -256,7 +256,7 @@ function blogapi_set_post_categories($req_params) {
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$nid = $params[0];
@@ -272,7 +272,7 @@ function blogapi_get_recent_posts($req_params) {
$params = blogapi_convert($req_params);
$user = blogapi_validate_user($params[1], $params[2]);
if (!$user->uid) {
- return blogapi_error(t('error validating user'));
+ return blogapi_error(t('error validating user'));
}
$res = db_query_range("SELECT n.nid, n.title, n.body, n.created, u.name FROM {node} n, {users} u WHERE n.uid=u.uid AND n.type = 'blog' AND n.uid = %d ORDER BY n.created DESC", $user->uid, 0, $params[3]);
@@ -303,13 +303,13 @@ function blogapi_convert($params) {
function blogapi_error($message) {
global $xmlrpcusererr;
-
+
return new xmlrpcresp(0, $xmlrpcusererr + 1, $message);
}
function blogapi_validate_user($username, $password) {
global $user;
-
+
$user = user_load(array('name' => $username, 'pass' => $password, 'status' => 1));
if (!user_access('access blog API')) {