I just downloaded image.module, and it seems like there is something wrong with content creation. node/add/image (from link create content --> image) shows up blank (with only breadcrumbs and title).

Does anyone else experience this? I use drupal cvs.

Comments

TDobes’s picture

Assigned: Unassigned » TDobes

I sent bruno a patch to image.module which corrects this and a variety of other issues with running image.module on the current drupal CVS. He is currently reviewing the changes. In the meantime, the patch is posted below.

Index: image.module
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/image/image.module,v
retrieving revision 1.87
diff -u -r1.87 image.module
--- image.module 6 Dec 2003 01:38:43 -0000 1.87
+++ image.module 6 Dec 2003 18:21:43 -0000
@@ -192,12 +192,12 @@

// Add a create link for images if the user has create permission
if ($type == "system" && (user_access("create images") || _image_can_personal())) {
- menu("node/add/image", t("image"), "page");
+ menu("node/add/image", t("image"), "node_page");
}

// Add view personal gallery link if the user has personal gallery permission if ($type == "system" && user_access("access images") && _image_can_personal()) {
- menu("image/uid/$user->uid", t("my photo gallery"), "page");
+ menu("image/uid/$user->uid", t("my photo gallery"), "image_page");
}

// Add links to an image node
@@ -324,7 +324,7 @@
/**
* Display the image node
*/
-function image_view($node, $main = 0) {
+function image_view($node, $main = 0, $page = 0) {

$rand = "";

@@ -336,7 +336,7 @@
$node->teaser = check_output($node->teaser);
$node->body = check_output($node->body);

- return theme("node_image", $node, $rand, $main);
+ return theme("node_image", $node, $rand, $main, $page);
}

/**
@@ -1144,18 +1144,14 @@

$galleryuser = user_load(array( "uid" => $uid));

- print theme("header");
- print theme("box", t("%u's Photo Gallery", array("%u" => $galleryuser->name)), theme("image_gallery_album", $null, $null, $images, $null, $galleryuser));- print theme("footer");
+ print theme("page", theme("image_gallery_album", $null, $null, $images, $null, $galleryuser), t("%u's Photo Gallery", array("%u" => $galleryuser->name)));
}
else if (!$tid) {
// show albums
$albums = taxonomy_get_children(0, variable_get("image_nav_vocabulary", ""));
$albums = _image_album_properties($albums);

- print theme("header");
- print theme("box", t("Photo galleries"), theme("image_gallery_home", $albums));
- print theme("footer");
+ print theme("page", theme("image_gallery_home", $albums), t("Photo galleries"));
}
else {
// browse albums; show subalbums and thumbs
@@ -1188,15 +1184,11 @@
$n++;
}

- print theme("header");
- print theme("box", t("Photo galleries"), theme("image_gallery_album", $parents, $subalbums, $images, $data));
- print theme("footer");
+ print theme("page", theme("image_gallery_album", $parents, $subalbums, $images, $data), t("Photo galleries"));
}
}
else {
- print theme("header");
- print theme("box", t("Access denied"), message_access());
- print theme("footer");
+ print theme("page", message_access(), t("Access Denied"));
}
}

@@ -1270,7 +1262,7 @@
* Theme function to render image node.
*
*/
-function theme_node_image($node, $rand, $main = 0) {
+function theme_node_image($node, $rand, $main = 0, $page = 0) {
global $base_url;

if ($main)
@@ -1291,7 +1283,7 @@
}
}

- return theme("node", $node, $main);
+ return theme("node", $node, $main, $page);
}

function theme_image_gallery_home($albums) {
@@ -1575,7 +1567,7 @@
}
closedir($handle);
}
- return $output;
+ print theme("page", $output);
}

function _image_check_configuration() {
@@ -1669,7 +1661,7 @@

$output .= theme("table", $headers, $rows);

- return $output;
+ print theme("page", $output);
}

/**

bruno’s picture

Assigned: TDobes » bruno
Priority: Critical » Normal

Yes. I'm working on it. Some of the recent changes to theme system are the cause of this.

But please don't forget that CVS is *constantly* changing, therefore don't expect to have everything working, especialy with contributed modules ;-) Use 'critical' priority with caution. Thanks.

bruno’s picture

Yes. I'm working on it. Some of the recent changes to theme system are the cause of this.

But please don't forget that CVS is *constantly* changing, therefore don't expect to have everything working, especialy with contributed modules ;-) Use 'critical' priority with caution. Thanks.

bruno’s picture

Thanks Tom. I'm incorporating your changes.

bruno’s picture

Patch applied to CVS. Fixes the empty page problem.

Anonymous’s picture

Anonymous’s picture

khiladi344’s picture

Title: Cannot create content (blank node/add/image) » Packers and Movers Hyderabad Offer you the best shifting packages.
Version: » 7.x-1.x-dev
Component: Code » documentation
Issue summary: View changes

Hyderabad Packers and Movers provides high-quality warehouse and storage area space area place assistance for anyone or companies. You can buy for customized warehousing needs that will be fulfilled by the efficient workforce of Hyderabad. Whether you are looking for a cost-effective solution for your company, we offer the best alternatives in the area. With an experience of managing quite a number of industrial and expert warehousing needs, we offer quick results with an awesome accuracy.

Packers and Movers Hyderabad

Not only industrial or expert warehousing, but we in addition offer domestic warehousing needs of clients when a temporary place is required for a few months. It may be due to a temporary shifting or a construction of the new house. We create sure you relocate close affiliates of individuals family members members affiliates aspects at an honest and successfully properly secured place. It saves your persistence in looking after the precious aspects. We keep you hassle-free for managing other essential activities.
If you occasion for shifting from Dwarka town to any other town in Regional local indian local you are here at where. At Hyderabad Packers and Movers in Dwarka we are advanced movers which leads stands for the name excellent, balance, affordability and assurance. We have extensive experience of over 4 years in transportation and shifting market.

Packers and Movers Bangalore
For More Information:-
http://www.movers5th.in/packers-and-movers-hyderabad/
http://www.movers5th.in/packers-and-movers-bangalore/