Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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']) {
Comment | File | Size | Author |
---|---|---|---|
copyright.diff | 4.82 KB | reikiman |
Comments
Comment #1
Robrecht Jacques CreditAttribution: Robrecht Jacques commentedComment #2
Robrecht Jacques CreditAttribution: Robrecht Jacques commentedComment #3
davebv CreditAttribution: davebv commentedHow is the deveolpement of this port? I am really interested. Thank you!
Comment #4
davebv CreditAttribution: davebv commentedis there anybody working on this? thanks!
Comment #5
toemaz CreditAttribution: toemaz commentedIn 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.
Comment #6
Robrecht Jacques CreditAttribution: Robrecht Jacques commentedI've started the port. ETA Sunday August 10th.
Comment #7
davebv CreditAttribution: davebv commentedthanks for your work!!
Comment #8
davebv CreditAttribution: davebv commentedany news on this?
Comment #9
davebv CreditAttribution: davebv commentedis the "diff" a patch for make it to work with drupal 6?
Comment #10
toemaz CreditAttribution: toemaz commented@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.
Comment #11
Robrecht Jacques CreditAttribution: Robrecht Jacques commented@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!
Comment #12
nirvanajyothi CreditAttribution: nirvanajyothi commentedwill be useful. subscribing.
Comment #13
MTecknology CreditAttribution: MTecknology commentedThis 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.
Comment #14
toemaz CreditAttribution: toemaz commented@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.
Comment #15
davebv CreditAttribution: davebv commented@toemaz
I think use this with integration with CCK could be a more than a great solution, maybe we can start a feature request.
Comment #16
MTecknology CreditAttribution: MTecknology commentedFeel free to reopen if this is still an issue. I'm just closing it because it's been inactive for a long time.