This is the community forum. For a developer response use the Client Area.
Follow us on Facebook, Twitter and YouTube!

Reply to comment
#11

I'm about 3/4 done. However, I need to know where the variables such as "$enabled_website" are set. I know about the place in "admin/includes/pages/form_enabled.php", but I'm not sure where they are set for the front-end. Thanks.

I'm giving you three guesses...
Reply
#12

What do you mean by "set"? The $enabled_website variable is extracted from the database by the get_settings() function in includes/functions/page.php

Have you completed the interview?
Reply
#13

Thank you. Exactly what I needed.

I'm giving you three guesses...
Reply
#14

I am done! At least I think I am. I'll show you my work tomorrow, so you'll have to wait until then Tongue. For now, you can get a program which shows differences between two files, such as Meld, but it's for Linux. Not sure what options Windows has (probably a bunch), but it's not necessary to have one, it'll just help a lot.

EDIT: This might do http://www.diffchecker.com/

I'm giving you three guesses...
Reply
#15

Good, I'm looking forward to this feature. I already use WinMerge.
http://winmerge.org/

Edit: You're the only one to have downloaded the beta so far so it would be nice to hear your feedback.

Have you completed the interview?
Reply
#16

Right, let me switch to Linux as the new Commentics is there. You should definitely promote development better. WinMerge looks ok, but Meld has a slicker interface. They run on the same engine (diff), so it's good.
OK, here are the files that I have edited. It still needs a little work, and I skipped the installer/upgrader. You'll need to create 4 new tables:
"comments" needs to have a "reply_to" column, with a default value of 0.
"settings" needs to have three rows called "enabled_reply" with a default of 1,"default_reply" with a value of 0, and "default_reply_name" with a value of "no one", all of which should be under the form category.

Among the planed changes are changing the form for replies from "type='text'" to "type='hidden'", improving the "currently replying to" text, better layout in "includes/template/form.php", improve the comments, and a couple of other small changes that I can't think of right now.

The only changes to the css that you need are
Code:
.reply_to {
margin-left: 5em;
margin-right: -5em;
}
And
Code:
.reply_link {
visibility: hidden;
float: right;
font-size: 0.8em;
}

.comment_box_1:hover .reply_link, .comment_box_2:hover .reply_link, .admin_comment_box_1:hover .reply_link, .admin_comment_box_2:hover .reply_link {
visibility: visible;
}

Here are the files which I edited. You should compare them with the originals to see what I changed. I'll explain most of it later, but it's pretty self-explanatory.

"includes/template/comments.php" -> [attachment=39]

"includes/functions/comments.php" -> [attachment=40] //Remove 2 in name


"includes/template/form.php" -> [attachment=41]

"includes/app/processor.php" -> [attachment=42]

"includes/functions/processor.php" -> [attachment=43] //Remove 2 in name


/*"includes/Admin/includes/pages/layout_form_enabled.php" ->
"includes/Admin/includes/pages/manage_comments.php" ->
"includes/Admin/index.php" ->
"includes/Admin/includes/pages/edit_comment.php" -> */ //In next post

PS: I ran into some trouble with the attachments. The names are the same, and it only lets me upload five at a time, so I had to rename the files a little, and split the post.

I'm giving you three guesses...
Reply
#17

Second part:

"includes/Admin/includes/pages/layout_form_enabled.php" -> [attachment=44]

"includes/Admin/includes/pages/manage_comments.php" -> [attachment=45]

"includes/Admin/index.php" -> [attachment=48] //Has some extra lines from an add-on I'm working on.

"includes/Admin/includes/pages/edit_comment.php" -> [attachment=47]

Now that I look at it, I should've zipped it upBlush. I need to wake up...
If you want me to zip it up, I'll do that.
Also note that "includes/functions/comments.php" has some extra styling that I forgot to remove. I think I used it to create different styles, so it doesn't affect this feature. One important change in that file is that I initialize $box as a blank string, to make the code less complicated. So only the first couple of lines and the lines after ~150 are relevant, but I'm not too sure. Check out the whole thing just to be sure.

I'm giving you three guesses...
Reply
#18

I get this error:
Quote:Parse error: parse error in C:\wamp\www\commentics_1.1 - Copy\comments\includes\template\form.php on line 585

I can't see anything wrong with line 585. However when I compare the file using WinMerge, it gives a warning saying that your file contains a different type of carriage return.

Have you completed the interview?
Reply
#19


I'm giving you three guesses...
Reply
#20

Same error I'm afraid. It works fine with the normal code but not with your code.

Have you completed the interview?
Reply


Possibly Related Threads…
Thread / Author Replies Views Last Post
Last Post by Deparis
20-Jul-2014, 09:08 AM
Last Post by mariank
22-Dec-2013, 09:36 AM
Last Post by Steven
21-Apr-2013, 05:26 PM
Last Post by Chafik
08-Jan-2011, 01:55 PM
Last Post by Static
03-Jan-2011, 04:00 PM
Last Post by Steven
14-Aug-2010, 12:29 PM

Forum Jump:


Users browsing this thread: 1 Guest(s)