Port to drupal 6.0
reikiman - February 17, 2008 - 02:48
| Project: | Copyright |
| Version: | HEAD |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Robrecht Jacques |
| Status: | closed |
Jump to:
Description
I ran the Coder (code review) module and it guided me through a few changes for porting to 6.
There are a few remaining issues:
Line 458: new hook_theme() function is required to register theme_ functions (Drupal Docs)
function theme_copyright_admin_overview($form) {
severity: critical Line 85: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
if ($may_cache) {
severity: critical Line 86: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
$items[] = array(
severity: critical Line 93: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
$items[] = array(
severity: critical Line 99: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
$items[] = array(
severity: critical Line 106: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
$items[] = array(
severity: critical Line 112: The menu system has been completely over-hauled in 6.x. (Drupal Docs)
$items[] = array(
severity: critical Line 533: hook_form_alter() parameters have changed (Drupal Docs)
function copyright_form_alter($form_id, &$form) {
copyright.install
severity: critical click to read more Line 17: A new schema API has been added in 6.x (Drupal Docs)
switch ($GLOBALS['db_type']) {
Explanation: This patch caused changes to the format of hook_install(), hook_uninstall(), and hook_update_N(). No longer are switch statements done on $GLOBALS['db_type']; instead, use the variety of schema API functions to perform table manipulation.
severity: critical Line 176: db_next_id is deprecated. Use db_last_insert_id instead. (Drupal Docs)
$cpyid = db_next_id("{copyrights}_cpyid");
severity: critical click to read more Line 208: A new schema API has been added in 6.x (Drupal Docs)
switch ($GLOBALS['db_type']) {
Explanation: This patch caused changes to the format of hook_install(), hook_uninstall(), and hook_update_N(). No longer are switch statements done on $GLOBALS['db_type']; instead, use the variety of schema API functions to perform table manipulation.
severity: critical click to read more Line 296: A new schema API has been added in 6.x (Drupal Docs)
switch ($GLOBALS['db_type']) {| Attachment | Size |
|---|---|
| copyright.diff | 4.82 KB |
#1
#2
#3
How is the deveolpement of this port? I am really interested. Thank you!
#4
is there anybody working on this? thanks!
#5
In order to get this issue alive and kicking again, Webchick released a nice screencast how one can easily help porting this module: http://www.lullabot.com/node/397/play
Give it a try if you really want this module for D6.
#6
I've started the port. ETA Sunday August 10th.
#7
thanks for your work!!
#8
any news on this?
#9
is the "diff" a patch for make it to work with drupal 6?
#10
@robrecht in case you have some code already for the D6 version of copyright, could you check it in? I have the time now to jump on this port job.
#11
@toemaz: no I don't have any code yet. If you can update the patch, I'll commit and work together on solving the remaining issues. Thanks!
#12
will be useful. subscribing.
#13
This port is important to the Ubuntu/Canonical community. I will be able to start work on it starting next week if it is still needed. However, if you think you can finish this in a timely fashion - I can let you work on nit.
#14
@MTecknology
I'm a user of this module as well on D5 and while considering to port it to 6, I was wondering whether this module has not become redundant if you consider that a CCK field can be used to add a license to a node as well. By overriding the theme for the license field, one can easily mimic the same behaviour of the copyright module. What's more: by using a CCK field, it will exposed to Views, Faceted Search, ... and it will be easier to maintain and upgrade the data when moving up to D7.
#15
@toemaz
I think use this with integration with CCK could be a more than a great solution, maybe we can start a feature request.
#16
Feel free to reopen if this is still an issue. I'm just closing it because it's been inactive for a long time.