Topic
  • 5 replies
  • Latest Post - ‏2007-01-04T22:49:31Z by SystemAdmin
SystemAdmin
SystemAdmin
161 Posts

Pinned topic Feedback on Part 6

‏2006-09-15T04:28:47Z |
Figure 2 (http://www-128.ibm.com/developerworks/ibm/library/i-osource6/#fig2) shows the base "modules" directory with a bunch of extra modules in it.

Best practices are to actually put all non-core modules in your local sites directory. So, for domain.com, you would put all non-core modules in code/sites/adtech.internet.ibm.com/modules[/code]. This keeps the main base of Drupal clean and easily updatable -- you can replace/upgrade everything not in the base directory without having to worry.

Also, as of 4.7, Drupal supports code.install[/code] files, which contain the SQL necessary to automatically initialize the database tables for you. This should be used instead of the code.sql[/code] file shown.

Keep up the good work!
Updated on 2007-01-04T22:49:31Z at 2007-01-04T22:49:31Z by SystemAdmin
  • SystemAdmin
    SystemAdmin
    161 Posts

    Re: Feedback on Part 6

    ‏2006-09-15T05:40:30Z  
    There is a function missing from the tutorial. I think it should be a part of listing 15:
    codefunction _announcement_unixtime2drupaldate($unixtime) {
    $drupal_date = array();
    $drupal_date = date("Y", $unixtime);
    $drupal_date = date("m", $unixtime);
    $drupal_date = date("d", $unixtime);
    return $drupal_date;
    }[/code]
  • SystemAdmin
    SystemAdmin
    161 Posts

    Re: Feedback on Part 6

    ‏2006-09-15T13:02:49Z  
    Great feedback. Thank you Boris. We'll update the article ASAP.
  • SystemAdmin
    SystemAdmin
    161 Posts

    Re: Feedback on Part 6

    ‏2006-09-15T13:04:21Z  
    There is a function missing from the tutorial. I think it should be a part of listing 15:
    codefunction _announcement_unixtime2drupaldate($unixtime) {
    $drupal_date = array();
    $drupal_date = date("Y", $unixtime);
    $drupal_date = date("m", $unixtime);
    $drupal_date = date("d", $unixtime);
    return $drupal_date;
    }[/code]
    Thanks for the feedback. Yep, were working with developerWorks to release the announcements module which includes this.
  • SystemAdmin
    SystemAdmin
    161 Posts

    Re: Feedback on Part 6

    ‏2007-01-03T21:18:42Z  
    not to sound too demanding hopefully (happy new year guys!)

    but would it be possible to include the

    announcement_all()

    function from the announcement module in the listing, or just post as a reply for me here.

    Unless of course, it is there and I have missed it completely.

    Thanks

    James
  • SystemAdmin
    SystemAdmin
    161 Posts

    Re: Feedback on Part 6

    ‏2007-01-04T22:49:31Z  
    First, kudos to you guys for all the work and an excellent job. Also, great to see IBM supporting Drupal and open source in general.

    One of the prior replies mentioned getting the the code for the announcement module. Having all the code for the tutorial would make resolving problems when doing the tutorial much easier. It would be great if it were downloadable.

    In any case, thanks and wishes for continued succes in the new year