Sign in to follow this  

My News system

Recommended Posts

Haggis    981

Ok i made a news system

Mainly because i wanted to see if i could do it

Ignore the actual site layout it was just thrown together so i could work on it as if it was part of a site

Is there anything you would add?

Anything you would change?

anything you would remove?

http://www.haggistech.co.uk/news

  • Like 1

Share this post


Link to post
Share on other sites
LUTZIFER    473

Sweet! Looks awesome so far. Good work!!

  • Like 1

Share this post


Link to post
Share on other sites
Haggis    981

Thanks

Been working on it for a wee while now just in between work etc

the main page i feel is about finished maybe some small tweaks

Totally lost on admin page though

Share this post


Link to post
Share on other sites
Lingwo    489

If you're not sure what to do for the admin page.

Look at bootstrap

http://twitter.github.com/bootstrap/

You can whip up a decent looking administration backend pretty quickly.

As for functionality, you will want to include things like comment moderation, News CRUD. A user CRUD also.

Share this post


Link to post
Share on other sites
game_over    709

Very good work, it's nice, clean and does it's job - just a couple of things:

- index.php?page=comments&id=54 - you should definitely use a slug: /facebook-hosting-windows-8-hackathon, page=comments is not a good representation

- also show news title in the page <title></title>, it still says Home even when it's not on 'home'

-

Posted by: Haggis @ November 30, 2012

.. you can't post something at @ a date.

Share this post


Link to post
Share on other sites
Haggis    981

Very good work, it's nice, clean and does it's job - just a couple of things:

- index.php?page=comments&id=54 - you should definitely use a slug: /facebook-hosting-windows-8-hackathon, page=comments is not a good representation

- also show news title in the page <title></title>, it still says Home even when it's not on 'home'

-

Posted by: Haggis @ November 30, 2012

.. you can't post something at @ a date.

Why is it not good to use ?page=comments for example?

I will fix the @ to on lol

and yeah the | home i will fix too

thanks :)

Share this post


Link to post
Share on other sites
WildWayz    100

Isn't it SEO indexing? You'll get more hits from search engines if the title is in the link...

Share this post


Link to post
Share on other sites
game_over    709

Why is it not good to use ?page=comments for example?

I will fix the @ to on lol

and yeah the | home i will fix too

thanks :)

Just my personal take on it: it's not really a comments page, it's a news page with comment functionality... Using something like the page/news title as the url is much more relevant.

And as above.. SEO purposes.

Share this post


Link to post
Share on other sites
ACTIONpack    360

It works but needs a lot of css styling. It's looks like a shell only.

Share this post


Link to post
Share on other sites
Haggis    981

It works but needs a lot of css styling. It's looks like a shell only.

Did you read the first post?

in particular thisbit

Ignore the actual site layout it was just thrown together so i could work on it as if it was part of a site

You forgot to validate the ID

http://www.haggistec...nts&id=mouhhaha

will give a mysql error...

Good job for a first try :) !

Thank you, i will fix that :)

Share this post


Link to post
Share on other sites
Sandor    436

What check are you doing on the ID to ensure it's legit?

Share this post


Link to post
Share on other sites
Haggis    981

Its been so long since i did that bit of code i will ahve to check lol

Share this post


Link to post
Share on other sites
game_over    709

yeah i tried to hijack it using the id and every attempt failed.

Share this post


Link to post
Share on other sites
Haggis    981

Awesome :)

Share this post


Link to post
Share on other sites
Sandor    436

i am escaping the string and should now not work like that

this should work now

http://www.haggistec...ents&id=muhahah

escaping...assuming you're using mysqli_real_escape_string then?

Not necessarily secure from what I've read.

Thoughts on:


$id = preg_match("/^[0-9]+$/", $_GET['id']);
[/CODE]

Doing anything on the page= btw?

Share this post


Link to post
Share on other sites
Haggis    981

yeah looking at the page= thing but not got a lot of time right now

so getting rid of the small problems first :)

Share this post


Link to post
Share on other sites
Sandor    436

is the page used in the query? if not, no need to worry. only need to worry about things directly used in the query really.

Share this post


Link to post
Share on other sites
Haggis    981

nah page is not used in the query only the id

I thought you meant the SEO stuff lol

Share this post


Link to post
Share on other sites
Sandor    436

These days it's a bit of a myth about the address affecting SEO. Always a good idea to include the title though simply because people search for words.

/news/index.php?page=comments&id=54 isn't brilliant.

/news/index.php?page=comments&title=facebook-hosting-windows-8-hackathon is better (and Google can understand query strings)

My own personal blog that I'm creating uses entry.php?entry=title-of-post&category=category-of-post

Share this post


Link to post
Share on other sites
Haggis    981

hmmm never thought about that :) i will have a look

Share this post


Link to post
Share on other sites
Haggis    981

ok got the mod_rewrite stuff working

does this look better now?

http://haggistech.co.uk/news/

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

Sign in to follow this  

  • Recently Browsing   0 members

    No registered users viewing this page.