Några frågor till Rickard medan jag skriver om Attachment modden :) (Page 1 of 2)

1. Låter bra. attach_ är lite tydligare, så det föreslår jag.

2. Förstår inte riktigt vad du menar. Om du uppdaterar config-tabellen måste du anropa generate_config_cache() som i t.ex. admin_options.php.

3. Japp, $pun_user finns alltid och innehåller vettig information.

4. Tja. 2.0 kan du nog köra på med gott samvete.

Det ska nog inte vara några problem att köra via POST nej.

Ser fin-fint ut! Fint jobbat Det enda jag har att påpeka på är att det känns lite knas att separera filändelser med punkt. Är inte mellanslag lite mer lämpligt?

Oh, en annan sak också. De där sakerna som står i fetstilt direkt till höger om textboxarna känns lite felplacerade. Kan du inte ta med det i själva brödtexten under textboxarna istället?

visstja, ang separationen, jag har inte bestämt mig, men det är ju faktiskt möjligt att ha filnamn med mellanslag i extension, men punkter är ju det som generellt används för att avgöra vad som är extension och inte (det är det jag använder för att kika vilken extension en fil har, plockar ut det som finns efter sista punkten)
T.ex. går det ju att skapa en fil "testfil.nisse hult" filsystemet klagar inte ... så om jag skulle använda mellanslag blir det dels att jag blir tvingad att ändra i funktionen som klurar ut extension, samt att det blir mer att hålla reda på för den som matar in

samma anledning varför jag använder * i 'arrayen' för ikonfilernas namn, så vitt jag vet får man inte använda * i namn (eftersom de ofta används i sökningar), mellanslag och punkter är ju tillåtna ...

Ev att jag kanske kan använda något som filnamn generellt inte får innehålla, typ slash eller dyl ... så att alla fält använder samma separator

Edit: (windows klagar när man försöker ändra filnamn till något med   \ / : * ? " < > |   ... så ev någon av de går att använda i alla rutor där jag behöver använda separatorer?)
Edit2: Har du någon bra koll vilka som *nix inte vill ha i filnamn? (så kanske man får något som båda ogillar, och kan använda)
Edit3:   ... har testat, och tja, det som ser bäst ut i inputfälten av ovanstående tror jag är "

ok, det går inte att skicka med get variabler till administrations plugin såg jag nu

"There is no plugin called 'AP_Attachment_Mod_2.0.php?alter_settings=1' in the plugin directory."

borde gå att göra så att man använder strängen framför ? för att klura ut vilken sida, så att inte GET variabler blir omöjliga? (t.ex. nu när jag uppdaterat inställningarna vill jag ju gärna att man kommer tillbaka till sidan med inställningar, som det är nu blir jag skickad tillbaka till 'huvudmenyn' för modden, för att skicka med POST variabler funkar väl inte med redirect grejen, eller hur?)
Ev att jag kan skriva om det från att använda en ifsats till många, och sätta lite variabler etc. men jag gillar if, elseif, else slingor

Det där är inte hela sanningen. Ja, foreach är snabbare i vissa fall, men i gengäld arbetar den på en kopia av arrayen, så minnesanvändningen är mycket större.

h1 är nog inte så lämplig att använda där med tanke på att det är en header "under" h2. h3 och neråt går bra.

Det beror nog fan på (jag kommer faktiskt inte ihåg). Vilket skript är det du undrar över?

Jag har problem!

Ett klassiskt "query till databasen skriver över befintlig query" fall verkar det som vid första anblick ... använder egna $result_attach och $result_attach_two, istället för $result, för det som $db->query returnerar ... men ändå stannar outputten i viewtopic efter första posten i en tråd med 3 poster... kommenterar jag ut mitt attachment block med kod funkar det som det skall... klipper in det här...

    // Attachment Mod Block Start
    $attach_allow_download = false;
    $attach_output = '';
    $attach_num = 0;
    // Check if this post has any attachments
    $result_attach = $db->query('SELECT af.id, af.filename, af.extension, af.size, af.downloads FROM '.$db->prefix.'attach_2_files AS af WHERE af.post_id='.$cur_post['id']) or error('Unable to fetch if there were any attachments to the post', __FILE__, __LINE__, $db->error());
    $attach_num = $db->num_rows($result_attach);
    if($attach_num > 0){
        //fetch the rules of the forum, and check so that the user is allowed to download.
        $result_attach_two = $db->query('SELECT ar.rules FROM '.$db->prefix.'attach_2_rules AS ar, '.$db->prefix.'topics as t WHERE ar.group_id=\''.$pun_user['group_id'].'\' AND t.id=\''.$id.'\' AND ar.forum_id=t.forum_id LIMIT 1')or error('Unable to fetch rules for the attachments', __FILE__, __LINE__, $db->error());
        if($db->num_rows($result_attach_two)==1){
            list($attach_rules)=$db->fetch_row($result_attach_two);
            //check if the user is allowed to download it.
            $attach_allow_download = attach_rules($attach_rules,ATTACH_DOWNLOAD);
            if($attach_allow_download||$is_admin_group){
                if($is_admin_group)$attach_allow_download = true;
                $attach_output .= $lang_attach['Attachments:'].' ';
                while(list($attachment_id, $attachment_filename, $attachment_extension, $attachment_size, $attachment_downloads)=$db->fetch_row($result_attach)){
                    $attach_output .= '<br />'."\n".attach_icon($attachment_extension).' <a href="./attachment.php?id='.$attachment_id.'">'.$attachment_filename.'</a>, '.$lang_attach['Size:'].' '.number_format($attachment_size,0,''.'\'').' '.$lang_attach['bytes'].', '.$lang_attach['Downloads:'].' '.$attachment_downloads;
                }
            }
        }else{
            // we haven't any rules set, so basic rule is to allow nothing...
        }
    }
    // Attachment Mod Block End

Den där borde väl inte avbryta while loopen för poster, då den while loopen använder $result ... fast ... tja ... jag vet inte ...
inga felmeddelanden heller

Edit: mina funktioner använder inte databasen heller

Edit: är alltså den while loop som börjar på rad 189 som bryts (mitt block ligger just innan utmatningen av posten)

Den parametern ser till att databasadaptern använder t.ex. mysql_unbuffered_query istället för mysql_query. En fördel med detta är att det är att PHP inte buffrar resultatet av queryn, så minnesanvändningen sjunker drastiskt. I gengäld betyder det att man måste hämta ut alla radera ur resultatet för att inte blocka. Man kan heller inte använda t.ex. mysql_num_rows() på resultatet.

sådär ... en liten preview hur det kommer vara i viewtopic
Flera attachments per post!

hittat en sak jag inte är überlycklig med, och det är img taggen, att det tvingas att bli ny rad före o efter ... men kanske bara jag som använder små bilder (för stora så tror jag mycket väl att det här sättet är bättre) ... men jag överlever ... lärde mig ju att leva utan subforum, och nu märker jag att jag nästan tycker subforum är dåligt!

:)

attachment.php (nedladdning + View Image) som jag fixat nu fungerade alldeles för bra för att vara lugnande ... efter att ha fixat en liten sak(kika efter $pun_user['g_id']==PUN_ADMIN för att overrida (har inte matat in några regler ännu)), så var den enda 'buggen' att jag hade matat in fel filnamn i databasen så att filerna hette fel o hade fel antal bytes ... men efter den ändringen så vips så funkade fpassthough grejen ...

aja ... blir väl jobbigare saker senare ... kanske :D

hur som helst ... skall kika hur bild taggen matas ut ... skall försöka använda den på Image View sidan också så att det blir lite mer likt (nu ser den sidan rätt mycket ut som 1.1.5:an ... eller ... aja ... vill göra om den lite hur som helst :D

får se om jag hinner påbörja post.php något   (endast 4 editeringar i viewtopic.php, 3 nya ställen med kod, samt att ändra true til false i den där queryn... känns som om folkd skall lyckas med den ändringen ... hoppas det är lika få ändringar i de andra filerna :))

Edit: Image view är nu en kombination av vad som matas ut vid 'message()' (egen title på det blocket annar lika) och att jag använder samma typ av markup som [ img] har, funkar fin fint :) ... o har nu plottrat ihop attach_delete_attachment() funktionen ... otestad ännu ... men tror det skall funka som den är nu :D

Edit2: Uj... går mycket bättre än jag trott, lyckats fixa till en massa funktioner som behövs (fast har inte hunnit testa alla ännu), och även kritiska som den som skapar nya subfolders o kopierar in lite säkerhetsfiler dit (en .htaccess o en index.html) ... testade just den och det enda som jag behövde fixa var att se till att $pun_config['attach_basefolder'] måste vara den absoluta katalogen till min katalog där alla subfolders lagras ... som sagt ... det går för bra för att kännas riktigt bra :) Nu är det hur som helst natt för min del ... har jag lite o göra nästa vecka borde jag hinna färdigt :)

Edit3: Sådär, då har man kodat lite mer i dag, optimize funkar på tabellerna för filer, och gjorde även list attachments ... tror den här typen av listning är mycket smidigare än den jag hade på den gamla modden ... och så länge man är inne på list attachments sidan så kommer den ihåg vilka val man har valt (så man kan enkelt söka efter attachments som tar mest bandbredd, som är störst, som har mest downloads och de senaste ... ganska enkelt :)) Tror inte jag vågar testa delete ännu, tror jag fixar upload innan (jobbigt att använda backup grejor eller mata in nya filer manuellt :D)
Tillbaka till kodning :)