All Addons are DONE (except for templates)

I just finished the coding for the Answers addon, this means all the addons code is DONE. The one exception is that the links templates are NOT done. I am now starting to package them up and will post the ZIP file in a little while.

Views: 74

47 thoughts on “All Addons are DONE (except for templates)”

  1. nexia dit :

    oh, i thought you’d be doing a birthday cake with it….

    zzzzzz

    rofl… i’ll be waiting for it. i’m a bit slow these days so it will make me take a better beat, i was sleeping on my keyboard.

  2. Drew dit :

    Ok so here are the files.. Obviously beta stage (beta warning here).

    There are definitely some bugs and some smaller errors somewhere but for what i have tested it works.

  3. nexia dit :

    i’ll take a look, and i’m refilling my paypal account… lol

  4. nexia dit :

    ok, first bug, i THINK it’s a major one:

    Warning: require_once([path]/mtf/class_mtf.php) missing file… 🙂

    hey, did you clean the files? the file « mtf_master_functions.php » is still there even if not needed anywhere…

  5. Drew dit :

    Ok here is the updated version

  6. nexia dit :

    here is the update. i tested only the installation and found a lot of bugs, like missing TABLE_PREFIX all around… always work with a prefix, it help debugging.

    also, there are missing fields in the mtf_forumfield table, i don’t know if i added the right ones, i suppose we need to find all about this.

    also, compare the files, there is some edits all around.

  7. Drew dit :

    Ok sorry about the bugs 🙁 i was going for a beta version..

    Anyway here is a script i made to make that makes the install code for the mods. Put it in the forum root with a URL of mtf_installcode.php?id=prouductid

  8. nexia dit :

    no problem for the bug, ya know, i’m the beast here, if i see no bugs, i would not hire you.. lol

    also, that’s why i always say to work on a clean vB, with nothing else than a test ground… add prefix, default everything…

    oh, i forgot …. THANKS!… you’ve done some great job yet, and it’s promissing.

  9. nexia dit :

    note: missing a space between the two parts here: FROM » …. (just find where.. lol)

  10. Drew dit :

    @nexia 20895 wrote:

    note: missing a space between the two parts here: FROM » …. (just find where.. lol)

    On the install code generator?

  11. nexia dit :

    😉 …

    you’re quick… rofl

  12. Drew dit :

    lol i try.. btw do you think i should make the install code generator into a product so when we Export a product it will add it automatically?

  13. Drew dit :

    Btw: Due to how many errors there, just dont even bother with this version i will work with it a little more then re-release it with a lot less errors

  14. nexia dit :

    this part is not so important right now anyway, we fix the core right now, so we add extensions when everything is fine… we’re on beta stage, it can take a lot of time… 🙂

  15. Drew dit :

    Yeah i think the beta will actually go pretty quickly a lot of the errors are being fixed with a condition (i.e. foreach errors)

  16. nexia dit :

    not only the foreach, i forgot to mention the « array_merge()… »

    there is one that block us on the forumdisplay, i am not sure how to fix it to work…

  17. Drew dit :

    Ok here is an updated version with a lot less errors (only on the master engine and the Answers engine)

  18. nexia dit :

    a lot less, you mean you planned all these errors ?!

    i’m confused… rofl

  19. nexia dit :

    oh. btw, another feature: « answered/supported » …

    it’s simple i think, but can be exploided largerly in the future. in forum X, when staff answer a thread officially « with the voice of the staff », that post can become the support answer, and be tagged as well… so the members find the staff answers easily instead of reading hundred of answers from anybody, without finding the official answer…

    i first find that idea from the Howard Stern’s official forum (customized by me)… it was asked that each time Howard answer in a thread, his posts have a different postbit, flashy, so everybody knows he answered it… and in the forumdisplay the threads where he answered are flagged…

  20. Drew dit :

    Yeah that would make sense have the « staff » answer and then the members answer and in each add like an image marking it as so ( i always planned the image part but i hate photo shop cs3 :p)

  21. nexia dit :

    no need for an image, just an additional class in the table… will make it different color (like when you check it for inline mod…)

    i can do that simply.

    .. and i work with photoshop everyday

  22. Drew dit :

    Yeah I will add the code for the classes and then you can add the CSS

    btw: i use to like photo shop but once i got my mac i didn’t seem to like it as much

  23. nexia dit :

    ok, i still face the error on all showthread pages:

    Quote:
    Warning: array_merge() URL= »https://unpapapro.com/your-questions-our-answers/event-forums-update/4832/function.array-merge »]function.array-merge[/URL: Argument #1 is not an array in [path]/showthread.php(984) : eval()’d code on line 2

    Warning: array_merge() URL= »https://unpapapro.com/your-questions-our-answers/event-forums-update/4832/function.array-merge »]function.array-merge[/URL: Argument #2 is not an array in [path]/showthread.php(984) : eval()’d code on line 2

    i do not understand that array_merge, so i can not fix it… i think it’s the only bug that block me from using this version…

  24. Drew dit :

    Yeah i’ve fixed it i will upload a update version after one last thing (probably today).

  25. Drew dit :

    Ok one other thing it doesn’t look like we can replace the CSS of the postbit for staff or best answer posts.. another idea?

  26. nexia dit :

    did i mention i was the one dealing with CSS ??… tsss kids.

    just give me a working version and i’ll add these details. 🙂

  27. Drew dit :

    Oh ok i’ll get it up ASAP

  28. nexia dit :

    you did not correct my error report in post #24:

    $mtf_fields = ($foruminfo == ‘default’ OR $foruminfo ==  ») ? array_merge($vbulletin->mtf_postfield, $vbulletin->mtf_postfield) : $vbulletin->mtf_postfield[« $foruminfo[mtf_type] »];

    the array_merge … 1, i don’t know what it is used for, we can not merge two variables that are not arrays, and 2, we need to check if they exists before the merge… 🙂

  29. Drew dit :

    Yeah i thought i had fixed it.. Well here is an update version..

  30. nexia dit :

    ok, bug fixed…nope… anyway, found why…

    hook: fetch_threadinfo_query … we absolutely need a mtf_postfield to make this error go, so we need to verify everything. i had that error until i create a post field.

    i suppose it’s the same for the other arrays… so we need to make sure all these foreach are correct.

    note to myself: kick Drew’s ass about a clean install…

  31. Drew dit :

    I don’t know why the error never comes up on my test bored :confused:

    Anyway whats the new error?

  32. nexia dit :

    @Drew 21024 wrote:

    I don’t know why the error never comes up on my test bored :confused:

    because your board was bored, you say it yourself.. lol

    error wrote:
    Warning: Invalid argument supplied for foreach() in [path]/includes/functions.php(1237) : eval()’d code on line 7

    Warning: Cannot modify header information – headers already sent by (output started at [path]/includes/class_core.php:3277) in [path]/includes/functions.php on line 3087

    when we have no post field, the error come up…

  33. Drew dit :

    hmm.. I am gonna keep looking at it.. i’ll get it in a few minutes

  34. Drew dit :

    i think i know what it is.. I think it has some thing to do with old postfield cache.. Because i think i had that error until i cleaned the cache i will add a new page to the MTF admin called « Clean Up » where you can rebuild all the caches and that should really fix most errors..
    And the reason adding a new field worked was because that re built the cache.

  35. nexia dit :

    when i delete the postfield the error is back, so it’s not only cache, it’s empty values…

  36. Drew dit :

    hmm it is on this site right?

  37. nexia dit :

    yeap…

    but uninstalled… can not afford the site down too long.. 🙂

  38. Drew dit :

    GOT IT! An easy fix just needed an extra conditional here is the updated product.

  39. nexia dit :

    yeap, fixed… 😉

  40. nexia dit :

    ok, i’m not really happy with all that i had to fix this morning. i had to recode a lot of things and fix a lot of typos during install of any addon, and i’m not even able to make them work because the postfields have no phrases related to them, there is duplicate functions that are not even related to anything in our code, and the system was not made to delete the postfield_data fields so i had to fix that also…

    sorry to see what was NOT done in the last 3 weeks since you said all and everything was done and working. you never tested the installation, you never checked on a clean vBulletin and you did not even take the time to clean your code, so i have to make it myself… so i lost a month of work waiting for you to bring me a solid result, as you said it would me.

  41. Drew dit :

    Yeah finally 🙂

    Also now what is happening? Do you want me to test for performance? and do bug fixes or do you want to templates then performance checking?

  42. nexia dit :

    nah, performances are ok, there is not much to test… vBulletin code is vBulletin, we can not really test the PHP structure itself, even if i know it’s better than the default vBulletin.

  43. Drew dit :

    Ok. And just making sure you are making the templates for the addons right?

  44. nexia dit :

    yeap. i’ll be testing the installation by the day and try to finish the templates (which is easy, but hard to understand when you know nothing of CSS)…:)

    that’s one thing i know for sure…

    so that’s it for the day.

  45. Drew dit :

    Ok, while your doing that i will be testing and working on MTF Releases (which will be ready for June btw)

  46. nexia dit :

    that’s all your engine, you do not have to ask for permissions… 🙂

    i’ll post my updated MTF-Master in no long… i did not re-write that much, but i cleaned the indentation and some comments etc…

  47. Drew dit :

    Ok, sounds good. After that a little more testing and i think with out any huge delays May
    15 could be the release date.

Laisser un commentaire

Ce site utilise Akismet pour réduire les indésirables. En savoir plus sur comment les données de vos commentaires sont utilisées.

En savoir plus sur Un Papa Pro

Abonnez-vous pour poursuivre la lecture et avoir accès à l’ensemble des archives.

Continue reading