Pagina 1 van 1

[DEV] responsive banners

Geplaatst: 24 feb 2016, 23:01
door Theriddler
Extension Name: Responsive Banners
Author: theriddler en John P
Former Author: Cece74


Extensie beschrijving:
phpBB 3.1 extensie module om (reclame) banners te tonen onder het forum in de footer.
Gebasseerd op de ABD extensie code van Nivoslider by cece74 maar met andere events en js code.
Groep Permissies zorgen ervoor dat je de banners alleen aan bepaalde groepen kunt laten zien.

Demo: http://www.peppersrus-forum.nl

Extensie Versie: 1.0.5


Version 1.0.3
***30/02/2016***
+ [Added] Group perrmissions (which group can see banners)

Version 1.0.4
***01/03/2016***
+ [Fix] Group permissions (John P)
+ Large code changes (John P)
+ Code cleanup (John P)

Version 1.0.5
***02/03/2016***
+ [Added] missing {L_COLON} in acp template (dmzx)
- [Removed] Ext.php as it is not neccesary (dmzx)
+ Cleanup adding TABS. (dmzx)

Version 1.0.6
***08/03/2016***
+ [Fix] banners not showing when Enable URL Rewriting was set to off (dmzx)


Extension is still in [DEV] it is not recommended for live forums as it's still in development.
Use at your own risk.


Big credits goed to John P (forumhulp.com) for helping out! and making this a better candidate to be released to the phpbb.com extension database.

Extensie Download:
https://www.phpbbservice.nl/forum/viewt ... 188#p76188


Installatie:
  1. 1. Download het pakket
    2. Pak deze uit.
    4. Upload de map `theriddler` naar `/ext/`
    5. Navigeer in het beheerderspaneel naar `aanpassingen -> beheer extensies`
    6. Zoek in de lijst naar `responsive banners` onder de uitgeschakelde Extensies , en klik op de`inschakelen` link.
    7. Upload banner afbeeldingen via tabblad extensies -> responsivebanners -> beheer
Updaten:
  1. 1. Navigeer in het beheerderspaneel naar `aanpassingen -> beheer extensies`
    2. Zoek in de lijst naar `responsive banners` onder de uitgeschakelde Extensies , en klik op de`uitschakelen` link.
    3. Verwijder de map responsivebanners uit /ext/theriddler
    4. Download het nieuwe pakket
    5. Pak deze uit.
    6. Upload de map `theriddler` naar `/ext/`
    7. Navigeer in het beheerderspaneel naar `aanpassingen -> beheer extensies`
    8. Zoek in de lijst naar `responsive banners` onder de uitgeschakelde Extensies , en klik op de`inschakelen` link.
note:
De maximale breedte van de banner kan geregeld worden in \responsivebanners\styles\all\template\event\overall_footer_after.html

Verander:

Code: Selecteer alles

maxwidth: 468,
wellicht maak ik daar nog een configuratie voor in het acp.
zodat je gewoon een breedte kunt invullen, zonder aanpassingen aan dat bestand.

Re: [DEV] responsive banners

Geplaatst: 24 feb 2016, 23:24
door El torro
Ik neem aan dat het ook zeer wordt afgeraden op deze dev versie op een live forum te gebruiken?

Re: [DEV] responsive banners

Geplaatst: 25 feb 2016, 09:26
door Theriddler
El torro schreef:Ik neem aan dat het ook zeer wordt afgeraden op deze dev versie op een live forum te gebruiken?
In de regel wel inderdaad maar ik kan je verzekeren dat het prima werkt.
De acp code van nivoslider om te uploaden van cece47 heb ik gebruikt als basis voor deze extensie.
Het enige wat veranderd is ,is wat andere css codes, een andere js lib en wat andere events/templates.
zodat de banners onderin responsive zijn en dus verkleint als men op een mobiel of klein scherm kijkt.
Ik kan wel zeggen dat je deze gerust kan installeren.

Feedback hoor ik graag.

een ander idee wat ik nog had is permissie toevoegen zodat je per gebruiker groep kan instellen of men de banner kan zien, denk aan bijv vips die dan geen reclame gaan zien.

Re: [DEV] responsive banners

Geplaatst: 25 feb 2016, 10:18
door John Leeuwis
Theriddler schreef:........... een ander idee wat ik nog had is permissie toevoegen zodat je per gebruiker groep kan instellen of men de banner kan zien, denk aan bijv vips die dan geen reclame gaan zien.
Dat zou de extensie een stúk interessanter maken.
Gewone leden etc. zien de reclame wel en de donateurs zijn daar vrij van.
Gaat zeker werken (voor de kas) :D

Re: [DEV] responsive banners

Geplaatst: 25 feb 2016, 12:46
door Paul
Theriddler schreef:
El torro schreef:Ik neem aan dat het ook zeer wordt afgeraden op deze dev versie op een live forum te gebruiken?
In de regel wel inderdaad maar ik kan je verzekeren dat het prima werkt.
Tja, dat zegt iedere extensie author, ook degene met security issues erin :). Prima werkt betekend niet dat hij veilig is (Ik zeg ook niet dat hij niet veilig is ;)), zolang het extensie team op phpBB.com er niet naar gekeken heeft zou je hem gewoon niet live moeten gebruiken.
Ook zou ik niet zomaar de extensies van cece pakken. Er zijn een groot aantal van zijn extensies afgekeurd door phpBB.com vanwege allerlei issues. Als ik even snel kijk naar jouw ACP module zie ik al dat die van jouw ook niet door de validatie komt :),

Code: Selecteer alles

$target_dir = '' . $phpbb_root_path . 'ext/theriddler/responsivebanners/styles/all/theme/images/';
Extensies mogen geen files uploaden namelijk in hun eigen extensie folder, maar moeten dit doen in een folder die blijft bestaan als extensie is verwijderd, bijvoorbeeld store/.

Re: [DEV] responsive banners

Geplaatst: 25 feb 2016, 18:24
door Theriddler
thnx paul voor de feedback en duidelijk, ik ga er naar kijken.

@John ook daar ga ik binnenkort mee aan de slag. staat op mijn todo lijstje :)

Re: [DEV] responsive banners

Geplaatst: 01 mar 2016, 13:52
door Theriddler
*** Extensie Geupdate

Re: [DEV] responsive banners

Geplaatst: 01 mar 2016, 23:07
door dmzx
In "version": "1.0.4"

ext.php? + * Extension Quickstart Package.

In acp_responsivebanners_body.html gebruikt {L_COLON} voor :

In listener.php gebruik TAB key ;)

In permissions_responsivebanners.php EN en NL staat:

Code: Selecteer alles

/**
*
* BB Points extension for the phpBB Forum Software package.
*
* @copyright (c) 2015 ForumHulp.com <http://forumhulp.com>
* @license Proprietary
*
*/
grtz dmzx

Re: [DEV] responsive banners

Geplaatst: 02 mar 2016, 16:46
door Theriddler
dmzx schreef:In "version": "1.0.4"

ext.php? + * Extension Quickstart Package.

In acp_responsivebanners_body.html gebruikt {L_COLON} voor :

In listener.php gebruik TAB key ;)

In permissions_responsivebanners.php EN en NL staat:

Code: Selecteer alles

/**
*
* BB Points extension for the phpBB Forum Software package.
*
* @copyright (c) 2015 ForumHulp.com <http://forumhulp.com>
* @license Proprietary
*
*/
grtz dmzx
Thnx dmzx voor het reviewen, de problemen/punten die je aangaf zijn inmiddels verholpen. een update komt er zo spoedig aan.
V 1.0.5 ;)
Extensie Geupdate naar 1.0.5
- Theriddler

Re: [DEV] responsive banners

Geplaatst: 05 mar 2016, 22:31
door Solidjeuh
Bedankt! Ik had dit al op mijn forum, waar was geen extensie..
Is veel makkelijker nu!

Juist 1 klein vraagje, is het ook mogelijk om de banners random te tonen?
Ik had dit in het script toegevoegd ( startSlide: "shuffle" )maar hoe kan ik dit in jouw extensie toevoegen of wijzigen?

En volgens mij mist deze vertaling nog in het ACP bestandje:

Code: Selecteer alles

	'TITLE'							        => 'Titel',
Nogmaals bedankt voor de extensie! :D

Re: [DEV] responsive banners

Geplaatst: 07 mar 2016, 14:25
door Theriddler
Solidjeuh schreef:Bedankt! Ik had dit al op mijn forum, waar was geen extensie..
Is veel makkelijker nu!

Juist 1 klein vraagje, is het ook mogelijk om de banners random te tonen?
Ik had dit in het script toegevoegd ( startSlide: "shuffle" )maar hoe kan ik dit in jouw extensie toevoegen of wijzigen?
Ja dat is mogelijk.

open: \responsivebanners\styles\all\template\event\overall_footer_after.html

Zoek:

Code: Selecteer alles

auto: true,
Plaats daaronder op een nieuwe regel:

Code: Selecteer alles

random: true,
Vergeet niet de forum cache te legen wanneer je het bestand terug upload.

Meer opties vind je hier:
http://responsiveslides.com/

kijk bij: 5. Options you can customize

Wellicht in een latere versie maak ik daar ook nog aan en uit schakelaars voor zodat je helemaal geen code hoeft aan te passen. ;)

Solidjeuh schreef:En volgens mij mist deze vertaling nog in het ACP bestandje:

Code: Selecteer alles

	'TITLE'							        => 'Titel',
Nogmaals bedankt voor de extensie! :D
Graag gedaan en Thnx ga ik onderzoeken.

Re: [DEV] responsive banners

Geplaatst: 07 mar 2016, 20:37
door Solidjeuh
Super! daar had ik dus niet gekeken :oops: :lol:

== EDIT ==

Had ik dus wel gekeken, maar ik had 'shuffle' gedaan ipv true
Daarom dat het dus niet werkte :)

Re: [DEV] responsive banners

Geplaatst: 08 mar 2016, 18:53
door dmzx
Linken op het forum met app.php geven de banner niet weer. (Enable URL Rewriting Uit)

Zie: Image

Verander in listener.php onderstaande:

Code: Selecteer alles

public function add_page_footer_slider($event)
	{
		if ($this->auth->acl_get('u_viewbanners'))
		{
			$board_url = generate_board_url();
			
			$sql = 'SELECT file_name, img_alt, img_title, img_url FROM ' . $this->tr_banners_table . ' ORDER BY order_img';
			$result = $this->db->sql_query($sql);
			while ($row = $this->db->sql_fetchrow($result))
			{
				$this->template->assign_block_vars('banners', array(
					'SLIDE_IMG'      => $board_url .'/images/responsivebanners/'. $row['file_name'],
					'IMG_ALT'		=> $row['img_alt'],
					'IMG_TITLE'		=> $row['img_title'],
					'IMG_URL'		=> $row['img_url']
				));
			}
			$this->db->sql_freeresult($result);
		}
	}
De $board_url = generate_board_url(); is toegevoegd

grtz

Re: [DEV] responsive banners

Geplaatst: 08 mar 2016, 19:30
door Theriddler
dmzx schreef:Linken op het forum met app.php geven de banner niet weer. (Enable URL Rewriting Uit)

Zie: Image

Verander in listener.php onderstaande:

Code: Selecteer alles

public function add_page_footer_slider($event)
	{
		if ($this->auth->acl_get('u_viewbanners'))
		{
			$board_url = generate_board_url();
			
			$sql = 'SELECT file_name, img_alt, img_title, img_url FROM ' . $this->tr_banners_table . ' ORDER BY order_img';
			$result = $this->db->sql_query($sql);
			while ($row = $this->db->sql_fetchrow($result))
			{
				$this->template->assign_block_vars('banners', array(
					'SLIDE_IMG'      => $board_url .'/images/responsivebanners/'. $row['file_name'],
					'IMG_ALT'		=> $row['img_alt'],
					'IMG_TITLE'		=> $row['img_title'],
					'IMG_URL'		=> $row['img_url']
				));
			}
			$this->db->sql_freeresult($result);
		}
	}
De $board_url = generate_board_url(); is toegevoegd

grtz
Dat heb ik niet meegenomen in mijn test, ik had app.php uitstaan. :roll:
Bedankt voor de toevoeging/verbetering dmzx. Zal deze zo snel mogelijk toevoegen.

Edit:
Extensie geupdate V1.0.6
- Theriddler

Re: [DEV] responsive banners

Geplaatst: 12 aug 2017, 20:41
door Froddelaar
Enkele foutjes in de vertaling

Code: Selecteer alles

'De %s is met success geladen.',
veranderen naar:
'De %s is met succes geladen.',

Code: Selecteer alles

'Sorry, het bestand bestaat rreds.',
veranderen naar:
'Sorry, het bestand bestaat reeds.',

Code: Selecteer alles

'Illegal type: Bestand is geen afbeelding.',
verandern naar:
'Illegaal type: Bestand is geen afbeelding.',

Code: Selecteer alles

'bestand %s os met success verwijderd.',
veranderen naar:
'bestand %s is met succes verwijderd.',
Ik probeer ook mijn website html5 te maken.
Maar als ik bij deze extensie een css regel aanmaak om het te centreren werkt het niet.
De banner komt niet perfect in het midden te staan.. weet jij een oplossing?

Code: Selecteer alles

.customcenter {
    text-align: center;
}

Re: [DEV] responsive banners

Geplaatst: 21 aug 2017, 07:23
door Theriddler
zo te zien is het je al gelukt ? zoniet laat het dan nog even weten.

Re: [DEV] responsive banners

Geplaatst: 21 aug 2017, 13:28
door Froddelaar
Theriddler schreef:
21 aug 2017, 07:23
zo te zien is het je al gelukt ? zoniet laat het dan nog even weten.
Neen, ik heb de standaard <center> terug geplaatst, want lukte me niet..

Re: [DEV] responsive banners

Geplaatst: 22 aug 2017, 08:08
door Theriddler
Froddelaar schreef:
21 aug 2017, 13:28
Theriddler schreef:
21 aug 2017, 07:23
zo te zien is het je al gelukt ? zoniet laat het dan nog even weten.
Neen, ik heb de standaard <center> terug geplaatst, want lukte me niet..
Als je de <center> weghaald
en in het bestand: /ext/theriddler/responsivebanners/styles/all/theme/responsiveslides.css

het volgende veranderd:

Code: Selecteer alles

.rslides_container {
  position: relative;
  /*float: left;*/
  width: 100%;
  /*! max-width: 900px; */
  }
Naar:

Code: Selecteer alles

.rslides_container {
  position: relative;
  width: 100%;
  }
Zou het gecentreerd moeten zijn.

Re: [DEV] responsive banners

Geplaatst: 22 aug 2017, 11:52
door Froddelaar
Yeps dat werkt! Bedankt!
Het is nu ook html5 valid :)

Re: [DEV] responsive banners

Geplaatst: 30 sep 2017, 10:20
door nl1sms
Leuke extensie. Ik kan alleen nergens de hoogte van een banner instellen en ik kan geen banner in de overall_header krijgen ?!?! Hij zet ze alleen maar in de footer ??? kan ik dit op de een of andere manier fixxen? thx

edit: ik ben bang dat ik niets aan deze extensie heb :( meerdere banners slide hij en ik wil er gewoon 4 onder elkaar hebben zonder slide.

edit2: als je de extensie uitschakelt blijft hij zichtbaar in de ACP bij extensies:
ACP_RESPONSIVE_BANNERS

ACP_RESPONSIVE_BANNERS_MANAGE