Welcome to Geeklog, Anonymous Thursday, March 28 2024 @ 09:35 am EDT

Geeklog Forums

Comment hacks


Status: offline

radix2

Forum User
Newbie
Registered: 11/01/04
Posts: 1
G'day,

We are soon to embark upon some coding of some customised comment features and I was wondering if someone could help me with a couple of questions.

1. Is there a field in the comments table that I could use to encode (bitmask) 4 optional attributes? If not, what is the chance of the maintainers adding a "status" field to the official comments schema? The status would use bits 0-3 of a byte field, with 0 being standard (default behaviour), decimal 1, 2 and 4 representing other statuses - some mutually exclusive. Obviously, if accepted, we would have to document what each bit represents.

2. do duplicate functions in lib-custom.php automatically overide those in other libraries (specifically lib-common.php)? If not, what mechanism would be recommended to overide the comments output and administration?

With one (1) above I could of course create another table linked on the comment id, but I was hoping to avoid another join for performance reasons and I want to stick with the officially supported schema if at all possible. Hope someone can point me in the right direction.

Cheers
 Quote

Status: offline

vinny

Site Admin
Admin
Registered: 06/24/02
Posts: 352
Location:Colorado, USA
1. There is no such db column, but you can just a column to the comment table. Geeklog does not clober extra columns on tables during upgrades.

2. No, unfortunately PHP does not support function overriding at all. We could clobber together support for this my doing some magic, but it doesn't seem worth the trouble. A better method would be to request plugin hooks where they don't exist and are needed (within the function to override function behavior on a per plugin basis).

-Vinny
 Quote

Status: offline

Dirk

Site Admin
Admin
Registered: 01/12/02
Posts: 13073
Location:Stuttgart, Germany
There's also a discussion about this on the geeklog-devtalk mailing list.

bye, Dirk
 Quote

All times are EDT. The time is now 09:35 am.

  • Normal Topic
  • Sticky Topic
  • Locked Topic
  • New Post
  • Sticky Topic W/ New Post
  • Locked Topic W/ New Post
  •  View Anonymous Posts
  •  Able to post
  •  Filtered HTML Allowed
  •  Censored Content